Bug #99432 for Git-Sub: indexing conflict with Git dist

Sat Oct 11 10:07:28 2014 msouth [...] gmail.com - Ticket created

Hi, In January I sent the below email to your cpan address. I have not heard back. I ended up just bumping Git.pm's version number to .40 to leapfrog your .130270 version mentioned below. I think you might be able to just delete the $git::VERSION line and everything should still work? You have a VERSION for Git::Sub and I don't think you're using $git::VERSION anywhere. However, I'm not a big PAUSE expert so let me know if there's something else that we need to do to coordinate. ack VERSION ./lib/Git/Sub.pm: $Git::Sub::VERSION = '0.130270'; ./lib/Git/Sub.pm: $git::VERSION = '0.130270'; ./lib/Git/Sub.pm:=head1 VERSION ./Makefile.PL: "VERSION" => "0.130270", ./Makefile.PL:unless ( eval { ExtUtils::MakeMaker->VERSION(6.56) } ) { ./Makefile.PL: unless eval { ExtUtils::MakeMaker->VERSION(6.52) }; Hi Olivier, I received the following notice when I tried to upload the latest version of Git.pm from the git distribution: Status: Decreasing version number ================================= module: Git version: 0.03 in file: Git-0.03/lib/Git.pm status: Not indexed because Git-Sub-0.130270/lib/Git/Sub.pm in D/DO/DOLMEN/Git-Sub-0.130270.tar.gz has a higher version number (0.130270) My purpose in putting Git.pm on CPAN is to make it so that people using things like a perlbrewed perl can get Git from CPAN to satisfy the dependency chain when using things like Git::Hooks. I believe/guess that this section of code in your module: package git; { $git::VERSION = '0.130270'; } is confusing the CPAN indexer into thinking your module is a more recent version of Git.pm than the one I am trying to upload. I don't know why, since the cases don't match. Perhaps it's a bug in the indexer. Does your code need that bit for something? Is there a way to do it that doesn't make it look to the PAUSE system as if you're setting $Git::VERSION? Thanks for any help you might be able to give with this. mike

Sat Oct 11 14:03:37 2014 ether [...] cpan.org - Subject changed from (no value) to 'bad versions'

Thu Apr 23 06:15:09 2015 dolmen [...] cpan.org - Correspondence added

Subject:	indexing conflict with Git dist
RT-Send-CC:	ether [...] cpan.org

Le 2014-10-11 16:07:28, MSOUTH a écrit :
Show quoted text

> Hi,

Hi,

My Git::Sub module injects functions in the git:: (lower case) namespace.

So that is on purpose that I have $git::VERSION set in Git::Sub, in order to reserve the git:: namespace on PAUSE.

But my distribution is a bit special as I reserve the 'git::' namespace without providing a git.pm file. I don't intend to provide one just to leave filesystem space for Git.pm (on case-sensitive filesystems).

PAUSE tells me that I have rights on git:: and you have rights on Git:: so I don't see an issue.
https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=git
https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=Git

If you still have issues publishing your Git distribution, fill an issue on PAUSE and add my as a Cc (write "Cc: @dolmen" in the issue or in a comment) :
https://github.com/andk/pause/issues

Olivier.

Show quoted text

>
> In January I sent the below email to your cpan address. I have not
> heard back. I ended up just bumping Git.pm's version number to .40 to
> leapfrog your .130270 version mentioned below.
>
> I think you might be able to just delete the $git::VERSION line and
> everything should still work? You have a VERSION for Git::Sub and I
> don't think you're using $git::VERSION anywhere. However, I'm not a
> big PAUSE expert so let me know if there's something else that we need
> to do to coordinate.
>
> ack VERSION
> ./lib/Git/Sub.pm: $Git::Sub::VERSION = '0.130270';
> ./lib/Git/Sub.pm: $git::VERSION = '0.130270';
> ./lib/Git/Sub.pm:=head1 VERSION
> ./Makefile.PL: "VERSION" => "0.130270",
> ./Makefile.PL:unless ( eval { ExtUtils::MakeMaker->VERSION(6.56) } ) {
> ./Makefile.PL: unless eval { ExtUtils::MakeMaker->VERSION(6.52) };
>
> Hi Olivier,
>
> I received the following notice when I tried to upload the latest
> version of Git.pm from the git distribution:
> Status: Decreasing version number
>
>

> =================================

>
>
>
> module: Git
>
>
> version: 0.03
> in file: Git-0.03/lib/Git.pm
>
>
> status: Not indexed because Git-Sub-0.130270/lib/Git/Sub.pm in
>
>
> D/DO/DOLMEN/Git-Sub-0.130270.tar.gz has a higher version
>
>
> number (0.130270)
> My purpose in putting Git.pm on CPAN is to make it so that people
> using things like a perlbrewed perl can get Git from CPAN to satisfy
> the dependency chain when using things like Git::Hooks.
> I believe/guess that this section of code in your module:
> package git;
> {
> $git::VERSION = '0.130270';
> }
> is confusing the CPAN indexer into thinking your module is a more
> recent version of Git.pm than the one I am trying to upload.
>
> I don't know why, since the cases don't match. Perhaps it's a bug in
> the indexer.
>
>
>
> Does your code need that bit for something? Is there a way to do it
> that doesn't make it look to the PAUSE system as if you're setting
> $Git::VERSION?
>
> Thanks for any help you might be able to give with this.
>
>
>
> mike

--
Olivier Mengué - http://perlresume.org/DOLMEN - https://gratipay.com/dolmen/

Thu Apr 23 06:15:09 2015 The RT System itself - Status changed from 'new' to 'open'

Thu Apr 23 06:15:40 2015 dolmen [...] cpan.org - Subject changed from 'bad versions' to 'indexing conflict with Git dist'

Thu Apr 23 06:15:52 2015 dolmen [...] cpan.org - Broken in 0.130170 deleted

Thu Apr 23 06:15:53 2015 dolmen [...] cpan.org - Broken in 0.130270 deleted

Thu Apr 23 15:55:36 2015 ether [...] cpan.org - Correspondence added

RT-Send-CC:

modules [...] perl.org

On 2015-04-23 03:15:09, DOLMEN wrote: Show quoted text

> Le 2014-10-11 16:07:28, MSOUTH a écrit :

> > Hi,

> > Hi, > > My Git::Sub module injects functions in the git:: (lower case) > namespace. > > So that is on purpose that I have $git::VERSION set in Git::Sub, in > order to > reserve the git:: namespace on PAUSE. > > But my distribution is a bit special as I reserve the 'git::' > namespace without > providing a git.pm file. I don't intend to provide one just to leave > filesystem > space for Git.pm (on case-sensitive filesystems). > > PAUSE tells me that I have rights on git:: and you have rights on > Git:: so I > don't see an issue. > https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=git > https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=Git

Namespaces in PAUSE are case-insensitive, because some filesystems are case-insensitive. However, it looks like the 06perms.txt file (and the underlying database) are organized in a case sensitive fashion, so it looks like the two modules are entirely separate, but they're not. I'm cc-ing modules@perl.org as it looks like you'll need to get this sorted out at the PAUSE level.

Tue Jun 28 09:40:10 2016 dolmen [...] cpan.org - Correspondence added

RT-Send-CC:

modules [...] perl.org

Le 2015-04-23 21:55:36, ETHER a écrit :
Show quoted text

> On 2015-04-23 03:15:09, DOLMEN wrote:

> > Le 2014-10-11 16:07:28, MSOUTH a écrit :

> > > Hi,

> >
> > Hi,
> >
> > My Git::Sub module injects functions in the git:: (lower case)
> > namespace.
> >
> > So that is on purpose that I have $git::VERSION set in Git::Sub, in
> > order to
> > reserve the git:: namespace on PAUSE.
> >
> > But my distribution is a bit special as I reserve the 'git::'
> > namespace without
> > providing a git.pm file. I don't intend to provide one just to leave
> > filesystem
> > space for Git.pm (on case-sensitive filesystems).
> >
> > PAUSE tells me that I have rights on git:: and you have rights on
> > Git:: so I
> > don't see an issue.
> > https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=git
> > https://pause.perl.org/pause/authenquery?pause99_peek_perms_by=ml&pause99_peek_perms_query=Git

>
> Namespaces in PAUSE are case-insensitive, because some filesystems are
> case-insensitive.

If the issue is about filesystems, this shows that permissions should be applied to file paths, not to package names.
This conflict would not happen if that was the case because my distribution does not provides a "git.pm" file.

I did not intent to export $git::VERSION ($Git::Sub::VERSION is the one that matter for this distribution), but it has been injected by my Dist::Zilla build.
I will make a new release that does not have $git::VERSION.

--
Olivier Mengué - http://perlresume.org/DOLMEN

Thu Jun 30 04:15:14 2016 dolmen [...] cpan.org - Correspondence added

Le 2016-06-28 15:40:10, DOLMEN a écrit :
Show quoted text

>
> I did not intent to export $git::VERSION ($Git::Sub::VERSION is the
> one that
> matter for this distribution), but it has been injected by my
> Dist::Zilla
> build.
> I will make a new release that does not have $git::VERSION.

Release 0.161810 is on CPAN.
https://metacpan.org/changes/distribution/Git-Sub

I've also asked PAUSE to remove Git-Sub-0.130270.tar.gz

--
Olivier Mengué - http://perlresume.org/DOLMEN

Thu Jun 30 04:15:25 2016 dolmen [...] cpan.org - Fixed in 0.161810 added

Tue Jul 05 05:18:00 2016 NEILB [...] cpan.org - Correspondence added

Thanks Olivier!

Fri Sep 06 17:03:22 2019 dolmen [...] cpan.org - Correspondence added

--
Olivier Mengué - http://perlresume.org/DOLMEN

Fri Sep 06 17:03:23 2019 dolmen [...] cpan.org - Status changed from 'open' to 'resolved'

Fri Sep 06 17:03:23 2019 dolmen [...] cpan.org - Taken

Fri Sep 06 17:03:23 2019 dolmen [...] cpan.org - Fixed in 0.130270 added

Fri Sep 06 17:03:23 2019 dolmen [...] cpan.org - Fixed in 0.161810 deleted