Kurt Roeckx wrote:
> > ++if test -n "$SOURCE_DATE_EPOCH"
> > ++then
> > ++ now=`LC_ALL=C date --utc --date="@$SOURCE_DATE_EPOCH"`
> > ++else
> > ++ now=`date`
>
> That should probably also use LC_ALL=C
I usually patch that as:
a) I don't want to change existing behaviour so that it is easier,
politically, to get it merged upstream.
b) The value will vary on timezone and — of course — the current time,
so fixing the language doesn't really do much. It could even be argued
it therefore makes patch misleading in that it only goes "half way".
Hope that helps.
Regards,
--
,''`.
: :' : Chris Lamb
`. `'` [email protected] / chris-lamb.co.uk
`-
