(optdeps) Fix incorrect optdep output introduced in 34d2deae / 31c31b8d
authorPeter Rabbitson <ribasushi@cpan.org>
Wed, 1 Apr 2015 20:52:41 +0000 (22:52 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Wed, 12 Aug 2015 03:08:56 +0000 (05:08 +0200)
commitd66442d3bcd1d22f7a7c7586aeb8fbb40def2810
treeeb3316e960e762994a33bc0c825056f6b44e1ce2
parent409a3b1e1774dcaceb591544e656dd09e3a75878
(optdeps) Fix incorrect optdep output introduced in 34d2deae / 31c31b8d

Too many thinkos for a simple feature like this. When given "foo" cpanm
actually tries to find '"foo"' on CPAN, and nothing works. The original
thinking was that the > in ~>= needs escaping when copy/pasted by a user, so
the ""-enclosing seemed like a valid approach. Of course this does not work
at all when the output is piped into cpanm directly:

 perl -Ilib -MDBIx::Class::Optional::Dependencies=-list_missing,deploy | cpanm

Moreover, all of the above is *utterly unnecessary*, as Foo~>=2 is entirely
equivalent to Foo~2, which requires no escaping (thus quoting) whatsoever.

Hopefully this is the end of the embarrassing flailing on this feature :(
lib/DBIx/Class/Optional/Dependencies.pm
xt/extra/internals/optional_deps.t