mention that --trace will be enabled by default in EasyBuild 5.x
#178
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
boegel
reviewed
Jul 5, 2023
| @@ -48,7 +46,12 @@ feedback then please comment in the GitHub issue for the proposal. | |||
|
|
|||
| Various significant enhancements are included in EasyBuild v5.0, including: | |||
|
|
|||
| * *(no significant enhancements implemented so far in the `5.0.x` branches)* | |||
| * [enable `--trace` by default][eb5_trace] | |||
There was a problem hiding this comment.
I guess you could debate whether this is an enhancement or not, rather than just a changed default, but fine as is.
boegel
requested changes
Jul 5, 2023
| ### `--trace` enabled by default {: #eb5_trace } | ||
|
|
||
| The [`--trace` option is enabled by default](https://github.com/easybuilders/easybuild-framework/pull/4250). This makes | ||
| the steps and commands that EasyBuild is running clearer to the user. |
There was a problem hiding this comment.
Should include a link to https://docs.easybuild.io/tracing-progress, and we should mention how to configure EasyBuild to not produce trace output, in case someone gets annoyed by it?
boegel
requested changes
Jul 5, 2023
| The [`--trace` option is enabled by default](https://github.com/easybuilders/easybuild-framework/pull/4250). This makes | ||
| the [steps and commands that EasyBuild is running clearer][trace] to the user. | ||
|
|
||
| Use `--disable-trace` to remove this extra information. |
There was a problem hiding this comment.
I would mention all 3 ways to disable trace output:
To disable trace output, either:
- Use the
--disable-tracecommand line option; - Set the
$EASYBUILD_DISABLE_TRACEenvironment variable; - Disable trace mode in a configuration file:
[override]
trace=0
Co-authored-by: Kenneth Hoste <kenneth.hoste@ugent.be>
boegel
changed the title
--trace enabled by default--trace will be enabled by default in EasyBuild 5.x
boegel
approved these changes
Sep 13, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.