Skip to content

Conversation

tresat
Copy link
Member

@tresat tresat commented Jun 16, 2022

Fixes #20964

…help messages if the build should fail with an exception
@tresat tresat added a:feature A new functionality in:console output progress tty mintty cygwin cli labels Jun 16, 2022
@tresat tresat added this to the 7.6 RC1 milestone Jun 16, 2022
@tresat tresat requested a review from a team June 16, 2022 20:49
@tresat tresat self-assigned this Jun 16, 2022
@tresat
Copy link
Member Author

tresat commented Jun 16, 2022

@bot-gradle test QFL

@bot-gradle
Copy link
Collaborator

OK, I've already triggered the following builds for you:

@big-guy
Copy link
Member

big-guy commented Jun 24, 2022

I don't think we should do this, but I agree that the current output could be more useful. I'd rather make the output more useful in some small way over turning it off entirely.

@big-guy big-guy modified the milestones: 7.6 RC1, 8.0 RC1 Jul 1, 2022
@tresat tresat removed this from the 8.0 RC1 milestone Oct 21, 2022
@github-actions
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. Given the limited bandwidth of the team, it will be closed if no further activity occurs. If you intend to work on this pull request, please ask the team to reopen the PR or push a new PR. Thank you for your contributions.

@github-actions github-actions bot added the stale label Dec 20, 2022
@tresat tresat removed the stale label Dec 22, 2022
@doublep
Copy link

doublep commented Oct 13, 2023

For what it's worth Gradle's own help output contains this:

-q, --quiet                        Log errors only.

I don't see how this annoying and repetetive help message qualifies as an "error". I would rather prefer that -q suppressed it, just like the help actually promises. If I execute ./gradlew -q run and the application dies with an exception, I don't want to see ten lines of useless crap that does nothing but distract me of actual application output. It's not like -q is the default, so if I put it there then it is an active user decision, but apparently it only suppresses build progress output.

Tested with Gradle 8.4.

@big-guy big-guy removed the request for review from a team October 30, 2023 16:44
@donat
Copy link
Member

donat commented Nov 14, 2023

One thing to note here is that there's an inconsistency in the build output when --quiet is present. For successful builds, the BUILD SUCCESSFUL message is not printed. For failing builds, the BUILD FAILED message is present.

I'm leaning towards implementing the proposed behavior.

At the same time, we currently don't have the capacity to work on this. Would anyone be interested in creating a pull request?

@donat donat added this to the 9.0 RC1 milestone Nov 14, 2023
@tresat
Copy link
Member Author

tresat commented Mar 29, 2024

Related: #28373.

I think we should try to get something done IRT this. There seems to be a good amount of sustained interest.

@big-guy big-guy removed this from the 9.0 RC1 milestone Apr 21, 2025
Copy link
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. If you don't want the stale bot to close it, then set a milestone for it.

@github-actions github-actions bot added the stale label May 21, 2025
@tresat
Copy link
Member Author

tresat commented May 21, 2025

@donat - You closed the linked issue as Not Planned. Should this be closed, too?

@tresat tresat added this to the 9.x milestone May 21, 2025
@big-guy big-guy closed this Jun 4, 2025
@big-guy big-guy removed this from the 9.x milestone Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a:feature A new functionality in:console output progress tty mintty cygwin cli stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Turn off "what went wrong" message

5 participants