Patchwork D10066: fuzz: if the caller of our makefile sets CC and CXX, trust them

login
register
mail settings
Submitter phabricator
Date Feb. 24, 2021, 5:41 p.m.
Message ID <differential-rev-PHID-DREV-etwfqlklcnfd2z7zp4uc-req@mercurial-scm.org>
Download mbox | patch
Permalink /patch/48381/
State Superseded
Headers show

Comments

phabricator - Feb. 24, 2021, 5:41 p.m.
durin42 created this revision.
Herald added a reviewer: hg-reviewers.
Herald added a subscriber: mercurial-patches.

REVISION SUMMARY
  This should fix the broken fuzzing build, because we've been
  explicitly using clang++ but are now being given a CXX=afl++, which
  does extra stuff.

REPOSITORY
  rHG Mercurial

BRANCH
  default

REVISION DETAIL
  https://phab.mercurial-scm.org/D10066

AFFECTED FILES
  contrib/fuzz/Makefile

CHANGE DETAILS




To: durin42, #hg-reviewers
Cc: mercurial-patches, mercurial-devel

Patch

diff --git a/contrib/fuzz/Makefile b/contrib/fuzz/Makefile
--- a/contrib/fuzz/Makefile
+++ b/contrib/fuzz/Makefile
@@ -1,5 +1,5 @@ 
-CC = clang
-CXX = clang++
+CC ?= clang
+CXX ?= clang++
 
 # By default, use our own standalone_fuzz_target_runner.
 # This runner does no fuzzing, but simply executes the inputs