Re: [PATCH v2 14/14] git-cvsexportcommit: port to SHA-256
- From
brian m. carlson <sandals@crustytoothpaste.net>
- Date
- Aug 3, 2020, 23:50 UTC
- Message-ID
- <20200803235007.GN6540@camp.crustytoothpaste.net>
- In-Reply-To
- <87zh7bo9so.fsf@igel.home>
On 2020-08-03 at 18:37:27, Andreas Schwab wrote:
Show 13 quoted lines
> On Jun 22 2020, brian m. carlson wrote:
>
> > diff --git a/git-cvsexportcommit.perl b/git-cvsexportcommit.perl
> > index fc00d5946a..6483d792d3 100755
> > --- a/git-cvsexportcommit.perl
> > +++ b/git-cvsexportcommit.perl
> > @@ -22,6 +22,10 @@
> > my $repo = Git->repository();
> > $opt_w = $repo->config('cvsexportcommit.cvsdir') unless defined $opt_w;
> >
> > +my $tmpdir = File::Temp->newdir;
>
> File::Temp in perl 5.10 doesn't have the newdir method.That method was added in File::Temp 0.19, which was added in 2007. Does
my $tmpdir = File::Temp::tempdir(CLEANUP => 1);
do the right thing on your Perl 5.10? I no longer use CentOS 6 at work and it won't run in Docker on modern Debian, so I can't test. If so, I'll send a patch.
For the record, I plan to propose that we drop support for Perl versions earlier than 5.14 on December 2, since CentOS 6 will be dead at that point. I think a ten-year lifespan for an OS is quite generous and we're still considering Perl 5.8.8, which nobody is publicly supporting anymore.
-- brian m. carlson: Houston, Texas, US