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

Make cpan faster #8

Merged
merged 6 commits into from Apr 4, 2015

Conversation

Projects
None yet
2 participants
@spechenin
Contributor

spechenin commented Mar 29, 2015

This is just a technology preview, it is not meant to merge to master branch.
Bootstrap-perl runs cpan executable multiple times. And every time it is going to initialize CPAN.pm databases which is time-consuming. I offer to spawn cpan only once and use it as daemon.

@renormalist

This comment has been minimized.

Owner

renormalist commented Apr 1, 2015

Interesting. I like it but prefer to play around with it a bit, I try to have a look soon(tm).

Similar to the other PR, I can't merge without conflicts, if you could rebase I will probably just go ahead and merge, just because I like it and would simply follow your approach.

Thank you very much, I really appreciate it!

spechenin added some commits Mar 29, 2015

print $USER
Print all messages to STDOUT through $USER.
This is helpful when testing to be silent, to collect and test all user output.
add early exit point
if $no_run is declared
thus we can load bootstrap-perl and test subs while testing
@spechenin

This comment has been minimized.

Contributor

spechenin commented Apr 4, 2015

The PR is rebased on PR#7

renormalist added a commit that referenced this pull request Apr 4, 2015

@renormalist renormalist merged commit 6493ba4 into renormalist:master Apr 4, 2015

@renormalist

This comment has been minimized.

Owner

renormalist commented Apr 5, 2015

From a couple of sample runs it seems to work well. Thanks!
I just made a release to CPAN.

The only thing I changed is to call warn() instead of die() because my shell code in system is still assuming fails are ok and needs to better work with multiple consecutive runs.

@spechenin spechenin deleted the spechenin:pr-cpan branch Apr 5, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment