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

more specific compiler warning string grep #34

Open
nickdesaulniers opened this issue Feb 20, 2019 · 5 comments
Open

more specific compiler warning string grep #34

nickdesaulniers opened this issue Feb 20, 2019 · 5 comments

Comments

@nickdesaulniers
Copy link
Contributor

for kernelCI, I see reports like:

Warnings summary:

    111  1 warning generated.
    14   2 warnings generated.
    7    drivers/net/ethernet/intel/iavf/iavf_osdep.h:49:18: warning: 'format' attribute argument not supported: gnu_printf [-Wignored-attributes]
...
    4    3 warnings generated.
    3    4 warnings generated.
...
   2    drivers/char/hw_random/optee-rng.c:177:31: warning: suggest braces around initialization of subobject [-Wmissing-braces]
    2    5 warnings generated.
    2    41 warnings generated.

when grepping logs for compiler warnings, I find that grepping for warning:, specifically with the end colon, helps filter to just the warnings. This would remove the count of warnings per file from the logs, which I think would be a little clearer. Also, I'm not sure if the "warnings generated" lines are polluting the total warning count at the top of the report? cc @khilman @broonie

@gctucker gctucker transferred this issue from kernelci/kernelci-core-staging Mar 22, 2019
@nickdesaulniers
Copy link
Contributor Author

obsoleted by: kernelci/kernelci-backend#144?

@gctucker
Copy link
Contributor

This issue is about build reports, whereas kernelci/kernelci-backend#144 is about test reports. So it still needs to be resolved.

@gctucker gctucker changed the title more specific warning string grep more specific compiler warning string grep Jul 29, 2019
@gctucker gctucker added this to the Emails milestone Jul 29, 2019
@gctucker gctucker removed this from the Short email reports milestone Feb 9, 2023
@mgalka
Copy link

mgalka commented Jul 13, 2023

@gctucker @JenySadadia Is it something that should be addressed in the new pipeline?

@gctucker
Copy link
Contributor

Yes, right now the kernel builds don't parse the logs but we should be doing something about it at some point to report warnings etc.

@gctucker
Copy link
Contributor

I think however that this issue should be closed, and instead we should have a new issue as part of the backlog to implement kernel build log parsing.

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

3 participants