Skip to content

Cherry-pick Fix --quiet option not working with swift run #8844 (#8858) #8907

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

Open
wants to merge 6 commits into
base: release/6.2
Choose a base branch
from

Conversation

dschaefer2
Copy link
Member

Cherry-pick #8858 to 6.2.

Fixes an issue where the --quiet option had no effect when used with swift run and swift build command.

Motivation:

When using the --quiet option with swift run only error messages should be logged.

Modifications:

Add isQuiet property to filter out messages based on the log level during build phases of the native, xcode and swiftbuild build system.
Remove duplicated declaration of the isVerbose property from different targets.
Result:

When the --quiet option is used with the swift run and swift build command only error messages are logged.

…ng#8858)

Fixes an issue where the `--quiet` option had no effect when used with
`swift run` and `swift build` command.

### Motivation:

When using the `--quiet` option with `swift run` only error messages
should be logged.

### Modifications:

- Add `isQuiet` property to filter out messages based on the log level
during build phases of the `native`, `xcode` and `swiftbuild` build
system.
- Remove duplicated declaration of the isVerbose property from different
targets.

### Result:

When the `--quiet` option is used with the `swift run` and `swift build`
command only error messages are logged.
@dschaefer2
Copy link
Member Author

Hmm, tests failing on Linux. I'll double check.

@dschaefer2
Copy link
Member Author

@swift-ci please test

@dschaefer2
Copy link
Member Author

@swift-ci please test windows

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants