Project

General

Profile

Actions

Bug #7445

closed

strptime('%s %z') doesn't work

Added by felipec (Felipe Contreras) over 11 years ago. Updated over 10 years ago.

Status:
Rejected
Target version:
ruby -v:
ruby 1.9.3p327
Backport:
[ruby-core:50207]

Description

See:

ruby -e "require 'date'; p DateTime.strptime('0 +0100', '%s %z').strftime('%s %z')"

The timezone is ignored.


Files

bug-7445.patch (1.34 KB) bug-7445.patch Anonymous, 11/27/2012 10:33 PM
0001-time-fix-strptime.patch (1.44 KB) 0001-time-fix-strptime.patch felipec (Felipe Contreras), 10/06/2013 10:17 AM

Related issues 1 (0 open1 closed)

Related to Ruby master - Feature #9794: DateTime.strptime() doesn't work correctly for '%s %z'Closedtadf (tadayoshi funaba)05/02/2014Actions

Updated by Anonymous over 11 years ago

I've attached a patch that fixes this.

Updated by usa (Usaku NAKAMURA) over 11 years ago

  • Category set to lib
  • Status changed from Open to Assigned
  • Assignee set to tadf (tadayoshi funaba)
  • Target version set to 2.0.0

Updated by tadf (tadayoshi funaba) over 11 years ago

  • Priority changed from Normal to 3
  • Target version changed from 2.0.0 to 3.0

this is intentional behaviour.
%s cannot represent localtime with offset.
%s implies utc.
this is just odd as a date.

but, it's an option.
we may be able to adopt this feature on next major.
i'm not sure at the present time.

Updated by felipec (Felipe Contreras) almost 11 years ago

It works perfectly fine in C:

./test '%s' '%s %z'
1369626774
1369626774 -0500
#include <stdio.h>
#include <time.h>

int main(int argc, char *argv[]) {
	struct tm *tm;
	time_t t;
	char buf[0x100];
	t = time(NULL);
	tm = localtime(&t);
	for (int i = 0; i < argc; i++) {
		strftime(buf, sizeof(buf), argv[i], tm);
		printf("%s\n", buf);
	}
	return 0;
}

Updated by tadf (tadayoshi funaba) almost 11 years ago

DateTime also does.

$ ruby -r date -e 'dt=DateTime.now; ARGV.each{|f| p dt.strftime(f)}' '%s' '%s %z'
"1369651261"
"1369651261 +0900"

so what?

Updated by felipec (Felipe Contreras) almost 11 years ago

tadf (tadayoshi funaba) wrote:

DateTime also does.

$ ruby -r date -e 'dt=DateTime.now; ARGV.each{|f| p dt.strftime(f)}' '%s' '%s %z'
"1369651261"
"1369651261 +0900"

so what?

./test '0 +0100' '%s' '%s %z'
0
0 +0100
#define _XOPEN_SOURCE
#include <stdio.h>
#include <time.h>

int main(int argc, char *argv[]) {
	struct tm tm;
	char buf[0x100];
	for (int i = 2; i < argc; i++) {
		strptime(argv[1], argv[i], &tm);
		strftime(buf, sizeof(buf), argv[i], &tm);
		printf("%s\n", buf);
	}
	return 0;
}

Updated by tadf (tadayoshi funaba) almost 11 years ago

but you can't preserve offset with mktime(3).
if you want substring which denote %z, you can call _strptime.

DateTime._strptime('0 +0100', '%s %z') #=> {:seconds=>0, :zone=>"+0100", :offset=>3600}

you told about C's stdlib.
why not talk about ruby's Time?
Time is based on stdlib.
why not want it with Time?

Updated by tadf (tadayoshi funaba) over 10 years ago

  • Status changed from Assigned to Rejected

there is no evidence of bug.

Updated by felipec (Felipe Contreras) over 10 years ago

tadf (tadayoshi funaba) wrote:

but you can't preserve offset with mktime(3).

We are not talking about mktime(3), we are talking about strptime(). This is a red herring argument.

The documentation of Date.strptime() clearly mentions C's strptime(3) and strftime:

"See also strptime(3) and strftime."

And both of them work perfectly fine with '%s %z'. I showed in the code above how C's strptime works with '%s %z', and here I'm showing Ruby's strftime does as well:

require 'date'
d = DateTime.new(1970,1,1,1,0,0,'+01:00')
d.strftime('%s %z')
=> "0 +0100"

So both strptime(3) and DateTime.strftime() accept '%s %z', and Charlie Somerville attached a patch that can be applied on top of Ruby's trunk, and DateTime.strptime() would work just like strptime(3), and would be consistent with DateTime.strftime().

What reason could you possibly have to not accept this patch? What use-case would it break? How could it possibly affect negatively anybody?

If you truly think '%s %z' does not make sense, then throw an error when both %s and %z are used at the same time, both in DateTime.strptime(), DateTime._strptime(), and DateTime.strftime(). Also, update the documentation to mention that it's not exactly the same as strptime(3), and neither strftime(3). This of course would be totally stupid.

Just accept the patch and everything will be consistent.

if you want substring which denote %z, you can call _strptime.

No, I want strptime to work correctly.

you told about C's stdlib.
why not talk about ruby's Time?
Time is based on stdlib.
why not want it with Time?

That English does not parse.

Updated by felipec (Felipe Contreras) over 10 years ago

tadf (tadayoshi funaba) wrote:

DateTime also does.

$ ruby -r date -e 'dt=DateTime.now; ARGV.each{|f| p dt.strftime(f)}' '%s' '%s %z'
"1369651261"
"1369651261 +0900"

so what?

So you accept strftime() works correctly with '%s %z'? Then why do you not accept that strptime() doesn't?

Updated by felipec (Felipe Contreras) over 10 years ago

tadf (tadayoshi funaba) wrote:

this is intentional behaviour.

Then update the documentation to say so.

%s cannot represent localtime with offset.

Yes it can. I've shown how it can in C strptime(3), and even you showed that it works with DateTime.strftime(3).

but, it's an option.
we may be able to adopt this feature on next major.
i'm not sure at the present time.

The patch is right there and it works perfectly, why aren't you applying it?

If you don't reopen this I'll escalate the issue through different channels.

Updated by tadf (tadayoshi funaba) over 10 years ago

俺の英語は通用しないらしいから日本語で書いてあげます。

あなたこそ DateTime の話をしている事を忘れないで下さい。

参照でその先に書いてある事がそのまま通用するわけではありません。参照は
理解を深めるための参考に過ぎません。当たり前です。常識です。

more(1) より引用

SEE ALSO
vi(1), less(1)

moreless と等価か? 並外れてますね。

そもそも、もう言った筈ですけれど、あなたが言うように '%s %z' が完璧に機
能するなんて事はありません。あなたが言っているのは、構文レベルで「読め
た」というだけでしょう。それは、_strptime で出来ますと説明しました。そ
もそも C の場合、タイムゾーンのメンバーは標準ではありません。元からタイ
ムゾーンや時差について対称でありません。なんなら、ruby の組み込みクラス
Time でどのようにやるのか見せて下さい。

lcoaltime(3) より引用

          struct tm {
              int tm_sec;         /* seconds */
              int tm_min;         /* minutes */
              int tm_hour;        /* hours */
              int tm_mday;        /* day of the month */
              int tm_mon;         /* month */
              int tm_year;        /* year */
              int tm_wday;        /* day of the week */
              int tm_yday;        /* day in the year */
              int tm_isdst;       /* daylight saving time */
          };

DateTime だけでなく、組み込みクラスの Time も、C のライブラリも同じです
が、正しく受け入れるには、構文レベルだけでなく、その意味を解釈しなけれ
ばなりません。矛盾する要素や最初から当てにしていないものは無視される事
になります。あなたはまったくそこに気付かないふりをして胡麻化しています。

%stime_t を意図していますが、struct tm にはそのようなメンバーはない
ので実際には、分解されて自明の地方時として格納される筈です。仮に時差が
格納されていたとしてもそれは少しおかしい事だと気づなかければいけません。
その時差が地方時のそれと一致していればいいでしょうが、そうとは限りませ
ん。そもそも格納されるかどうかは実装次第でしょう。mktime(3) がそれを無
視すれば辻褄が合います。いずれにしてもこのあたりの振舞いは未定義だと思
います。俺達が議論する意義はないです。

#define _XOPEN_SOURCE
#include <stdio.h>
#include <time.h>

int main(int argc, char *argv[]) {
  struct tm tm;
  char buf[100];
  strptime(argv[1], argv[2], &tm);
  strftime(buf, 100, "%F %T %z", &tm);
  printf("%s\n", buf);
  return 0;
}
$ ./a.out '3 +0100' '%s %z'
1970-01-01 09:00:03 +0100
$ ./a.out '3 -0100' '%s %z'
1970-01-01 09:00:03 -0100

この場合、たしかに時差は読みこまれてはいました。

$ date -d '1970-01-01 09:00:03 +0100' +%s
28803
$ date -d '1970-01-01 09:00:03 -0100' +%s
36003

でも、ただ読んでるだけです。対称性がどうとかいう話じゃないんです。

DateTimeTime のインスタンスはオブジェクトなので、mktime(3) 的なもの
を排除できません。矛盾する要素を孕んでそのままにしておく事は DateTime
Time ではありえません。オブジェクトは struct tm ではありません。あなた
が言っている事は実際には C でも機能していないし、DateTime でもそのよう
な用意をするつもりもないです。あなたの言ってる事はてんで出鱈目です。

> Time.strptime('1 +1200', '%s %z')
#=> 1970-01-01 09:00:01 +0900
> Time.strptime('1 +2400', '%s %z')
#=> 1970-01-01 09:00:01 +0900

実際の所、意味のない組み合わせは他にもあります。それに対して、いちいち
正しい解釈を用意しようとは思いません。この件であなたの言うように修正す
れば、それで完璧な対称性が得られるわけではないのです。

> Time.strptime('1 0', '%s %w').wday
#=> 4

> Time.strptime('1 PM', '%s %p').hour
#=> 9

> DateTime.new(2001,2,3,16,5,6).strftime('%F %T %p')
#=> "2001-02-03 16:05:06 PM"
> DateTime.strptime('%F %T %p', "2001-02-03 16:05:06 PM")
ArgumentError: invalid date

俺は '%s %z' なんて書式の日付は見た事がありません。タイムゾーンや時差が
必要なのは、本質的にそれが地方時だからでしょう。%s が示しているのは、
time_t で、time_t にはタイムゾーンや時差の情報はありません。Unix ではそ
れは所与のものとして、自明なもの、説明する必要のないものとして、ただひ
とつあるだけ、これが基本です。

たとえば、ISO 8601 を見てみましょう。地方時に時差はあるが協定世界時には
ないでしょう。これを積極的に受け入れる理由がわかりません。少くともバグ
ではありません。あなたが受け入れて当然のように言う根拠がわかりません。

俺はタイムゾーンや時差は修飾語のようなものだと思っていますが、'%s %z'
では対象になるものがありません。'%s %p' なんかも同じです。こういうもの
に意味を与えなければならないとは思いません。

あなたが表現しようとしている '%s %z' は日付としておかしいと思います。そ
れを無理に解釈するならば、日付ではなくて、日付オブジェクトの不完全なコ
ピーのようなものにすぎないと思います。

それから、time_t は閏秒を含む場合があるので、移植性が乏しいです。だから、
これが色々な所で読み書き出来たとしても、あまり積極的に利用を勧められる
ようなものではないという事も言っておきたいです。ruby の組み込みクラスに
対してすら通用しない事を無理してやる意義を感じません。新しい機能として
ならまだしも「これはバグでした。バグが直ったからドンドンつかって下さい」
とは絶対に言いたくありません。

あるいは将来、未定義の振舞いとして有り得るのかもしれません。しかし、今
の所、受け入れたいとは全く思いません。逆に今すぐ受け入れなければならな
い理由はあるのでしょうか。今まで誰にも文句を言われた事はありません、あ
なた以外には。

コードを見ればわかりますが、意識的に無視しています。意図してやってます。
今も考えは変っていませんし、あなたに説得される気がしません。これで納得
できない場合は他のエライ人に相談して下さい。

俺の結論は「これはバグじゃない」です。

Updated by felipec (Felipe Contreras) over 10 years ago

tadf (tadayoshi funaba) wrote:

I will give written in Japanese English because I heard not hold water .

I've used Google translate which does a poor job, but you have left me no choice.

Do not forget that you're talking about what you DateTime.

That is written to the destination in the reference is not necessarily as it is Spoken. References
It is not just a reference for a better understanding. Is natural. Is common sense.

Totally unparsable.

quoted from more(1)

SEE ALSO
vi(1), less(1)

or equivalent to less? more I have exceptional.

First of all , you should you have said already , but the machine perfectly '%s z%' as you say
Nothing to be is! Performance . The syntax level you 're talking about is , read "
It will only say "he . It has been described as possible with _strptime. Its
For C, the members of the time zone is not a standard Somo also . Thailand from the original
It is not symmetric about the time difference and time zones . What if , built-in classes of ruby
Show How do you do in Time.

quoted from (3) lcoaltime

          struct tm {
              int tm_sec;         /* seconds */
              int tm_min;         /* minutes */
              int tm_hour;        /* hours */
              int tm_mday;        /* day of the month */
              int tm_mon;         /* month */
              int tm_year;        /* year */
              int tm_wday;        /* day of the week */
              int tm_yday;        /* day in the year */
              int tm_isdst;       /* daylight saving time */
          };

I don't know what you are talking about, but we are not talking about Time, nor lcoaltime, which I pressume is actually localtime().

Time doesn't even have strptime(), so what's the point of bringing it up? I have no idea.

Well as DateTime, Time of built-in class , library of C also is the same
But, in order to properly accept , as well as the syntax level and have to interpret the meaning
Must . Those that do not rely on the first elements and conflicting be ignored
Will be . You're of sesame and pretend you do not notice it at all .

%s is intended to time_t , but there is no such a member to struct tm
In fact , it should be stored in some cases local self-evident is decomposed so . Even if time difference
I do not go , multiply Dzuna mind it's a little funny thing is , even if it is stored.
The difference would be good if they match that of the local time , but it is not necessarily so
Do . It would be implemented as soon as the first place and whether it is stored . mktime (3) is free it
Coherence will fit if you view. Behavior of this area think that it is undefined in any
There are . Is not the significance of ours to discuss .

#define _XOPEN_SOURCE
#include <stdio.h>
#include <time.h>

int main(int argc, char *argv[]) {
  struct tm tm;
  char buf[100];
  strptime(argv[1], argv[2], &tm);
  strftime(buf, 100, "%F %T %z", &tm);
  printf("%s\n", buf);
  return 0;
}
$ ./a.out '3 +0100' '%s %z'
1970-01-01 09:00:03 +0100
$ ./a.out '3 -0100' '%s %z'
1970-01-01 09:00:03 -0100

In this case, the difference was to be read indeed .

$ date -d '1970-01-01 09:00:03 +0100' +%s
28803
$ date -d '1970-01-01 09:00:03 -0100' +%s
36003

But , just I 'm just reading . Or something if symmetry 'm not a story .

Instance of Time and DateTime is an object, what mktime (3 ) basis
You can not eliminate . I should have to as it is fraught elements inconsistent or DateTime
It is not impossible in Time. Object is not a struct tm. You
C still does not work , in fact that you are saying , and so any DateTime
There is not even going to prepare Na . The fact that you said is nonsense at all .

Again, Time is irrelevant, we are talking about DateTime, so mktime() or whatever backend is used is irrelevant, the documentation mentions strptime(3), and strptime(3) accepsts '%s %z', therefore DateTime.strptime() should accept it too, and it can if the attached patch is applied.

> Time.strptime('1 +1200', '%s %z')
#=> 1970-01-01 09:00:01 +0900
> Time.strptime('1 +2400', '%s %z')
#=> 1970-01-01 09:00:01 +0900

Again, there is no strptime in Time.

Time.strptime('1 +1200', '%s %z')
NoMethodError: undefined method `strptime' for Time:Class
	from (irb):1
	from /usr/bin/irb:12:in `<main>'

In fact , there are other combinations do not make sense . On the other hand , one by one
I do not think it is trying to provide a correct interpretation . And be modified as you say in this matter
Lever , so although I have not yet obtained the perfect symmetry .

> Time.strptime('1 0', '%s %w').wday
#=> 4

> Time.strptime('1 PM', '%s %p').hour
#=> 9

> DateTime.new(2001,2,3,16,5,6).strftime('%F %T %p')
#=> "2001-02-03 16:05:06 PM"
> DateTime.strptime('%F %T %p', "2001-02-03 16:05:06 PM")
ArgumentError: invalid date

That's because you are passing the arguments the other way around:

DateTime.strptime("2001-02-03 16:05:06 PM", '%F %T %p')
=> #<DateTime: 2001-02-03T16:05:06+00:00 ((2451944j,57906s,0n),+0s,2299161j)>

Geezus.

I have never seen that the date formatなんて's% z%' to me. Time difference and time zone
What is needed , it will be because it is local time in nature. % s 's shown ,
In time_t, there is no information of the time difference and time zone to time_t. Its on Unix
As given , self-explanatory , as there is no need to explain , just fire Re
Only some convex , this is the basic .

Again, time_t is totally and completely irrelevant. strptime(3) and strftime(3) do NOT use it.

For example, let's look at the ISO 8601. There a difference in local time , but the Universal time
Would not . I do not know why actively accept this . Bug at least
Not a . I do not know reason to say as a matter of course , you'd been accepted .

I have thought and time zone difference as something like modifier but , '% s% z'
In There is nothing of interest. something 's% p%' is the same . Such things
I do not think you must give a meaning to .

It doesn't matter what you think. strptime(3) does give a meaning to '%s %z'. Period.

I think it strange that as a date you 's% z%' is attempting to represent . Its
If interpret force the level , it is not a date , an incomplete copy of the date object
I think that it is not only a kind of copy .

It is not incomplete, every date can be represented with '%s %z'.

That's why Git, the most successful DVCS by far, which is headed to
replace Subversion as the most popular VCS in general uses precisely
this:

% git cat-file -p trunk
tree 075a08c3e84f214ab9426957b3682b94c56b3046
parent bbf366bad8b51c491820b67338125d36b88c8fbe
author zzak <zzak@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> 1380550541 +0000
committer zzak <zzak@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> 1380550541 +0000

* ext/objspace/objspace.c: [DOC] Cleaned up many rdoc formatting
  issues and several duplicate grammar bugs.



git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@43099 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Oh, look at that '1380550541 +0000', so '%s %z', that's all Git uses to
represent dates, and it's not missing any information. Probably hundreds
of thousands, maybe even millions of people don't have a problem with
these dates.

Oh, and since I've worked on Mercurial and Bazaar, I can tell you that
these two fields is all they need as well.

From it , because it may contain a leap second time_t, portability is poor . So ,
This is even able to read and write in various places , is recommended the use of less active
The fact that it will not also like I want to say . the built-in classes of ruby
I do not feel the significance that I'll overdo it that you do not Spoken and even for . A new feature
If " Use steadily . Bug was a bug because I grew out of it by this " Madashimo
I do not want to say absolutely is .

Or future , it might be The possible as undefined behaviors . However , now
I do not think at all of place , if you want to accept . It must be accepted right now to reverse
Will there why not . Never complains about anyone until now , Oh
Other than thou is .

You can see if you look at the code , but ignore consciously . Has been doing intended .
Thought does not Hen~tsu also now, I feel to be persuaded you will not . Consent this
Consult your great men of the other if you can not .

I don't understand what you mean, but it is a bug. The documentation says this.

"See also strptime(3) and strftime."

  1. strptime(3) does work correctly with '%s %z'

  2. strftime()

"This is not a bug " is the conclusion of me.

Well you are wrong, and your are completely unreasonable and uncooperative attitude is not helping, so I'm going to escalate to other channels.

Updated by felipec (Felipe Contreras) over 10 years ago

Also, if '%s' is used for UTC, then why do we get this?

require 'time'
Time.strptime('0', '%s')
=> 1969-12-31 18:00:00 -0600

I'm attaching a patch that does exactly what we want:

Time.strptime('0', '%s')
=> 1970-01-01 00:00:00 +0000
Time.strptime('0 +0100', '%s %z')
=> 1970-01-01 01:00:00 +0100

Updated by Anonymous over 10 years ago

+1 Patch looks good to me

Updated by felipec (Felipe Contreras) over 10 years ago

So let's recap the current situation.

Time.strptime() has been fixed in Ruby Core, and Rubinious has fixed strptime() in both Time, and DateTime in rubysl.

So we have this:

Ruby MRI

puts Time.strptime('1 +0100', '%s %z').strftime('%s %z')
=> 1 +0100

Rubinious

puts Time.strptime('1 +0100', '%s %z').strftime('%s %z')
=> 1 +0100

puts DateTime.strptime('1 +0100', '%s %z').strftime('%s %z')
=> 1 +0100

C

#define _XOPEN_SOURCE
#include <stdio.h>
#include <time.h>

int main(int argc, char *argv[])
{
  struct tm tm;
  char buf[0x100];
  strptime("1 +0100", "%s %z", &tm);
  strftime(buf, sizeof(buf), "%s %z", &tm);
  printf("%s\n", buf);
  return 0;
}
=> 1 +0100

Perl

use DateTime::Format::Strptime;

my $format = DateTime::Format::Strptime->new(
  pattern => '%s %z',
);

my $dt = $format->parse_datetime('1 +0100');
print $dt->strftime('%s %z'), "\n";
=> 1 +0100

And you still maintain that this is desirable?

puts DateTime.strptime('1 +0100', '%s %z').strftime('%s %z')
=> 1 +0000

Come on, just apply the patch.

Updated by felipec (Felipe Contreras) over 10 years ago

felipec (Felipe Contreras) wrote:

Time.strptime() has been fixed in Ruby Core, and Rubinious has fixed strptime() in both Time, and DateTime in rubysl.

I guess the lack of reply means that you won't do the sensible thing here, even though it makes sense to pretty much everyone else, you just don't want to accept you were wrong.

Updated by tadf (tadayoshi funaba) almost 10 years ago

  • Related to Feature #9794: DateTime.strptime() doesn't work correctly for '%s %z' added
Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0Like0