[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [lmi] [lmi-commits] master e653dbd 03/10: Refactor: transplant two r

From: Greg Chicares
Subject: Re: [lmi] [lmi-commits] master e653dbd 03/10: Refactor: transplant two recipe lines
Date: Thu, 2 May 2019 01:34:35 +0000
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.1

On 2019-05-01 23:05, Vadim Zeitlin wrote:
> On Tue, 30 Apr 2019 22:17:55 -0400 (EDT) Greg Chicares <address@hidden> wrote:
> GC> branch: master
> GC> commit e653dbd9d57f5d490864b7efb67c90657e9aae18
>  Sorry, but this doesn't look right:
>  But second, and worse, removing it from the "initial_setup" target is
> actively harmful because this means that extracting the archive with bsdtar
> using --directory=$(ad_hoc_dir) will fail: while bsdtar(1) is not totally
> clear about whether the directory specified by this option must exist, a
> simple test shows that it really must and that bsdtar exits with "could not
> chdir to $directory" error if it doesn't.


Thanks for catching that.

>  So I'm afraid we really need to move at least the second mkdir command
> back. AFAICS creating $(prefix) just before copying into it should be fine,
> however.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]