automake-patches
[Top][All Lists]
Advanced

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

Re: automake/483: "missing" doesn't handle "--output" for missing makein


From: Ralf Wildenhues
Subject: Re: automake/483: "missing" doesn't handle "--output" for missing makeinfo
Date: Sat, 22 Apr 2006 14:00:56 +0200
User-agent: Mutt/1.5.11+cvs20060403

http://sources.redhat.com/cgi-bin/gnatsweb.pl?cmd=view%20audit-trail&database=automake&pr=483

Hi Dennis,

Sorry for the long delay, and thanks for the bug report.

Looking at the `missing' script, there are a couple more issues:
- the sed scripts for matching `-o' and `--output' aren't all the
  same; furthermore, they are less strict than they could be: we
  can know for sure both flags will be preceded by a space, and
  there will be exactly one space or equal sign between `--output'
  and its argument (all due to how we echo the args before);
- autom4te isn't documented in --help output.

So then I got adventurous and wrote another one of those testsuite
tests that are supposed to entertain the reader.  ;-)
(I'll leave it as an exercise to find out why I removed usage of `['
from `missing'.)

OK to apply both attached patches?

Cheers,
Ralf

        * lib/missing (sed_minuso, sed_output): New variables.
        (autom4te, help2man, makeinfo): Use them.  Unifies detection of
        `-o FILE', `--output FILE', `--output=FILE', stricter regex.
        Fixes `missing' to detect `--output' for help2man (PR 483).
        Report by Dennis J. Linse <address@hidden>.
        (autom4te): Document in `missing --help'.
        * THANKS: Update.

        * lib/missing: Remove superfluous quotes.  Replace all uses of
        `[' by `test', for consistency, and for..
        * tests/missing4.test: ..this new test.
        * tests/Makefile.am: Update.

Attachment: missing-output.diff
Description: Text document

Attachment: missing-test-quotes.diff
Description: Text document


reply via email to

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