Add --for-packaging option. #583
Open
+53
−32
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Used only with --local-lib-contained, it attempts to install only runtime
requirements in the local lib. All test, config, and build requirements are
installed in standard site destination. The idea is to install everything iun
a directory that can be bundled up and distributed, minimizing the amount of
unnecessary stuff to include for a purely runtime use case.
There are a couple of issues working againsrt this plan, however:
* Not all distributions properly specify requirements. Older distributions,
in particular, just list everything in the `PREREQ_PM` section of the
`Makefile.PL`, and therefor must be assumed to be runtime. See the
Devel-Caller `Makefile.PL` for an example:
https://metacpan.org/source/RCLAMP/Devel-Caller-2.06/Makefile.PL.
* There are bugs. I think I got most of it worked out, but when I tried to
use it with the Sqitch `./Build bundle` command, it ran into an error
when trying to install B::Hooks::OP::Check:
````
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
ERROR: Can't create '/B/Hooks/OP'
mkdir /B: Permission denied at /Users/david/.plenv/versions/5.28.0/lib/perl5/5.28.0/
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
```
Presumably I've just overlooked some config that needs to be overridden.
I think it's close, enough to give the idea, and might be worth revisiting or
cribbing from at some point.
|
I think you might want to check carmel. |
|
Thanks @skaji, Carmel looks useful, I'll keep an eye on it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This may or may not be a good idea, but thought it worth submitting a PR to get some feedback.
Used only with
--local-lib-contained,--for-packagingattempts to install only runtime requirements in the local lib. All test, config, and build requirements are installed in standard site destination. The idea is to install everything in a directory that can be bundled up and distributed, minimizing the amount of unnecessary stuff to include for a purely runtime use case. The use cases I'm thinking of are Homebrew formulas, minimalist Docker images (like this, and binary installers for GUI OSes like macOS and Windows.There are a couple of issues working againsrt this plan, however:
Not all distributions properly specify requirements. Older distributions,
in particular, just list everything in the
PREREQ_PMsection of theMakefile.PL, and therefor must be assumed to be runtime. See theDevel-Caller
Makefile.PLfor an example.There are bugs. I think I got most of it worked out, but when I tried to
use it with the Sqitch
./Build bundlecommand (in progress), it ran into an errorwhen trying to install B::Hooks::OP::Check:
I think it's close, enough to give the idea, and might be worth revisiting or cribbing from at some point. Thoughts?