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-build: avoid env script for plenv- scripts #204683

Merged
merged 2 commits into from
Jan 19, 2025
Merged

Conversation

cho-m
Copy link
Member

@cho-m cho-m commented Jan 17, 2025

This allows plenv help to work correctly. Only perl-build script needs the Perl modules. plenv-install runs perl-build while plenv-uninstall is just a bash script that runs rm and plenv.

Also expose perl-build.1 man page.

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

Closes #202333

@github-actions github-actions bot added the perl Perl use is a significant feature of the PR or issue label Jan 17, 2025
@cho-m cho-m force-pushed the perl-build-updates branch from 810f10b to ca64821 Compare January 18, 2025 04:43
This allows `plenv help` to work correctly. Only `perl-build` script
needs the Perl modules. `plenv-install` runs `perl-build` while
`plenv-uninstall` is just a bash script that runs `rm` and `plenv`.

Also expose `perl-build.1` man page.
@cho-m cho-m force-pushed the perl-build-updates branch from ca64821 to 9d18c45 Compare January 18, 2025 05:34
@cho-m cho-m marked this pull request as ready for review January 18, 2025 19:37
Copy link
Contributor

🤖 An automated task has requested bottles to be published to this PR.

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Jan 19, 2025
@BrewTestBot BrewTestBot enabled auto-merge January 19, 2025 07:21
@BrewTestBot BrewTestBot added this pull request to the merge queue Jan 19, 2025
Merged via the queue into master with commit 9700b78 Jan 19, 2025
15 checks passed
@BrewTestBot BrewTestBot deleted the perl-build-updates branch January 19, 2025 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. perl Perl use is a significant feature of the PR or issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

plenv install usage is not displayed correctly
3 participants