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.

Sign up for GitHub

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

Jump to bottom

$Config{vendorlib} unset? #32

Closed
rysiekpl opened this issue Jul 23, 2017 · 2 comments · Fixed by #33
Closed

$Config{vendorlib} unset? #32

rysiekpl opened this issue Jul 23, 2017 · 2 comments · Fixed by #33

Comments

@rysiekpl
Copy link

Running perl -MConfig -e 'print $Config{vendorlib}' in the container returns an empty string. It should return a path.
rysiekpl added a commit to occrp/kuvert that referenced this issue Jul 23, 2017
@rysiekpl
a different approach for getting the install location, due to Perl/do…
f8638c9 
…cker-perl#32
@zakame
Copy link
Member

zakame commented Jul 24, 2017

Hi @rysiekpl, thanks for the report!

Yep, I think @Perl/docker doesn't define a vendorlib upon building Perl for Docker, as vendorlib is normally used mainly by OS distributions to allow them to install their Perl packages in a location separate from sitelib and privlib.

I checked on occrp/kuvert@f8638c9; since you're installing a module, you can use sitelib instead of vendorlib; I don't recommend using $INC[0] because that will usually prefer the XS path first (e.g. /usr/local/lib/perl5/site_perl/5.24.1/x86_64-linux) which may not be what you want.

Adding Configure settings for vendorlib should be easy enough though, so let me put up a PR here that we can merge in later. Thanks again!

zakame added a commit to zakame/docker-perl that referenced this issue Jul 24, 2017
@zakame
Add vendor paths
399c2a6 
Let `vendorprefix=/usr/local` so `vendorlib` and `vendorarch` can point
to paths in `/usr/local/lib/perl5/vendor_perl`.  Extend the generator.pl
also so we can add more `Configure` settings if we needed it later.

Fixes Perl#32.
@zakame zakame mentioned this issue Jul 24, 2017
Merged
@rysiekpl
Copy link
Author

Good suggestion, thanks!

rysiekpl added a commit to occrp/kuvert that referenced this issue Jul 24, 2017
@rysiekpl
Makefile: using a saner install path as suggested by @zakame
ba72137 
Perl/docker-perl#32 (comment)
zakame added a commit that referenced this issue Jul 28, 2017
@zakame
Add vendor paths
78ff90a 
Let `vendorprefix=/usr/local` so `vendorlib` and `vendorarch` can point
to paths in `/usr/local/lib/perl5/vendor_perl`.  Extend the generator.pl
also so we can add more `Configure` settings if we needed it later.

Fixes #32.
zakame added a commit that referenced this issue Jul 28, 2017
@zakame
Update Dockerfiles for -Dvendorprefix
6ec3445 
Needed for #32 so all Perl images support installing to
`/usr/local/lib/perl5/vendor_perl`.
@tianon tianon mentioned this issue Sep 30, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add vendor paths zakame/docker-perl
2 participants
@zakame @rysiekpl