[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Tidy-up of node "Why doesn't automake support wildcards?"
From: |
Ralf Wildenhues |
Subject: |
Re: Tidy-up of node "Why doesn't automake support wildcards?" |
Date: |
Tue, 26 Feb 2008 23:23:39 +0100 |
User-agent: |
Mutt/1.5.17+20080114 (2008-01-14) |
Hello Reuben,
* Reuben Thomas wrote on Mon, Feb 25, 2008 at 05:40:50PM CET:
> I attach a diff against current git. It fixes the following problems:
Thanks for the patch.
> 1. The English is a bit ropey in places.
>
> 2. @file was used once where @emph was needed.
>
> 3. The point in the itemized list starting "Listing files" is redudant,
> it's just a restatement of the previous two points.
While I agree that there is partial redundancy here, IMVHO the other
items by themselves do not make it clear enough that 'make dist' will
complain about missing files.
> @@ -9860,24 +9860,18 @@ you remember to @samp{cvs add} it.
> @item
> Using wildcards makes easy to distribute files by mistake. For
There is an 'it' missing before 'easy', no?
> instance, some code a developer is experimenting with (a test case,
> -say) but that should not be part of the distribution.
> +say) that should not be part of the distribution.
> address@hidden
> -Finally it's really hard to @file{forget} adding a file to
> address@hidden, because if you don't add it, it doesn't get
> +Finally, it's really hard to @emph{forget} to add a file to
> address@hidden, because if you don't add it it is neither
I prefer if that comma remains in there, both because I think it is
justified by the grammar, and because otherwise the sentence is more
difficult to read.
> compiled nor installed, so you can't even test it.
> @end itemize
Can you be bothered to resubmit the patch with these items and the one
Tim found addressed, to the automake-patches list, including a ChangeLog
entry?
Thanks,
Ralf