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.

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

Rework the META file to make indexable by Pinto #31

Merged

Conversation

hartzell
Copy link

@hartzell hartzell commented May 1, 2018

Pinto has trouble indexing this package because it doesn't understand
the use of the META.yml in the provides clause of the META info.

The spec also allows referring to the file that generates the package
in that statement, but pinto (apparently following PAUSE's lead)
doesn't consider ReadKey_pm.PL to be a "simile" for that package, the
rule seems to be that ".pm.PL" is the acceptable form.

This commit

  • renames ReadKey_pm.PL -> ReadKey.pm.PL;
  • touches up the Makefile.PL and MANIFEST; and
  • changes the META.yml file to refer to ReadKey.pm.PL

See #29 and thaljef/Pinto#241.

Pinto has trouble indexing this package because it doesn't understand
the use of the META.yml in the provides clause of the META info.

The spec also allows referring to the file that generates the package
in that statement, but pinto (apparently following PAUSE's lead)
doesn't consider ReadKey_pm.PL to be a "simile" for that package, the
rule seems to be that ".pm.PL" is the acceptable form.

This commit

- renames ReadKey_pm.PL -> ReadKey.pm.PL;
- touches up the Makefile.PL and MANIFEST; and
- changes the META.yml file to refer to ReadKey.pm.PL

See jonathanstowe#29 and thaljef/Pinto/#241.
@hartzell
Copy link
Author

hartzell commented May 1, 2018

ps. This might also fix the fact that metacpan.org returns META.yml as the file for the TermReadKey tarball when one searches for Term::ReadKey.

screen shot 2018-04-30 at 9 47 40 pm

@Grinnz
Copy link

Grinnz commented Oct 27, 2018

While META.yml is a valid target according to CPAN::Meta::Spec there is no reason not to point at the actual file generating the source code, as that's what this field is supposed to be for.

@jonathanstowe jonathanstowe merged commit fba4054 into jonathanstowe:master Jan 5, 2019
@jonathanstowe
Copy link
Owner

Thanks,
sorry I took a while to get round to this.

@jonathanstowe
Copy link
Owner

I've just uploaded 2.38 to PAUSE which contains this fix. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants