[Libguestfs] [PATCH libnbd] podwrapper.pl.in: Use short commit date

Richard W.M. Jones rjones at redhat.com
Mon Nov 29 13:12:23 UTC 2021


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 <nsoffer at redhat.com>
> > ---
> >  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

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-top is 'top' for virtual machines.  Tiny program with many
powerful monitoring features, net stats, disk stats, logging, etc.
http://people.redhat.com/~rjones/virt-top




More information about the Libguestfs mailing list