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/
signature.asc
Description: PGP signature