Try to find llvm-config through clang first before looking at $PATH; r=glandium

RESOLVED FIXED in mozilla37

Status

()

Core
Build Config
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Away for a while, Assigned: Away for a while)

Tracking

unspecified
mozilla37
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Right now, if the LLVMCONFIG variable is not set in the .mozconfig, we
first look for the system default llvm-config and only then we ask clang
itself, which breaks building with the clang plugin if you make $CC and
$CXX point to a non-default clang binary.  This patch fixes the issue
by reversing the search order.
(Assignee)

Comment 1

3 years ago
Created attachment 8538228 [details] [diff] [review]
Try to find llvm-config through clang first before looking at $PATH
Attachment #8538228 - Flags: review?(mh+mozilla)
Attachment #8538228 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/4d9a7a26e09a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
(Assignee)

Updated

3 years ago
Assignee: nobody → ehsan.akhgari
You need to log in before you can comment on or make changes to this bug.