-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Build.PL missing (or README -> installation outdated) #5
Comments
As far as using the Git repo goes, I should add some extra instructions. This dist uses Dist::Zilla, so if you want to build from the repo directly (assuming you have cpanminus installed): cpanm Dist::Zilla To run the test suite: To install deps: dzil listdeps | cpanm I'm not sure if that will work without having installed the Dist::Zilla plugins listed in dist.ini first. Maybe you could try that and let me know? |
and doing so it takes care of installing the required plugins. I'm having a hard time to believe how much perl modules are pulled (by cpanm, dist::zilla and then authordeps)... It's a very long install process for a single perl module ;) Anyway, I can't even get it to install on our target platform (CentOS 5.6) because it's running perl 5.8.8 and Dist-Zilla-Plugin-Git requires 5.10..
That said, may I suggest you put in a 'devel' section with the commands you gave me above in the README? |
Fair enough! The dependency chain for Dist::Zilla is big, but at least the end user doesn't have to deal with it. :) Having said that, I will add your notes to a Devel section. So, you'll only need dzil to build and package the distro. Are you able to install a custom perl on your development platform? If you haven't looked at perlbrew, I'd suggest trying it out. It's great for exactly this sort of thing: |
Thanks for sharing that perl wisdom! Following your instructions I was able to run the test suite and make a build successfully. I also reproduced your 'YAML_LOAD_WARN_DUPLICATE_KEY' issue but I guess you fixed that when you migrated to json? |
Thanks for letting me know! Yeah, switching to JSON got rid of that error. I think it was just a problem with quoting, but JSON is probably enough for this sort of serialization. YAML feels like overkill and the parsers are a pain to deal with. |
A reference to this ticket has been added to the Pod, to make getting started with Dist::Zilla and perlbrew a bit easier. |
README says:
but there's no such thing as a Build.PL. Being unfamiliar with Build.PL maybe I'm missing something obvious.
The text was updated successfully, but these errors were encountered: