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

[Stefano Lattarini]

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.

Thanks,
  Stefano

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