Skip to content
This repository has been archived by the owner on Mar 7, 2019. It is now read-only.

create a config.site file for autoconf #113

Merged
merged 2 commits into from Mar 13, 2015
Merged

create a config.site file for autoconf #113

merged 2 commits into from Mar 13, 2015

Conversation

plicease
Copy link
Contributor

This creates a config.site file for autoconf so that it will make autoconf use the same compiler, compiler flags and linker flags as Perl.

This makes source code builds on Solaris with 64 bit Perls work. (tested). It stands a decent chance of also working on HP-UX and AIX with 64 bit Perls, since it relies on the Perl Config module. (untested).

The problem with platforms like Solaris, HP-UX and AIX is that they are mixed 32/64 bit environments where 32 bit is usually they default.

Caveats:

  • I'm stripping out -D (Define) flags as they are Perl specific.
  • If CONFIG_SITE is already set, then we defer to that. The assumption is that if integrators / users have set this variable they have set it for the correct compiler.

@plicease
Copy link
Contributor Author

btw- I considered generating the config.site file when Alien::Base is installed, and putting it in a share directory for that, but I know that people do sometimes update Config.pm, and it may be useful to allow people do that so they can fix stuff.

@mohawk2
Copy link
Contributor

mohawk2 commented Feb 28, 2015

Yes, I've locally changed my Config setup (specifically Config_heavy's "make" setting) to test different things, and would be unimpressed if a tool like A::B ignored that. Very good work! Aye.

@plicease
Copy link
Contributor Author

Just a heads up, I plan on merging this and #112 on Friday and doing a dev release unless someone speaks up against against before then.

@zmughal
Copy link
Member

zmughal commented Mar 12, 2015

+1

@plicease plicease merged commit f97195f into Perl5-Alien:master Mar 13, 2015
@plicease plicease deleted the site_config branch March 13, 2015 08:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants