[Bug 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

bugzilla at redhat.com bugzilla at redhat.com
Thu Jan 13 23:45:18 UTC 2011


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=517191

--- Comment #29 from Christof Damian <christof at damian.net> 2011-01-13 18:45:17 EST ---
(In reply to comment #28)
> PHP Warning:  date(): It is not safe to rely on the system's timezone settings.
> You are *required* to use the date.timezone setting or the
> date_default_timezone_set() function. In case you used any of those methods and
> you are still getting this warning, you most likely misspelled the timezone
> identifier. We selected 'America/Chicago' for 'CST/-6.0/no DST' instead in
> /usr/share/pear/PEAR/Registry.php on line 1012
> 
> with a stack trace any time PHP gets called at all.  Since nobody else has
> complained of this I suspect I'm the only one seeing it.

No, this is fixable. We do it for other PHP packages too. Just people who
develop in PHP have usually set this in the global config already. I obviously
didn't check the mock logs.

> 
> The source in the srpm differs from the tarball I get from the upstream web
> site.

There might be a difference in the tarball from the website and the one from
the pear channel, I will check.

> I note that this spec manages to break vim's highlighting so everyhing after
> the first sed call in %install turns up pinkish-purple.  I guess it's the
> escaped single quotes in those sed lines.

I will also check if this is fixable.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.



More information about the package-review mailing list