GNU bug report logs - #71534
[PATCH] aclocal: support ACLOCAL_PATH correctly on OS/2

Previous Next

Full log

Message #14 received at 71534 <at> debbugs.gnu.org (full text, mbox):

From: Karl Berry <karl <at> freefriends.org>
To: zack <at> owlfolio.org
Cc: 71534 <at> debbugs.gnu.org, komh78 <at> gmail.com
Subject: Re: [bug#71534] [PATCH] aclocal: support ACLOCAL_PATH correctly on
 OS/2
Date: Thu, 13 Jun 2024 16:37:38 -0600

    Isn't there a $variable already that's specifically the path
    separator? 

What I see is core Perl:

use Config;
... $Config{"path_sep"} ...

    I thought we had one. 

The above is not currently used anywhere in Automake.
There are pervasive uses of $PATH_SEPARATOR in shell code,
but no determination of the path separator in Perl code that I can see.

    Check in Perl's built-in Config module 

Indeed. I guess {path_sep} has been available forever,
but I still fear breaking new ground.

Also, since the Config module is based on the compile-time Configure
output, there are warnings that it "cannot always be trusted".
Maybe it would never come up for Automake, but then again, maybe it would.

    as well as automaker proper.

Nothing in Automake::Config or anything else that I can find.

I have to admit I am tempted to just match $^O and be done with it,
instead of using something that could lead to more problems in the
future. From what I've seen of other code, using $^O is a lot more
common than using $Config. (Anecdotally speaking.) --thanks, karl.

Previous Next

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