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
Mac modernization: allow listings 1.9, and don't allow errors in a pipe #6206
Conversation
This check breaks every time there is a new version, and is perhaps not pulling its weight.
The MacOS shell does not seem to tolerate a failure of the first part of the "make quiet" pipe.
run: make quiet | ||
|
||
- name: make (MacOS) | ||
if: matrix.cfg.os == 'macos-12' | ||
run: make full |
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.
So, this one is weird: the ||
in the Makefile seems to not work on MacOS anymore.
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.
So, what should I say?
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.
Nothing is needed, but it's an odd change of behaviour, which I imagine should show up elsewhere... but I think we can live with it.
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.
The default shell changed from bash to zsh in modern macOS versions, but that shouldn't matter. It seems more likely that the behaviour of latexmk changed so that -silent causes it to exit with status 0 even if the build failed.
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.
Ohh, that's interesting! That means we'll see the same on Linux when that version changes, I expect!
@jensmaurer We need to allow listings 1.9 for MacOS. Is there any point in having the version check at all?