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.

Sign up for GitHub

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

Jump to bottom

use HTTP::Tiny instead of LWP #7

Closed
karenetheridge opened this issue Aug 13, 2013 · 3 comments
Closed

use HTTP::Tiny instead of LWP #7

karenetheridge opened this issue Aug 13, 2013 · 3 comments

Comments

@karenetheridge
Copy link
Contributor

I cribbed your cpanidx code for Dist::Zilla::Plugin::Test::NewVersion, and then later changed the use of LWP to HTTP::Tiny.

Poke me when you cycle back to this dist and I'll send a PR. ;)
@rjbs
Copy link
Owner

rjbs commented Aug 13, 2013

  • Karen Etheridge notifications@github.com [2013-08-13T13:02:54]

    I cribbed your cpanidx code for Dist::Zilla::Plugin::Test::NewVersion, and
    then later changed the use of LWP to HTTP::Tiny.

    Poke me when you cycle back to this dist and I'll send a PR. ;)

If you send the PR first, it'll be wiaitng when I get here!

rjbs

@rjbs
Copy link
Owner

rjbs commented Dec 26, 2013

I'm back to it now, but now I'm going to keep moving…

@karenetheridge
Copy link
Contributor Author

better would be to hook into xdg's CPAN::Common::Index - an approach I plan to explore for other plugins "soon". (it might be useful to expose this functionality to dzil plugins via a role?)

dagolden pushed a commit to dagolden/Dist-Zilla-Plugin-CheckPrereqsIndexed that referenced this issue Nov 20, 2014
@xdg
use cpanmetadb instead of cpanidx
95b5ba8 
CPANIDX has been down for a while.  cpanmetadb is what cpanm uses, so is
more likely to stay up or get fixed quickly.  (I considered MetaCPAN but
they require SSL.)

This also swaps to HTTP::Tiny and YAML::Tiny instead of LWP and JSON,
which should make @ether happy, fixing rjbs#7 and rjbs#10.
@dagolden dagolden mentioned this issue Nov 20, 2014
Closed
rjbs pushed a commit that referenced this issue Nov 21, 2014
@xdg @rjbs
use cpanmetadb instead of cpanidx
80df9ae 
CPANIDX has been down for a while.  cpanmetadb is what cpanm uses, so is
more likely to stay up or get fixed quickly.  (I considered MetaCPAN but
they require SSL.)

This also swaps to HTTP::Tiny and YAML::Tiny instead of LWP and JSON,
which should make @ether happy, fixing #7 and #10.
@rjbs rjbs closed this as completed in fc0a136 Nov 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@rjbs @karenetheridge