Chris Lamb wrote on Wed, Oct 24, 2018 at 08:38:36 -0400:
> @@ -17,7 +15,7 @@
> -+    @$(call ised,"s/{{ANTIGEN_REVISION_DATE}}/$$(date --date 
> @$$(dpkg-parsechangelog -STimestamp) --rfc-3339=seconds)/",${TARGET})
> ++    @$(call ised,"s/{{ANTIGEN_REVISION_DATE}}/$$(date --date --utc 
> @$$(dpkg-parsechangelog -STimestamp) --rfc-3339=seconds)/",${TARGET})

Chris, are you sure that's right?  According to date(1), the --date
option takes an argument, so that would seem to take "--utc" as the date
to parse:

% date --date --utc @1 
date: the argument ‘@1’ lacks a leading '+';
when using an option to specify date(s), any non-option
argument must be a format string beginning with '+'
Try 'date --help' for more information.
zsh: exit 1     date --date --utc @1
% date --utc --date @1
Thu Jan  1 00:00:01 UTC 1970
% 

>  -    @$(call ised,"s/{{ANTIGEN_REVISION}}/$$(git log -n1 --format=%h -- 
> src)/",${TARGET})
>  -    @$(call ised,"s/{{ANTIGEN_REVISION_DATE}}/$$(git log -n1 --format='%ai' 
> -- src)/",${TARGET})
>  +    @$(call ised,"s/{{ANTIGEN_REVISION}}/Debian/",${TARGET})

Tangentially, Michael, here you might consider changing the hardcoded
"Debian" to the package version, $$(dpkg-parsechangelog -SVersion).  The
zsh and zsh-syntax-highlighting packages do this, too.

Cheers,

Daniel
(not subscribed to the bug, please Cc)

Reply via email to