Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

get config_deps from prereqs if available #204

Merged
merged 1 commit into from Feb 27, 2013

Conversation

kazeburo
Copy link
Contributor

get config_deps from {meta}->{prereqs}->{configure} if available.
configure_requires is deprecated in spec 2

@miyagawa
Copy link
Owner

Looks great! Do you have a sample dist that fails with this?

@kazeburo
Copy link
Contributor Author

sample dist
https://github.com/kazeburo/CPAN-Test-Dummy-ConfigDeps
Build.PL requires Module::CPANfile.

failed log

% cpanm --self-upgrade
--> Working on App::cpanminus
Fetching http://www.cpan.org/authors/id/M/MI/MIYAGAWA/App-cpanminus-1.6001.tar.gz ... OK
Configuring App-cpanminus-1.6001 ... OK
Building and testing App-cpanminus-1.6001 ... OK
Successfully installed App-cpanminus-1.6001 (upgraded from 1.5021)
1 distribution installed
% perl -MModule::CPANfile -e ''
Can't locate Module/CPANfile.pm in @inc (@inc contains: /.....).
BEGIN failed--compilation aborted.
% cpanm -L extlib -n .
--> Working on .
Configuring CPAN-Test-Dummy-ConfigDeps-0.01 ... N/A
! Configure failed for CPAN-Test-Dummy-ConfigDeps-0.01. See /home/kazeburo/.cpanm/build.log for details.

miyagawa added a commit that referenced this pull request Feb 27, 2013
miyagawa added a commit that referenced this pull request Feb 27, 2013
@miyagawa miyagawa merged commit 47a3476 into miyagawa:master Feb 27, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants