search

thaljef / Pinto

Curate your own repository of Perl modules
https://metacpan.org/module/Pinto::Manual
66 stars 49 forks source link

I pulled Algorithm::Diff and ended up with a very strange release #148

Closed akarelas closed 10 years ago

akarelas commented 10 years ago

I did pinto pull Algorithm::Diff and ended up with:

./authors/id/N/NE/NEDKONZ/Algorithm-Diff-1.15.tar.gz

Author NEDKONZ doesn't seem to have any distributions under his account: https://metacpan.org/author/NEDKONZ

The distribution I wanted is: https://metacpan.org/pod/Algorithm::Diff

I'll download the tar by hand and put it in, but this doesn't seem right, I could have ended up with a bug.

thaljef commented 10 years ago

Or specify the full author/distfile (easier than downloading by hand).

Yes, that is bug.

thaljef commented 10 years ago

It is odd because NEDKONZ actually did release the first versions of Algorithm::Diff, and they all had higher version numbers than what is on CPAN now. At some point, I guess MJD took over and changed the numbering scheme.

But this illustrates that Stratopan needs different logic for indexing BackPAN[1]. Specifically, it needs to exclude distributions that aren't on CPAN any more (but only when requesting a module without a specific version).

[1] Under the hood, Pinto is using a Stratopan API to locate stuff.

thaljef commented 10 years ago

I take that back...NEDKONZ releases came after MJD.

But this is probably what you really want:

https://metacpan.org/release/TYEMQ/Algorithm-Diff-1.1902

So I would argue the dist Pinto gave you is actually closer to what you really want (but still not entirely correct).

thaljef commented 10 years ago

I figured it out...

It's because the dist contains Algorithm::DiffOld, and TYEMQ doesn't have permission on that namespace, so Stratopan excluded it. Not sure if that's the right thing to do or not.

thaljef commented 10 years ago

Now I'm sure of it. Stratopan should exclude the package, but not the dist. That's how PAUSE behaves anyway.

thaljef commented 10 years ago

The indexing on Stratopan has been improved, and this appears to be fixed now. pinto pull Algorithm::Diff now gives you this release which is currently the latest version.