Closed Bug 1471638 Opened 2 years ago Closed 2 years ago
--enable-fuzzing coverage build is inconsistently named
linux64-fuzzing-asan-opt-cov should be named either: linux64-fuzzing-ccov-asan-opt or linux64-ccov-fuzzing-asan-opt for consistency with other builds. -cov is not used for any other coverage builds, and breaks tools like https://github.com/MozillaSecurity/fuzzfetch which look for -ccov in the namespace.
Comment on attachment 8988264 [details] bug 1471638 - fix name for coverage fuzzing build https://reviewboard.mozilla.org/r/253530/#review260176 ::: taskcluster/ci/build/linux.yml:615 (Diff revision 1) > config: > - builds/releng_base_firefox.py > - builds/releng_base_linux_64_builds.py > script: "mozharness/scripts/fx_desktop_build.py" > secrets: true > custom-build-variant-cfg: fuzzing-cov Do you want to change this too? I guess it doesn't matter for tools like fuzzfetch though.
Attachment #8988264 - Flags: review?(mcastelluccio) → review+
I downloaded the build from this changeset on try using fuzzfetch and it's recognized as a coverage build. Thanks!
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/0a6be4e929e3 fix name for coverage fuzzing build r=marco
You need to log in before you can comment on or make changes to this bug.