Bug #88367 for App-cpanminus-reporter: cannot create a report from installing a dist from a local tarball

Mon Sep 02 21:08:16 2013 ether [...] cpan.org - Ticket created

Subject:

local dists should be ignored in the build log

I installed a dist from a local file ("cpanm ." after uploading a tarball), and then ran cpanm-reporter: Parsing error. Found '/Users/ether/git/Dist-Zilla-Plugin-Test-Compile/Dist-Zilla-Plugin-Test-Compile-2.024' without fetching first. at /Users/ether/.perlbrew/libs/19.3@std/bin/cpanm-reporter line 25. Local files should be ignored and skipped, rather than resulting in a fatal error.

Mon Sep 09 12:51:09 2013 ether [...] cpan.org - Correspondence added

On 2013-09-02 18:08:16, ETHER wrote: Show quoted text

> I installed a dist from a local file ("cpanm ." after uploading a > tarball), and then ran cpanm-reporter: > > Parsing error. Found '/Users/ether/git/Dist-Zilla-Plugin-Test- > Compile/Dist-Zilla-Plugin-Test-Compile-2.024' without fetching first. > at /Users/ether/.perlbrew/libs/19.3@std/bin/cpanm-reporter line 25. > > Local files should be ignored and skipped, rather than resulting in a > fatal error.

The same thing also happens when doing 'cpanm --installdeps .' -- there is an entry in the build.log for the local dist, even though it itself wasn't installed, only its dependencies.

Tue May 13 09:27:12 2014 PLICEASE [...] cpan.org - Correspondence added

On Mon Sep 02 21:08:16 2013, ETHER wrote: Show quoted text

> I installed a dist from a local file ("cpanm ." after uploading a > tarball), and then ran cpanm-reporter: > > Parsing error. Found '/Users/ether/git/Dist-Zilla-Plugin-Test- > Compile/Dist-Zilla-Plugin-Test-Compile-2.024' without fetching first. > at /Users/ether/.perlbrew/libs/19.3@std/bin/cpanm-reporter line 25. > > Local files should be ignored and skipped, rather than resulting in a > fatal error.

I would like local files from a minicpan mirror to be included, perhaps indicated by an option.

Tue May 13 09:27:12 2014 The RT System itself - Status changed from 'new' to 'open'

Tue May 13 11:22:11 2014 ether [...] cpan.org - Correspondence added

Subject:	Re: [rt.cpan.org #88367] local dists should be ignored in the build log
Date:	Tue, 13 May 2014 08:22:01 -0700
To:	Graham Ollis via RT <bug-App-cpanminus-reporter [...] rt.cpan.org>
From:	Karen Etheridge <ether [...] cpan.org>

On Tue, May 13, 2014 at 09:27:13AM -0400, Graham Ollis via RT wrote: Show quoted text

> I would like local files from a minicpan mirror to be included, perhaps indicated by an option.

That would be even better, if possible.

Wed Jun 18 12:28:16 2014 ether [...] cpan.org - Correspondence added

Wed Jun 18 16:22:41 2014 ether [...] cpan.org - Subject changed from 'local dists should be ignored in the build log' to 'cannot create a report from installing a dist from a local tarball'

Wed Jun 18 16:43:28 2014 ambs [...] cpan.org - Correspondence added

The example posted is not correct. Ether wanted this to work for: cpanm Dist-Zilla-PluginBundle-Author-ETHER-0.065.tar.gz for example.

Wed Jun 18 16:58:05 2014 ambs [...] cpan.org - Correspondence added

On Wed Jun 18 16:43:28 2014, AMBS wrote: Show quoted text

> The example posted is not correct. > > Ether wanted this to work for: > > cpanm Dist-Zilla-PluginBundle-Author-ETHER-0.065.tar.gz > > for example.

A first solution was to add the file:/// scheme to the code (there is a PullRequest for that already). The problem, then, it where to find the author name, for example, to build the distfile AUTHOR/DIST. If we really want to support this kind of installation, then probably we could add a --author option... But not for me to decide :-) just bringing up the problems.

Tue Jun 24 18:48:23 2014 garu [...] cpan.org - Correspondence added

Alright everyone! Thanks for the report and analysis. As it turns out, supporting local mirrors is kind of tricky because (a) you can't reliably figure out the dist's resource path on CPAN (which is ultimately where the report will point at); and (b) it opens a big window of potential mistakes where modules that are *not* on CPAN get their reports uploaded. I don't think it works even with traditional CPAN::Reporter. So I'm inclined to reject the patch :( Feel free to prove me wrong though! But let's continue this conversation in: https://github.com/garu/App-cpanminus-reporter/issues/13 Thanks!

Tue Jun 24 18:48:23 2014 garu [...] cpan.org - Status changed from 'open' to 'rejected'