Re: [PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-16 Thread Derick Rethans
On Mon, 16 Feb 2015, Yasuo Ohgaki wrote:

 On Mon, Feb 16, 2015 at 2:40 PM, Stanislav Malyshev smalys...@gmail.com
 wrote:
 
   Are you sure that the ini file is created by the package 
   maintainer and it includes the “date.timezone” setting is true? 
   Last time I checked it
 
  I my experience, it is unfortunately not universally true - I've 
  seen Linux distros with ini files that don't do that. Setting TZ 
  correctly would require some non-trivial script work, and I'm 
  guessing whoever did the package did not want to do this work.

In an earlier thread I posted what Debian needed to do. It's trivial.

  They could of course just put UTC there but that didn't happen 
  either.
 
 I agree.
 
 The difficulty is choosing the right one. Linux is multi user system 
 and users may have their own locale settings. System's default and 
 user's default may differ also. Distributor cannot choose the right 
 timezone automatically.
 
 If it's impossible to determine right one automatically, PHP shall 
 have reasonable default which would be UTC.

The only reasonable thing is to let people chose what they want. That 
means, we do *not* set the timezone by default.

If distribution maintainers want to pick something for their users, 
that's fine, but I am vehemently against removing this warning.

cheers,
Derick
-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php

[PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-15 Thread Bob Weinand
I'm probably going to put this into vote tomorrow.

The RFC got some update from neutrality aspect and covers some feedback from 
the initial discussion thread.

The link is:
https://wiki.php.net/rfc/date.timezone_warning_removal 
https://wiki.php.net/rfc/date.timezone_warning_removal

Bob

Re: [PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-15 Thread Matteo Beccati

On 15/02/2015 19:02, Bob Weinand wrote:

I'm probably going to put this into vote tomorrow.

The RFC got some update from neutrality aspect and covers some
feedback from the initial discussion thread.

The link is: https://wiki.php.net/rfc/date.timezone_warning_removal
https://wiki.php.net/rfc/date.timezone_warning_removal


Are you sure that the ini file is created by the package maintainer and 
it includes the “date.timezone” setting is true? Last time I checked it 
wasn't true across the board, so it's sounds more like wishful thinking 
at the moment. But I could be wrong, indeed.


The current behaviour is also not described properly. The warning is 
displayed the first time some date/time functionality is employed. And 
while I'd agree that generally speaking date.timezone is not the most 
important setting, in that very moment it most likely is.


Your particular problem could be easily fixed by using a common 
--with-prefix=/opt/php-build and putting the timezone in 
/opt/php-build/lib/php.ini, so I don't understand the urgency to change 
the behaviour for everyone... Anyway, let the voting begin ;)



Cheers
--
Matteo Beccati

Development  Consulting - http://www.beccati.com/

--
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-15 Thread Ole Markus With


On 02/15/2015 08:28 PM, Matteo Beccati wrote:
 On 15/02/2015 19:02, Bob Weinand wrote:
 I'm probably going to put this into vote tomorrow.

 The RFC got some update from neutrality aspect and covers some
 feedback from the initial discussion thread.

 The link is: https://wiki.php.net/rfc/date.timezone_warning_removal
 https://wiki.php.net/rfc/date.timezone_warning_removal
 
 Are you sure that the ini file is created by the package maintainer and
 it includes the “date.timezone” setting is true? Last time I checked it
 wasn't true across the board, so it's sounds more like wishful thinking
 at the moment. But I could be wrong, indeed.
 

Gentoo uses one of the php.ini files from the tarball without any
modifications.

-- 
Ole Markus

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-15 Thread Stanislav Malyshev
Hi!

 Are you sure that the ini file is created by the package maintainer and
 it includes the “date.timezone” setting is true? Last time I checked it

I my experience, it is unfortunately not universally true - I've seen
Linux distros with ini files that don't do that. Setting TZ correctly
would require some non-trivial script work, and I'm guessing whoever did
the package did not want to do this work. They could of course just put
UTC there but that didn't happen either.

-- 
Stas Malyshev
smalys...@gmail.com

-- 
PHP Internals - PHP Runtime Development Mailing List
To unsubscribe, visit: http://www.php.net/unsub.php



Re: [PHP-DEV] [RFC] [PRE-VOTE] Remove the date.timezone warning

2015-02-15 Thread Yasuo Ohgaki
I all,

On Mon, Feb 16, 2015 at 2:40 PM, Stanislav Malyshev smalys...@gmail.com
wrote:

  Are you sure that the ini file is created by the package maintainer and
  it includes the “date.timezone” setting is true? Last time I checked it

 I my experience, it is unfortunately not universally true - I've seen
 Linux distros with ini files that don't do that. Setting TZ correctly
 would require some non-trivial script work, and I'm guessing whoever did
 the package did not want to do this work. They could of course just put
 UTC there but that didn't happen either.


I agree.

The difficulty is choosing the right one. Linux is multi user system and
users
may have their own locale settings. System's default and user's default may
differ
also. Distributor cannot choose the right timezone automatically.

If it's impossible to determine right one automatically, PHP shall have
reasonable default which would be UTC.

Regards,

--
Yasuo Ohgaki
yohg...@ohgaki.net