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

Honor EUMM's DESTDIR option #605

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

reyjrar
Copy link

@reyjrar reyjrar commented Jan 31, 2020

Maybe I'm crazy, but I'm using cpanm inside of a mock chroot to build packages. The idea is to use Carton to package a script and it's dependencies as an RPM that can be deployed and run with carton exec. The first step is getting cpanminus to work inside the build environment. This patch will enable both cpanm and carton to correctly install and find their installed modules when the target install directory is the /build/builddir/<package-name>-<package-version> that exists while staging files for an RPM build.

Maybe I'm crazy, but I'm using `cpanm` inside of a `mock` chroot to
build packages.  The idea is to use `Carton` to package a script and
it's dependencies as an RPM that can be deployed and run with `carton
exec`.  The first step is getting `cpanminus` to work inside the build
environment.  This patch will enable both `cpanm` and `carton` to
correctly install and find their installed modules when the target
install directory is the
`/build/builddir/<package-name>-<package-version>` that exists while
staging files for an RPM build.
@reyjrar
Copy link
Author

reyjrar commented Jan 31, 2020

I should add this differs from #583 as I'm trying to bootstrap a package building setup. I'm installing Perl into:

/opt/perl/530/bin/perl

And need to provide Carton to the build system and as a runtime requirement. The rest of the applications will be OK to use local::lib style directories, it's just Carton (possibly Carmel) that need to succeed the initial App::cpanminus bootstrap.

@Leont
Copy link

Leont commented Apr 27, 2024

A better approach might be to use ExtUtils::InstallPaths with PERL_MB_OPT. That way you support all install path options in one go. Though in practice install_base and destdir are probably the only two options people actually use so it might not really matter.

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.

2 participants