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
install fails on windows #3
Comments
it's not public api, so this prevents building on windows (and other platforms that don't expose everything for linking), and we're building the whole optree by hand at once, so there's no reason to use fixup functions like this whose only use is for when we're building the optree incrementally during parsing.
i think this is the right thing, not quite sure
Can you try the current version in the repository now? |
cpanm (App::cpanminus) 1.6005 on perl 5.016003 built for MSWin32-x64-multi-thread |
now
then
|
|
but same error
|
Can you show me the output of |
Oh, actually, just try |
Just updated the |
perl -MDevel::CallParser -E"say for Devel::CallParser::callparser_linkable"
|
Building and testing Dist-Zilla-Plugin-Test-Compile-2.013 ... FAIL there is no system('make'); on windows |
What is calling |
Oh, I see - just try using the |
using
again, hope this is what you wanted?
|
I filed https://rt.cpan.org/Ticket/Display.html?id=87481 to follow up on the dzil issue, for what it's worth. As for the new results, can you show me the contents of |
Keyword.def
|
Try building again? |
|
I think I'm going to have to look into this myself, I'll let you know what I come up with. |
Okay, I think this may have done it. Can you give this one a try? |
|
Can you show me what the generated Makefile looks like? |
|
if they use absolute paths on windows, we can end up with a makefile rule that looks like this: Keyword.o C:\...\CallParser.a : ... and make gets confused by that first colon.
cpanm (App::cpanminus) 1.6005 on perl 5.016003 built for MSWin32-x64-multi-thread ********************************* WARNING ********************************** This module uses Dist::Zilla for development. This Makefile.PL will let you Writing Makefile for Parse::Keyword Run 'dzil authordeps' to see a list of all required plugins.
dmake.exe: Error code 129, while making 'install' |
Awesome! That means that it did work - it's intentional that you can't actually install from the Makefile.PL in the repository. I'll get a release out and hopefully that'll actually resolve this. |
Can you try installing http://cpan.metacpan.org/authors/id/D/DO/DOY/Parse-Keyword-0.06.tar.gz? |
okay, seems to work with some warnings:
|
Alright, I think this is good enough for now. If that starts causing problems, I'll look into it further. |
The text was updated successfully, but these errors were encountered: