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>
Thu, 2 Apr 2015 07:50:47 +0000 (09:50 +0200)
commit34f354c33a21fdf20bee90696ef3c061823ce0ff
treea078f9784e2fe7f6b4a90e6c7e01fe36370c534f
parent137dca082e1ee8a1a2649b5ca0e0707efbc21c9f
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