On Tue, Jun 04, 18:16, Chris Lamb wrote
> Andre Noll wrote:
> 
> > Then I don't grok the purpose of the
> > 
> >     LC_ALL=C date -u -r '$(SOURCE_DATE_EPOCH)' '$(DATE_FMT)'
> > 
> > command. After all, at least GNU date(1) expects a filename as the
> > argument to -r.
> 
> I do not have access to such a system but I believe this is for BSD
> variants of date(1), hence the waterfall of ||'s.

Indeed: FreeBSD-12's date(1) has

        -r seconds
             Print the date and time represented by seconds, where seconds is
             the number of seconds since the Epoch (00:00:00 UTC, January 1,
             1970; see time(3)), and can be specified in decimal, octal, or
             hex.

and NetBSD-8 also has -r which expects a "seconds" argument.

I guess this is not relevant to the case at hand since all Debian
flavours use GNU userland, so we can rely GNU date(1). Unless you
have strong feelings about it I'll apply the patch as is, i.e. without
the above alternative for BSD compatibility.

Best
Andre
-- 
Max Planck Institute for Developmental Biology
Max-Planck-Ring 5, 72076 Tübingen, Germany. Phone: (+49) 7071 601 829
http://people.tuebingen.mpg.de/maan/

Attachment: signature.asc
Description: PGP signature

Reply via email to