Skip to content
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

Implement ekam-client -f #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

xortive
Copy link

@xortive xortive commented Feb 24, 2022

This mode prints only failures, and prints failures when they happen
(meaning even when ekam is not sure if they are a true failure or caused
by a missing dependency.) This is noisy as there are a lot of "false"
errors, but is an easy way to get much quicker feedback when iterating
on headers that are included in lots of places, where before this you'd
otherwise have to wait for the entire build to settle before seeing errors.

This mode prints only failures, and prints failures when they happen
(meaning even when ekam is not sure if they are a true failure or caused
by a missing dependency.) This is noisy as there are a lot of "false"
errors, but is an easy way to get *much* quicker feedback when iterating
on headers that are included in lots of places, where before this you'd
otherwise have to wait for the entire build to settle before seeing errors.
@kentonv
Copy link
Member

kentonv commented Feb 28, 2022

Hmm, why would you want to hide the successful actions? If anything, that seems like it should be a separate flag?

@kentonv
Copy link
Member

kentonv commented Feb 28, 2022

I'd say the two flags should be:

  • -q: Quiet mode: Don't show successful actions. Only show errors.
  • -b: Show blocked: Show error logs from "blocked" actions which may resolve later in the build.

If it's just one flag then it should be:

  • -M: Matt mode. Presents output the way Matt likes.

:)

@xortive
Copy link
Author

xortive commented Feb 28, 2022

I'm happy to split them up, as much as I would love to have a -M be part of ekam :p

I want to hide the successful actions because I can refer to the main ekam output to see that if I want -- my use case for this command is when I'm iterating on headers, so usually all I care about is seeing compiler output as quickly as possible.

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.

2 participants