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_prereqs.t now provides diag() dumps of $Config{cc},
$Config{gccversion}, and $Config{osname} to help sort out the issue.
Attached is a diff for Inline::CPP showing relevant modifications to
Makefile.PL, and 00load_prereqs.t. I actually think that #2 (above)
is the important change, and I might pull #1 back out of the Solaris
compiler detection code.
I'd appreciate if the code could be looked over and given the "seems
reasonable to me" seal of approval before I upload CPAN version
0.34_001 (dev). The diff (attached) should be enough to go by, but
anyone wanting to see the full Makefile.PL with modifications can pull
it from the dev branch of my github repo (daoswald).
Dave
On Thu, Feb 9, 2012 at 9:07 PM, Sisyphus <sisyph...@optusnet.com.au> wrote:
>
> ----- 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 just change that:
>
>
> if ($Config{cc} eq 'gcc') {
> to something like:
> if (($Config{cc} eq 'gcc') || $Config{gccversion}) {
>
> but I'm not at all familiar with solaris.
>
> Cheers,
> Rob
--
David Oswald
daosw...@gmail.com
ilcpp.diff
Description: Binary data
