-
Notifications
You must be signed in to change notification settings - Fork 15
Don't print messages from hook in --terse mode
#82
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
Conversation
The output from the hook such as "Using custom configure options" makes the output not machine-readable which it should be in `--terse` mode. Check the option in a hook-function before calling the original one
ocaisa
left a comment
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.
LGTM it will need a sync with the target branch (after #85 is merged)
|
@Flamefire Can you sync this now and I will get it deployed |
|
bot: build repo:eessi.io-2025.06-software instance:eessi-bot-mc-aws for:arch=x86_64/amd/zen2 |
|
New job on instance
|
|
New job on instance
|
|
The builds in EESSI/software-layer#1177 (comment) (which use EB 4.8.2) are failing with: I'm not sure when that option was added, but it looks like we need to be more careful here? edit: this was added in EB 4.9.1, see easybuilders/easybuild-framework#4407. another edit: #90 should fix this. |
The output from the hook such as "Using custom configure options" makes the output not machine-readable which it should be in
--tersemode.Check the option in a hook-function before calling the original one