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

Post setup step shows help #99

Open
Kicer86 opened this issue Sep 9, 2022 · 3 comments
Open

Post setup step shows help #99

Kicer86 opened this issue Sep 9, 2022 · 3 comments

Comments

@Kicer86
Copy link

Kicer86 commented Sep 9, 2022

In logs, in post setup step of this action you can see execution of ccache --help which seem a leftover.

@vadz
Copy link
Contributor

vadz commented Sep 25, 2022

There is no ccache --help but I've seen this happen when setting verbose input for the ccache versions too old to have it. The corresponding function should probably check for ccacheKnowsVerbosityFlag before using -v...

@Kicer86
Copy link
Author

Kicer86 commented Sep 28, 2022

@vadz
Copy link
Contributor

vadz commented Sep 28, 2022

Ah, sorry, I hadn't realized that getExecOutput() also shows the output, in addition to returning it. There doesn't seem to be any builtin function in the GitHub toolset to just get the output without showing it, but one could be written, I guess.

Another possibility would be to use ccache -V, get version from the output and compare it with 4. It would still show the version of ccache in the logs, but this would be less annoying and more useful than --help output.

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

No branches or pull requests

2 participants