Skip to content

Implement ekam-client -f#65

Open
xortive wants to merge 1 commit intocapnproto:masterfrom
xortive:malonso/ekam-client-failure-mode
Open

Implement ekam-client -f#65
xortive wants to merge 1 commit intocapnproto:masterfrom
xortive:malonso/ekam-client-failure-mode

Conversation

@xortive
Copy link
Copy Markdown

@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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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