This is the mail archive of the
automake@gnu.org
mailing list for the automake project.
Re: 10-check-am.patch
- To: Akim Demaille <akim at epita dot fr>
- Subject: Re: 10-check-am.patch
- From: Tom Tromey <tromey at redhat dot com>
- Date: 28 Jan 2001 18:57:20 -0700
- Cc: Automake <automake at gnu dot org>
- List-Id: Discussion list for automake <automake.gnu.org><mailto:automake-request@gnu.org?subject=unsubscribe>
- References: <E14MuTv-0006UY-00@nostromo.lrde.epita.fr>
- Reply-To: tromey at redhat dot com
>>>>> "Akim" == Akim Demaille <akim@epita.fr> writes:
Akim> * automake.in (handle_phony): Sort.
Akim> * automake.in (file_contents_with_transform): Catch `.PHONY:'.
Akim> * check.am: New file.
check.am wasn't in the patch. However, I trust it is just an
extraction from handle_tests.
Please add a ChangeLog entry for handle_tests. Also, the ChangeLog
entry shouldn't repeat `automake.in':
* automake.in (handle_phony): Sort.
(file_contents_with_transform): Catch `.PHONY:'.
* check.am: New file.
(This is an absurdly nitty nit.)
Finally, could you move the `Solaris 2.7' comment from handle_tests
into check.am and make it into a `##' comment? In fact, move it to
just above the line it comments.
It is fine to `cvs add' a file so that `cvs diff -N' creates the
correct diff. This applies generally -- if you don't have cvs write
access there is a script you can get that will do a phony `cvs add' by
manipulating CVS/Entries.
Tom