GNU bug report logs - #62896
[Configure] Bug with check for PERL when path has spaces (i.e. Windows)

Previous Next

Package: automake;

Reported by: Dan Rosser <danoli3 <at> gmail.com>

Date: Mon, 17 Apr 2023 06:54:02 UTC

Severity: normal

Done: Paul Eggert <eggert <at> cs.ucla.edu>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Mike Frysinger <vapier <at> gentoo.org>
To: Karl Berry <karl <at> freefriends.org>
Cc: 62896 <at> debbugs.gnu.org
Subject: bug#62896: [Configure] Bug with check for PERL when path has spaces (i.e. Windows)
Date: Sat, 2 Dec 2023 21:08:08 -0500
[Message part 1 (text/plain, inline)]
On 02 Dec 2023 15:53, Karl Berry wrote:
>     also, can we really not trust the exit status of grep ?
>             if echo "$PERL" | grep -q '[\t ]'; then
> 
> Exit status yes, but at least historically, grep -q has been considered
> non-portable, in favor of grep ... >/dev/null.

i get that `grep -q` is something historically we've avoided, but i think it's
one of those bad habits we should just get out of.  i'll note that gnuconfig
uses `grep -q` in a bunch of places.

not that i'm going to spend time going through the codebase to update all the
usage, so if automake itself currently does it this way, OK.

> I committed the change as below. --thanks, karl.

thanks, lgtm
-mike
[signature.asc (application/pgp-signature, inline)]

This bug report was last modified 1 year and 222 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.