autoconf-patches
[Top][All Lists]
Advanced

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

Re: FYI: Eight easy patches


From: Stepan Kasal
Subject: Re: FYI: Eight easy patches
Date: Wed, 25 May 2005 08:48:29 +0200
User-agent: Mutt/1.4.1i

Hi,

On Tue, May 24, 2005 at 09:51:22AM -0700, Paul Eggert wrote:
> > +       shift 2 ;;
> 
> You used this in a few places.  Could you please replace this with
> "shift; shift"?  "shift 2" was not in the 7th Edition Bourne shell,
> and it is also absent in many pre-POSIX shells that are still of some
> interest to Autoconf.  (I suppose this should be put into Shellology
> too, in the documentation.)  Thanks.

Actually, the script already contained one occurence; I added two more.
All three are now replaced; and I put a note to the manual.
See the attached patch.

Thank you for catching this.  (I had a dejavu feeling, but wasn't able
to find anything in the docs nor in the list archives.)

Have a nice day,
        Stepan Kasal

Attachment: autoconf-20050525-no-shift.patch
Description: Text document


reply via email to

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