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

Update MakeMaker's NAME argument to the "main module" Gearman::Client #3

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

miyagawa
Copy link

MakeMaker's NAME argument has to be set to the "main module" of the distribution and in this distribution it is considered Gearman::Client.

For comparison, Template-Toolkit has NAME set to 'Template' and libwww-perl has NAME set to LWP.

MakeMaker's NAME argument has to be set to the "main module" of the
distribution and in this distribution it is considered Gearman::Client.

For comparison, Template-Toolkit has NAME set to 'Template' and
libwww-perl has NAME set to LWP.

This information doesn't do anything useful other
than making an entry in MYMETA files, perllocal.pod and (with cpanm)
install.json metadata.

I keep DISTNAME to Gearman so the tarball is still
Gearman-VERSION.tar.gz and that makes distribution information and
URLs on CPAN not change at all.
@miyagawa
Copy link
Author

the real pain that can be remedied with this patch is described at perl-carton/carton#34

@miyagawa
Copy link
Author

Bump?

p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Oct 25, 2016
p-alik added a commit to p-alik/perl-Gearman that referenced this pull request Nov 22, 2016
@p-alik
Copy link

p-alik commented Dec 4, 2016

@miyagawa I did what you proposed but I have to revert to the old name because neither cpan nor testers identify the package by the name Gearman::Client.
In once of upcoming releases I'll rename base module Objects.pm to Gearman.pm to solve the issue in an other way.

@miyagawa
Copy link
Author

miyagawa commented Dec 4, 2016

What do you mean by "CPAN nor testers identify the package by the name?" If you mean users of the module do not depend on this district the name "German::Client" there is no way you can add this module as a dependency.

@p-alik
Copy link

p-alik commented Dec 4, 2016

There are two different distributions on CPAN now. See CPAN User Directory. Same in cpantesters matrix:

I'm going to ask CPAN admins what is the best approach in this situation and I'll replay to you when I get an answer.

@miyagawa
Copy link
Author

miyagawa commented Dec 4, 2016

That is because you changed the distribution name to -Client despite my original intent to keep it Gearman. If you have used make dist from the MakeMaker it would have generated it correctly.

@p-alik
Copy link

p-alik commented Dec 4, 2016

make dist produces Gearman-Client-*.tar.gz with regard to the NAME in Makefile.PL

@miyagawa
Copy link
Author

miyagawa commented Dec 4, 2016 via email

@p-alik
Copy link

p-alik commented Dec 4, 2016

If you put DISTNAME it should be preferred.

That's it! Many thanks for the advice.

@miyagawa
Copy link
Author

miyagawa commented Dec 4, 2016

well, look at my PR, it's there already. If you were using it as is you would have saved all this hassle.

@p-alik
Copy link

p-alik commented Dec 5, 2016

You bet! I was careless.

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.

2 participants