[Bug 1734967] Re: tzdata info for WGT/WGST broken

2018-11-07 Thread Bug Watch Updater
** Changed in: php7.0 (Debian) Status: New => Fix Released -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1734967 Title: tzdata info for WGT/WGST broken To manage notifications about this

Re: [Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-04 Thread Thomas M Steenholdt
I have started a dialogue on the TZ mailinglist to perhaps have the change (WGT/WGST at least) reverted. That's my goal anyway, so we'll see how it goes. /Thomas -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu.

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-04 Thread Eric Desrochers
The more I look into this the more I start to think it may not be a php bug as I first guess. php is relying on tzdata package so it should follow what tzdata bundle has in hand. I wonder if the change I mentioned in Comment#1 is valid and/or if it has the appropriate informations for

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-03 Thread Thomas M Steenholdt
Hi guys, The removal of WGT/WGST from tzdata is crazy to me. Living in Greenland, I can imagine a bunch of different issues this will cause, so I'll try to figure out what is going on and what possibilities we have of rectifying it. Starting with upstream. /Thomas -- You received this bug

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-02 Thread Eric Desrochers
Looking at the php-src git log (PHP upstream). I notice PHP is updated every time there is a new tzdata version: 56269af46c Updated to version 2017.2 (2017b) 1f9e5cc2bc Updated to version 2017.2 (2017b) eb13003f6c Updated to version 2017.2 (2017b) root@phpbisect:~/php7/php-src# git log --oneline

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-02 Thread Eric Desrochers
Nish, Yes the Ubuntu php build is configure with tzdata as follow : debian/rules: --with-system-tzdata and tzdata is a Depends: for a quit some derived binary package of php. (The above is observation from bionic php7.1 source pkg) # debian/php-common.README.Debian

Re: [Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-01 Thread Nish Aravamudan
Does the Ubuntu php build use the bundled tzdata? A lot of stuff comes from the system not the source (e.g, pcre iirc) On Dec 1, 2017 18:40, "Eric Desrochers" wrote: > Same thing with new php pkg release in debian/unstable (php - 7.1.12-1). > It still displays

Re: [Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-01 Thread Nish Aravamudan
Does the Ubuntu php build use the bundled tzdata? A lot of stuff comes from the system not the source (e.g, pcre iirc) On Dec 1, 2017 18:40, "Eric Desrochers" wrote: > Same thing with new php pkg release in debian/unstable (php - 7.1.12-1). > It still displays

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-01 Thread Eric Desrochers
Same thing with new php pkg release in debian/unstable (php - 7.1.12-1). It still displays America/Sao_Paulo That exclude my theory about the tz bundle update. I'll do a proper bisect next week. - Eric -- You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-01 Thread Eric Desrochers
I just notice a new version of php in debian was release --> 7.1.12-1 (sid) I'll give it a try today and see if problem still persist or not. This one should include the tz bundle change. https://buildd.debian.org/status/logs.php?pkg=php7.1 - Eric -- You received this bug notification because

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-12-01 Thread Eric Desrochers
Nish, Sorry if I didn't mention it earlier but I am able to reproduce in debian/unstable and ubuntu/bionic with php7.1. # lsb_release -a No LSB modules are available. Distributor ID: Ubuntu Description:Ubuntu Bionic Beaver (development branch) Release:18.04 Codename: bionic #

Re: [Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Nish Aravamudan
Is this already fixed in proposed (MRE to latest upstream php 7.0 and 7.1)? On Dec 1, 2017 03:48, "Bug Watch Updater" <1734...@bugs.launchpad.net> wrote: > ** Changed in: php7.0 (Debian) >Status: Unknown => New > > -- > You received this bug notification because you are subscribed to

Re: [Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Nish Aravamudan
Is this already fixed in proposed (MRE to latest upstream php 7.0 and 7.1)? On Dec 1, 2017 03:48, "Bug Watch Updater" <1734...@bugs.launchpad.net> wrote: > ** Changed in: php7.0 (Debian) >Status: Unknown => New > > -- > You received this bug notification because you are subscribed to

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Bug Watch Updater
** Changed in: php7.0 (Debian) Status: Unknown => New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1734967 Title: tzdata info for WGT/WGST broken To manage notifications about this bug go

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
I'll try to do more testing to conclude my assumption and see if anything else is needed. Meanwhile I have reported a bug in debbug : https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=883209 - Eric -- You received this bug notification because you are a member of Ubuntu Bugs, which is

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
I try to do more testing to conclude my assumption and see if anything else is needed. Meanwhile I have reported a bug in debbug : https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=883209 - Eric -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
** Bug watch added: Debian Bug tracker #883209 https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=883209 ** Also affects: php7.0 (Debian) via https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=883209 Importance: Unknown Status: Unknown -- You received this bug notification because

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
** Also affects: php7.1 (Ubuntu) Importance: Undecided Status: New -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1734967 Title: tzdata info for WGT/WGST broken To manage notifications

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
At code inspection I have strong believe this commit is the one changing the behaviour between comment #2 (America/Sao_Paulo) and comment#6 (UTC) * 8e3260376c Update timezonemap.h, which needs to match commit 8e3260376c65b762a51eee7c30ab560dadefa1f6 Author: Derick Rethans

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-30 Thread Eric Desrochers
I have compile the php upstream source code from : https://github.com/php/php-src # php --version PHP 7.3.0-dev (cli) (built: Nov 30 2017 14:52:31) ( NTS ) Copyright (c) 1997-2017 The PHP Group Zend Engine v3.3.0-dev, Copyright (c) 1998-2017 Zend Technologies # date Thu Nov 30 12:23:30 -03 2017

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Hans Joachim Desserud
** Tags added: artful xenial -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1734967 Title: tzdata info for WGT/WGST broken To manage notifications about this bug go to:

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Eric Desrochers
Thomas, I think the next step here is to verify if latest and greatest php upstream version have the same behaviour with the above tzdata upstream change. This will tell us if the situation has been already addressed (intentionally or by happenstance) upstream or not. I'm also afraid this

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Eric Desrochers
Took from Ubuntu php7 source code : # ext/date/lib/timezonemap.h 1718 { "wgst", 1, -7200, "America/Godthab" }, 1719 { "wgst", 1, -7200, "America/Danmarkshavn" }, 1720 { "wgt", 0, -10800, "America/Godthab" }, Clearly php is

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Nish Aravamudan
Just to be clear, this isn't a bug based upon comment 1 but new tasks were opened in comment 2? If it's purely an SRU regression in tzdata that should be figured out. Im guessing this is an upstream php issue. Has it been fixed there? -- You received this bug notification because you are a

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Nish Aravamudan
Just to be clear, this isn't a bug based upon comment 1 but new tasks were opened in comment 2? If it's purely an SRU regression in tzdata that should be figured out. Im guessing this is an upstream php issue. Has it been fixed there? -- You received this bug notification because you are a

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Eric Desrochers
I'll set this bug as affecting php package, since this tzdata change is modifying the behaviour of php timezone. ## REPRODUCER ## ubuntu@trusty:~# sudo ln -sf /usr/share/zoneinfo/America/Godthab /etc/localtime ubuntu@trusty:~# reboot ubuntu@trusty:~# ls -altr /etc/localtime lrwxrwxrwx 1 root

[Bug 1734967] Re: tzdata info for WGT/WGST broken

2017-11-29 Thread Eric Desrochers
Hi Thomas, I have verified and I confirm this is not a bug, this change was done in tzdata upstream intentionally. If you think this change cause a regression, the first step would be to report it to tzdata upstream. commit 2999bb5bee719acbba8b9dd50fb9fb00c7788623 Author: Paul Eggert