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

perl-Mail-SPF: fix lib path #29904

Closed
wants to merge 1 commit into from
Closed

perl-Mail-SPF: fix lib path #29904

wants to merge 1 commit into from

Conversation

valeriangalliat
Copy link
Contributor

@valeriangalliat valeriangalliat commented Sep 28, 2017

Motivation for this change

The MailSPF Perl module can't be included as a library in other Perl modules because it puts library code in $out/lib/perl5 instead of $out/lib/perl5/site_perl.

I'm not familiar with Perl and the way Nixpkgs works with Perl, but basically, without this patch, having MailSPF in buildInputs doesn't add it to the PERL5LIB environment variable, while all other Perl dependencies in buildInputs that install library code to $out/lib/perl5/site_perl are automatically added to PERL5LIB. After this patch, MailSPF is properly added to PERL5LIB.

Things done

Force the install library path to be $out/lib/perl5/site_perl.

@rycee
Copy link
Member

rycee commented Sep 28, 2017

Rebased to master in f68b1c1. Many thanks for the fix!

@rycee rycee closed this Sep 28, 2017
@rycee
Copy link
Member

rycee commented Sep 28, 2017

Also cherry-picked to 17.09 in e9e037f.

@valeriangalliat valeriangalliat deleted the perl-mail-spf/lib-path branch September 29, 2017 14:52
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.

None yet

2 participants