GNU bug report logs - #10766
CXX gets the value 'CC' on Cygwin, but CC is in fact cc

Previous Next

Package: automake;

Reported by: Peter Rosin <peda <at> lysator.liu.se>

Date: Wed, 8 Feb 2012 21:06:01 UTC

Severity: minor

Tags: patch

Done: Stefano Lattarini <stefano.lattarini <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


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

From: Stefano Lattarini <stefano.lattarini <at> gmail.com>
To: Eric Blake <eblake <at> redhat.com>
Cc: Peter Rosin <peda <at> lysator.liu.se>, Max Horn <max <at> quendi.de>,
	11893 <at> debbugs.gnu.org,
	"automake-patches <at> gnu.org" <automake-patches <at> gnu.org>,
	10766 <at> debbugs.gnu.org
Subject: Re: bug#10766: bug#11893: Regression in automake 1.12.1 on Mac OS
 X
Date: Thu, 12 Jul 2012 20:42:43 +0200
[Message part 1 (text/plain, inline)]
On 07/12/2012 07:06 PM, Eric Blake wrote:
> On 07/12/2012 10:23 AM, Stefano Lattarini wrote:
> 
>>> Other than that, the only supported method in existing POSIX for
>>> checking for equal files is by parsing 'ls -i' output; but I don't know
>>> if 'ls -i' is portable to ancient hosts.
>>>
>> I was fearing this would have got quickly complex and messy ...
>>
>> So I think I'll stick to my dumber and safer "test -f /bin/RMDIR" as a
>> way to detect case-insensible filesystems.
> 
> Assuming it was intentional, I like your re-wording of
> "case-insensible", as an apt description of the pain case-insensitive
> systems cause on portability :)
>
Sorry, no deliberate pun, just a lexical error.  But I can keep it if
you like it :-)

> That said, /bin/RMDIR is probably not completely portable, but seems
> like it would be reliable in practice.
>
That is my impression.  And anyway, there aren't too many people running
the Automake testsuite on Mac OS X or Cygwin, so if the patch manages to
fix Peter's and Max's issue, it is good enough for me (at least until
somebody else reports a similar issue).  Finally, consider that the
result of a botch-up of this new configure check would simple be one or
two spurious testsuite failures -- so just a small annoyance, nothing to
really worry about.

> POSIX doesn't reqiure /bin/rmdir to
> exist (about the only files that you can guarantee with POSIX are /tmp,
> /dev/null, /dev/tty, and /dev/console).  Since /dev/ generally doesn't
> exist on mingw, that almost limits you to just testing if /tmp and /TMP
> are the same directory to detect a case-insensitive system.  But I don't
> know of any systems that lack /bin/rmdir on a default installation, nor
> do I suspect many users intentionally create /bin/RMDIR (or even better,
> /bIn/rMdIr),
>
Good idea for an improved test; I've implemented it in my patch.

> so using it as a hueristic seems okay for a first cut at
> the problem at hand.  Or, maybe you should just create a file in /tmp
> under one spelling and check existence of the file by another spelling.
> 

In the light of our discussion, attached is the patch I'd finally like
to push.  WDYT?

Thanks,
  Stefano
[0001-tests-don-t-use-C-instead-of-C-compiler-on-case-inse.patch (text/x-diff, attachment)]

This bug report was last modified 13 years and 11 days ago.

Previous Next


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