Skip Menu |
 

This queue is for tickets about the App-FatPacker CPAN distribution.

Report information
The Basics
Id: 110299
Status: resolved
Priority: 0/
Queue: App-FatPacker

People
Owner: ether [...] cpan.org
Requestors: skaji [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: bin/fatpack conflicts with x_static_install = 1
Download (untitled) / with headers
text/plain 748b
From my understanding, if you specify x_static_install = 1 in META.json, executable files must be located in script/ directory. In fact, cpanm-menlo (which supports x_static_install) does not install bin/fatpack: ``` $ cpanm-menlo -nq -Llocal App::FatPacker Successfully installed App-FatPacker-0.010004 1 distribution installed $ find . -type f ./local/lib/perl5/App/FatPacker/Trace.pm ./local/lib/perl5/App/FatPacker.pm ./local/lib/perl5/darwin-2level/.meta/App-FatPacker-0.010004/install.json ./local/lib/perl5/darwin-2level/.meta/App-FatPacker-0.010004/MYMETA.json ./local/lib/perl5/darwin-2level/auto/App/FatPacker/.packlist ``` So * Could you remove x_static_install = 1 from META.json? * Or could you locate fatpack in script/ directory?
Download (untitled) / with headers
text/plain 432b
On 2015-12-12 20:28:32, SKAJI wrote: Show quoted text
> From my understanding, if you specify x_static_install = 1 in > META.json, executable files must be located in script/ directory.
While that restriction sounds reasonable, I do not recall that being on the list of proposed restrictions. The only written specification that I am aware of is here: https://metacpan.org/pod/Dist::Zilla::Plugin::StaticInstall#DESCRIPTION Do you know of another?
Download (untitled) / with headers
text/plain 994b
ether, thank you for your reply. First, the word "must" I used was inappropriate. It seems that Menlo refers to the behavior of Module::Build::Tiny (https://github.com/miyagawa/cpanminus/pull/467) and Module::Build::Tiny says "Any executables must be in script/" in https://metacpan.org/pod/Module::Build::Tiny#Directory-structure Actually I don't have strong opinion about where executable files are located, but let's add a note for directory structure to specification of static install. On 2015-12月-13 日 01:00:06, ETHER wrote: Show quoted text
> On 2015-12-12 20:28:32, SKAJI wrote:
> > From my understanding, if you specify x_static_install = 1 in > > META.json, executable files must be located in script/ directory.
> > While that restriction sounds reasonable, I do not recall that being > on the list of proposed restrictions. > The only written specification that I am aware of is here: > https://metacpan.org/pod/Dist::Zilla::Plugin::StaticInstall#DESCRIPTION > > Do you know of another?
Download (untitled) / with headers
text/plain 291b
On 2015-12-13 08:02:27, SKAJI wrote: Show quoted text
> Oh, I found this: > https://github.com/Leont/cpan- > static/blob/master/lib/CPAN/Static/Spec.pm#L71-L78
Ok super! I'll update App::FatPacker, and also update my dzil plugin to check for the execdir and to point to cPAN::Static::Spec in documentation.
I've removed the meta flag in 0.010005.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.