Re: [PATCH 02/15] scripts/sbom: integrate script in make process

[Luis Augenstein <luis.augenstein@tngtech.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 1074 bytes --]

>> Ick, this might get messy as you can modify the compiler with the CC=
>> option to be anything.  There are other build tools out there that do
>> much the same as ccache does (which I should have caught this as I use
>> ccache on my build systems), like distcc and friends, so this might just
>> want to look at the result of "CC" instead?
> 
> Yeah, it would be much more robust to just look at $(CC) directly if it
> is set (i.e., running within Kbuild) vs. having a separate parser like
> this. If you want to keep a fallback for standalone usage for
> development and such, that's fine, but we should use the information we
> have available to be as accurate as possible.
> 
> Cheers,
> Nathan

Ok, sounds good. I will work on that. Thanks.

-- 
Luis Augenstein * luis.augenstein@tngtech.com * +49-152-25275761
TNG Technology Consulting GmbH, Beta-Str. 13, 85774 Unterföhring
Geschäftsführer: Henrik Klagges, Dr. Robert Dahlke, Thomas Endres
Aufsichtsratsvorsitzender: Christoph Stock
Sitz: Unterföhring * Amtsgericht München * HRB 135082

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3207 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]