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
"Missing perl5.34" errors in test suite in 8.4.0 on macOS 12 #245
Comments
The errors are no longer there when I test 8.4.0_alpha3 on montague. And that perl is there now.
Maybe that perl5.34 is supplied by Xcode and its CLT. I installed those recently so I could do the build on montague, too. Could test this by switching back to a snapshot from before the Xcode installation. ...except I'm running montague on a trial version of Parallels which doesn't support snapshots. |
As of 2024-04-04, I'm thinking the problem is something like: on the octapp build box,
So the fix here might be "tell users to install Xcode CLT if they want to run the Octave test suite", which might be legit because it might need Xcode/CLT to test building of oct and mex files anyway. Or make a customized texinfo-octapp that uses a brewed perl. Might be an option bc there's nothing else depending on it, so I wouldn't have to make additional -octapp variants of other formulas as a result. Plus, when installed in a regular homebrew, could conflict with plain texinfo.
Though I like to minimize our number of custom formulas anyway; easier to support stuff that's standard instead of forked. Hmm. How often is texinfo updated? See history here (new) and here (old). Plus several formula-only changes in 2022 and 2023.
I think I'm going to avoid the customized texinfo and stick with "just install Xcode CLT if you want to run tests or other texinfo stuff", unless some users actually request doing something different. Leaving this ticket open until I can run a couple more tests to verify that installing Xcode/CLT is what reliably supplies perl. |
In the 8.4.0_alpha1 prerelease build, which I built on macOS 14, when running this on my "montague" fresh macOS 12 Intel testbed VM, when running
__run_test_suite__
, I get a bunch of complaints about a missing/usr/bin/perl5.34
:Progress
As of 2024-04-04, I think it's that the brewed texinfo depends on the macos perl, and perl is provided by Xcode CLT, so yeah, you have to install the CLT here. Might be able to get around that by making a customized texinfo that requires Xcode CLT. See comments.
The text was updated successfully, but these errors were encountered: