Module::Metadata can't parse MLEHMANN/common-sense

[xenu]

Module::Metadata can't handle MLEHMANN/common-sense. It's a very unusual distribution, looks like its author tried really hard to abuse PAUSE indexer as much as possible.

There are following problems:

• ->package_versions_from_directory ignores .PL files (package and $VERSION are declared in sense.pm.PL).
• stuff in __DATA__ is ignored (that's where package is declared in that distribution).
• As opposed to Module::Metadata, PAUSE indexer doesn't really care in which package $VERSION was declared. With some exceptions, the first $VERSION definition is used with all packages in the parsed file ($VERSION is declared in main, not in common::sense).

[xenu]

See also https://rt.cpan.org/Public/Bug/Display.html?id=77249

[karenetheridge]

If the shipped distribution deliberately obfuscates the code before installation, then we can't do much -- Module::Metadata will properly parse the installed code however.

[haarg]

It isn't obfuscation. It's generating a module because it has platform specific data in it. PAUSE has intentional support for extracting version information from a .PL file that is used to generate a module.

[Leont]

I'm not sure in what context this is a problem…

[shadowcat-mst]

@Leont Dist::Metadata uses Module::Metadata to index dists, which means that both metacpan and my App::opan end up with confused results indexing dists. We should probably do something about this, whether it's adding an extra API designed for indexing source rather than an installed tree or just adding support for it to the existing API