Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 1.10.16

Bug #18364 date.timezone errors for sh/bat files
Submitted: 2011-03-15 22:48 UTC
From: rquadling Assigned: dufuz
Status: Closed Package: PEAR (version SVN)
PHP Version: 5.3.5 OS: Windows XP SP3
Roadmaps: (Not assigned)    
Subscription  


 [2011-03-15 22:48 UTC] rquadling (Richard Quadling)
Description: ------------ Having installed pear and just released my first pecl package (win32service), the pecl.bat script fires out the date.timezone warning. No karma to commit patches to pear-core, so here is the patch. It is simply forcing GMT.

Comments

 [2011-03-15 22:48 UTC] rquadling (Richard Quadling)
 [2011-03-15 22:57 UTC] dufuz (Helgi Þormar Þorbjörnsson)
-Assigned To: +Assigned To: dufuz
 [2011-03-15 22:57 UTC] dufuz (Helgi Þormar Þorbjörnsson)
Thanks, I'm going to do this a different way. Forcing GMT is a last ditch resort IMO.
 [2011-03-15 23:24 UTC] rquadling (Richard Quadling)
Agreed, but http://news.php.net/php.pecl.dev/8058 and I do agree. But, if possible, go-pear.phar should be reading the date.timezone during install time and setting it in the pecl.bat / pecl.sh file. But any mechanism to get the date.timezone in is going to be fine really.
 [2011-03-19 22:35 UTC] dufuz (Helgi Þormar Þorbjörnsson)
-Summary: date.timezone errors for pecl.bat +Summary: date.timezone errors for sh/bat files
 [2011-03-19 23:00 UTC] dufuz (Helgi Þormar Þorbjörnsson)
-Summary: date.timezone errors for sh/bat files +Summary: date.timezone errors for sh/bat files when TZ is not set in php.ini -Status: Assigned +Status: Closed
This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better. I decided to force the date to UTC; while trying to figure out the TZ in go-pear et all, it has the danger of just being plain wrong or not being used by other distributions. The only danger is in the package validation, when we are comparing if the date of your XML is today when your real TZ may be still in yesterday or in fact tomorrow in comparison to UTC but I do believe it is a fair compromise anyway.
 [2011-05-10 01:54 UTC] tyrael (Ferenc Kovacs)
the description is bogus, I got those warnings with the correct timezone set in my php.ini for me, the correct behavior would be to pass the date.timezone with -d to the php script which gets executed with -n (hence it won't read my php.ini) Tyrael
 [2011-05-10 03:01 UTC] rquadling (Richard Quadling)
-Summary: date.timezone errors for sh/bat files when TZ is not set in php.ini +Summary: date.timezone errors for sh/bat files
I didn't notice the change in the subject. Setting the timezone in any php.ini file will make no difference to the reported bug as the -n option is used to call PHP, thereby suppressing any php.ini file.