I guess Drush should just specify UTC or whatever Unix time is uses. That would avoid the following error: "It is not safe to rely on the system's timezone ...."

Using php 5.3.3 on windows.

CommentFileSizeAuthor
#1 drush-gmdate.patch2.65 KBgreg.1.anderson

Comments

greg.1.anderson’s picture

greg.1.anderson commented
StatusFileSize
new drush-gmdate.patch2.65 KB

The problem is the date() function, which takes a time value that is in the units seconds since the epoch in GMT, and formats it to a string. In this instance, it is safe to rely on the system's timezone, but php does not believe that it is.

This patch replaces date() with gmdate(), the later of which works just like date(), but it formats in GMT rather than the local timezone. Anyway, it should do the trick.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Active » Needs review
moshe weitzman’s picture

moshe weitzman commented
Status: Needs review » Fixed

Committed. Thx.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.