On Mon, Nov 29, 2021 at 3:12 PM Richard W.M. Jones <[email protected]> wrote:
>
> On Mon, Nov 08, 2021 at 06:06:40PM +0000, Richard W.M. Jones wrote:
> > On Mon, Nov 08, 2021 at 07:55:51PM +0200, Nir Soffer wrote:
> > > We can use git short commit date format $cs. Maybe it was not available
> > > when podwrapper.pl was create.
> > >
> > > Signed-off-by: Nir Soffer <[email protected]>
> > > ---
> > >  podwrapper.pl.in | 3 +--
> > >  1 file changed, 1 insertion(+), 2 deletions(-)
> > >
> > > diff --git a/podwrapper.pl.in b/podwrapper.pl.in
> > > index 9b39d7c..86ef452 100755
> > > --- a/podwrapper.pl.in
> > > +++ b/podwrapper.pl.in
> > > @@ -233,8 +233,7 @@ my $date;
> > >  my $filename = "$abs_top_srcdir/.git";
> > >  if (!$date && -d $filename) {
> > >      local $ENV{GIT_DIR} = $filename;
> > > -    $_ = `git show -O/dev/null -s --format=%ci`;
> > > -    $date = $1 if /^(\d+-\d+-\d+)\s/;
> > > +    $date = `git show -O/dev/null -s --format=%cs`;
> > >  }
> > >  if (!$date) {
> > >      my ($day, $month, $year) = (gmtime($ENV{SOURCE_DATE_EPOCH} || 
> > > time))[3,4,5];
> >
> > This is fine.  Same patch should go into nbdkit too.
>
> Actually this caused a problem and I had to revert the commits in
> nbdkit & libnbd.  See:
>
> https://gitlab.com/nbdkit/nbdkit/-/commit/750ad5972bb082d188f17f8f71ef1ec0c616c676

Sorry, I tested the generated man pages with "man /path/to/manpage" and
they looked correct.

I think this should work:

 -    $_ = `git show -O/dev/null -s --format=%ci`;
 +    $_ = `git show -O/dev/null -s --format=%cs`;

Nir

_______________________________________________
Libguestfs mailing list
[email protected]
https://listman.redhat.com/mailman/listinfo/libguestfs

Reply via email to