-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
autoconf: fix Linux build #25610
autoconf: fix Linux build #25610
Conversation
Notifying maintainers: |
If there are any objections, I do not insist on anything for Linux, but this does fix the issue for me. |
depends_build-append \ | ||
port:help2man | ||
# https://trac.macports.org/ticket/70673 | ||
configure.perl ${prefix}/bin/perl5 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
help2man depends on perl5.34. perl5.34 provides the perl5.34 executable. The perl5 symlink is provided by the perl5 port. If you want to use the perl5 symlink here you have to depend on the perl5 port.
ping @barracuda156 - please address the comment by @ryandesign |
@reneeotten @ryandesign Yeah, will do this tomorrow. Sorry for a delay, Linux is not really a priority, but I want basic ports to work. |
15f3ed4
to
bba7583
Compare
@reneeotten Done, sorry for a delay here. |
You mixed up two changes in the last commit and the commit message does not reflect that. The "perl5" dependency for "autoconf" should go in the first commit. |
Closes: https://trac.macports.org/ticket/70673
Description
Non-macOS-only:
Use MacPorts perl5 on non-Darwin systems.
Also it may need
help2man
.Type(s)
Tested on
Linux riscv64
Verification
Have you
port lint --nitpick
?sudo port test
?sudo port -vst install
?