Skip Menu |

This queue is for tickets about the Module-Install CPAN distribution.

Report information
The Basics
Id: 33976
Status: resolved
Priority: 0/
Queue: Module-Install

People
Owner: Nobody in particular
Requestors: dagolden [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: (no value)
Fixed in: (no value)



Subject: cc_inc_paths replaces INC provided on command line
cc_inc_paths appears to replace INC rather than append to any INC provided on the command line. This causes compilation to fail on Strawberry Perl, for example, which sets INC (and LIB) as a Makefile.PL option in CPAN.pm. N.B. Problem detected on YAML::Syck. Symptom: make fails when run from CPAN.pm, but succeeds when Makefile.PL and make run "by hand" in a command shell without the default INC and LIB passed to Makefile.PL
What should the correct behaviour be?
On Thu Jul 17 12:56:25 2008, ADAMK wrote: Show quoted text
> What should the correct behaviour be?
Merge them? INC on command line is probably there for a reason and should be preserved. Not sure precedence/order between the two. Should command line come first, then cc_inc_paths? Maybe a question for p5p. -- David
Hi. Added a test in the trunk to ensure Module::Install does the same as EUMM, and tweaked to merge makemaker_args (set by helper function, or by makemaker_args itself) when possible. Hope this helps. On 2008-7-17 Thu 21:14:43, DAGOLDEN wrote: Show quoted text
> On Thu Jul 17 12:56:25 2008, ADAMK wrote:
> > What should the correct behaviour be?
> > Merge them? > > INC on command line is probably there for a reason and should be
preserved. Show quoted text
> > Not sure precedence/order between the two. Should command line come > first, then cc_inc_paths? Maybe a question for p5p. > > -- David
Hi. 0.95 is out. If you still have the problem, reopen this. Thanks.