Re: Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread Sisyphus
- Original Message - From: "David Oswald" To: "Sisyphus" ; Sent: Friday, February 10, 2012 6:02 PM Subject: Re: Solaris v2.11 C++ compiler detection for Inline::CPP I've implemented three changes that should fix the Solaris issue. 1: Makefile.PL now atte

Re: Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread David Oswald
I've implemented three changes that should fix the Solaris issue. 1: Makefile.PL now attempts to detect if $Config{gccversion} has anything meaningful in selecting the proper compiler. 2: Makefile.PL now detects $Config{cc} eq 'cc' version $Config{cc} eq 'CC' within the Solaris logic. 3: 00load

Re: Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread Sisyphus
- Original Message - From: "David Oswald" But if I remove that test and remove CC as an option I'll probably break some other Solaris install. Yes - probably don't remove the test. (Presumably CC must be a sane fall-thru value under some circumstance or other.) I'd be tempted to

Re: Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread David Oswald
On Thu, Feb 9, 2012 at 5:26 PM, Sisyphus wrote: > > - Original Message - From: "David Oswald" > > >> It looks to me like the 'if( $config{cc} eq 'gcc' ) " code is >> falling through to the else clause that sets $cc_guess to 'CC', and in >> this case probably shouldn't be. > > > Yes -

Re: Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread Sisyphus
- Original Message - From: "David Oswald" It looks to me like the 'if( $config{cc} eq 'gcc' ) " code is falling through to the else clause that sets $cc_guess to 'CC', and in this case probably shouldn't be. Yes - looking a bit more closely, I see that these failing reports sho

Solaris v2.11 C++ compiler detection for Inline::CPP

2012-02-09 Thread David Oswald
Inline::CPP v0.34 as of this moment has 96 smoke tests in, 79 passes, and 17 fails. None of the failures exhibit the Parse::RecDescent bug. Fixed! Of the 17 failures, 15 are on Solaris 2.11, with the failure point, "/bin/sh: CC: command not found" We're trying to detect the correct compiler to