clusterfuzzlite icon indicating copy to clipboard operation
clusterfuzzlite copied to clipboard

CFlite artifacts are called "cifuzz-****"

Open evverx opened this issue 3 years ago • 4 comments

I'm not sure how important this is but looking at https://github.com/evverx/libbpf/actions/runs/1728607427 it seems the artifacts should be called "cflite-***" instead. The only scenario I can imagine where it could break something is when both CFLite and CIFuzz are enabled at the same time though.

evverx avatar Jan 21 '22 12:01 evverx

@jonathanmetzman thoughts? It doesn't seem too important to me, and enabling both at the same time doesn't make much sense AFAICT.

oliverchang avatar Jan 24 '22 01:01 oliverchang

I think as long as CIFuzz is incompatible with forks due to https://github.com/google/oss-fuzz/issues/3731 it kind of makes sense to use both CIFuzz and CFLite. I don't think anyone does this in practice though.

evverx avatar Jan 24 '22 07:01 evverx

I think ideally we should use "clusterfuzzlite" rather than CIFuzz which isn't really the "brand" we are going to use moving forward. I can change the name but it will break existing users. Better to do this sooner rather than later but I'm leaning towards not changing it.

jonathanmetzman avatar Jan 26 '22 03:01 jonathanmetzman

There is another place where CFLite doesn't pop up even though I think it should. As far as I can tell there is no alternative to the CIFUZZ environment variable on CFLite.

evverx avatar Jan 26 '22 13:01 evverx

I think there is exactly one project where both CIFuzz and CFLite are used and I'm pretty sure it can live with artifacts named like this :-) Closing.

evverx avatar May 07 '23 22:05 evverx