Issue #9794 has been updated by Akira Tanaka.


tadayoshi funaba wrote:

> とりあえず、'%z' のみとかはダメらしいですから。

そうですね。
Time.strptime("+09:00", "%z") も受け付けていたので、原作者の意図するものではないということで変えてみました。

> ついでだけど、Time の変更がこのまま進むなら to_time とかも直したほうがいいよ。
> 元々無理のある仕様だったけど、もう完全にダメになりそう。

どうダメなのかわからないのでなんともいいようがない、というのが私の正直な感想です。


----------------------------------------
Feature #9794: DateTime.strptime() doesn't work correctly for '%s %z'
https://bugs.ruby-lang.org/issues/9794#change-46533

* Author: Felipe Contreras
* Status: Open
* Priority: Low
* Assignee: tadayoshi funaba
* Category: ext
* Target version: 
----------------------------------------
Time.strptime() works correctly:

    Time.strptime('0 +0100', '%s %z').strftime('%s %z')
    => "0 +0100"

But DateTime.strptime() doesn't:

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

In Rubinious it does work correctly:

    DateTime.strptime('0 +0100', '%s %z').strftime('%s %z')
    => "0 +0100"

This make the RubySL date space fail:

    DateTime#strptime parses seconds and timezone correctly FAILED
    Expected "1970-01-01T00:00:00+00:00"
     to equal "1970-01-01T01:00:00+01:00"

In addition, both C and perl preserver the offset correctly when doing '%s %z'.

So it's very clear DateTime.strptime() has to be fixed.

Patch attached.

---Files--------------------------------
0001-datetime-fix-strptime-s-z.patch (1.94 KB)


-- 
https://bugs.ruby-lang.org/