[Top][All Lists]

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

Re: bug#10878: "make dist" with read-only srcdir generates read-only tar

From: Stefano Lattarini
Subject: Re: bug#10878: "make dist" with read-only srcdir generates read-only tarball
Date: Sat, 25 Feb 2012 20:25:54 +0100

On 02/25/2012 07:10 PM, Nick Bowler wrote:
> Hi Stefano,
Hi Nick, and thanks for all the feedback.

> One comment below:
> On 2012-02-25 14:39 +0100, Stefano Lattarini wrote:
> [...]
>> And here is the documentation about the fact that a dist-hook should be ready
>> to deal with read-only files.  I will apply the attached patch soonish to 
>> master
>> if there is no objection.
> [...]
>> address@hidden
>> +Note that the @code{dist-hook} recipe shouldn't assume that the
>> +regular files in the distribution directory are writable; this
>> +might not be the case if one is packaging from a read-only source
>> +tree, or when a @code{make distcheck} is being done.  So, if the
>> address@hidden wants to modify the content of an existing file
>> +in the distribution directory, it should explicitly ensure to make
>> +it readable first:
>       ^^^^^^^^
>       writable
> I also would drop the words "ensure to".
Good catches.

I've fixed that, started doing a couple of other improvements, and finally
basically re-wrote the patch (hopefully in a better way).  The new version
of the patch, that I've pushed, is attached.  This should settle the issue,
so I'll close this report by tomorrow if there are no further objection.


Attachment: 0001-docs-improve-dist-hook-documentation.patch
Description: Text Data

reply via email to

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