Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:81260
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.192.50 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <alpine.DEB.2.11.1501270934100.4080@whisky.home.derickrethans.nl>
References: <BLU437-SMTP380D5038EBE4EF66BA1857E2320@phx.gbl>
	<CAJW__o1Z5T13P5HFAXL4kYNaW8x=bOGpD9oJDnzGDWKv3P+mAA@mail.gmail.com>
	<CAGa2bXaLTSN7xNKxmWfdCoyt3EKqJxbQsYASus-LY+w-AST6ag@mail.gmail.com>
	<CAEZPtU7C=7PHUqn9CdK60Mav2wn_Mj1P0fv9--Dk4ntNRPwjcw@mail.gmail.com>
	<alpine.DEB.2.11.1501270934100.4080@whisky.home.derickrethans.nl>
Date: Wed, 28 Jan 2015 03:16:27 +0700
Message-ID: <CAEZPtU4+uLTZzeocs--6MD-yrSW+FsSn2rJ=wbnUw=wF+DjzZw@mail.gmail.com>
To: Derick Rethans <derick@php.net>
Cc: Bob Weinand <bobwei9@hotmail.com>, Yasuo Ohgaki <yohgaki@ohgaki.net>, 
	Kalle Sommer Nielsen <kalle@php.net>, PHP internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7bea30cc0973c2050da7ee62
Subject: Re: [PHP-DEV] [RFC] Remove the date.timezone warning
From: pierre.php@gmail.com (Pierre Joye)

--047d7bea30cc0973c2050da7ee62
Content-Type: text/plain; charset=UTF-8

On Jan 27, 2015 4:36 PM, "Derick Rethans" <derick@php.net> wrote:
>
> On Tue, 27 Jan 2015, Pierre Joye wrote:
>
> > On Jan 27, 2015 11:25 AM, "Yasuo Ohgaki" <yohgaki@ohgaki.net> wrote:
> > >
> > > On Tue, Jan 27, 2015 at 12:56 PM, Kalle Sommer Nielsen <kalle@php.net>
> > > wrote:
> > >
> > > > I think the warning is fair as it is, if it is annoying for small
> > > > use cases like on the CLI then simply: php -d date.timezone=UTC -r
> > > > "echo date('H:i:s');" or the dirty way by using the silent
> > > > operator. It used to be a notice prior 5.3 I think or something.
> > > >
> > > > While I do agree that most applications should be using UTC, which
> > > > should be set by default we sometimes need to tell userland the
> > > > hard way how things work, and/or what they should care about, like
> > > > the E_WARNING, E_CORE_ERROR, E_DEPRECATED for old php.ini
> > > > settings.
> > > >
> > > > -1 for removing it from my side.
> > >
> > > I can understand your argument. Perhaps, we may reconsider to
> > > introduce E_DEBUG/E_USER_DEBUG for these purposes. There are many
> > > functions, e.g. file related, that I feel E_WARNING is excessive.
> >
> > I do not have a strong opinion on that. So keep it or make it UTC
> > default but please do not add yet another warning/notice/whatever.
> >
> > Also, setting a timezone is not about Dev or other fancy tasks, it is
> > about making datetime processing right.
> >
> > If anything I would enforce the default at configure/build time. So it
> > at least gets the correct one from a host point of view.
>
> That would be nice, but it's unfortuntately not really possible. You
> can't find out what timezone the OS is using in a portable way. Not even
> among different linux distributions, let alone having Windows in the
> mix. We tried this before, and because it caused so many issues, the
> current warning was added (instead of trying to guess a timezone).

Either we get a valid one at install time or we bail out during make
install. Do not worry about windows here, no biggie either.

I am not saying we must do that but it is by far friendlier that we do now
or what it is proposed.

> cheers,
> Derick

--047d7bea30cc0973c2050da7ee62--