Bug 139086

Summary: [EFL] libseccomp should pick up CC environment variable to be conform to other jhbuild modules
Product: WebKit Reporter: Csaba Osztrogonác <ossy>
Component: New BugsAssignee: Csaba Osztrogonác <ossy>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, gyuyoung.kim, ossy
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Csaba Osztrogonác
Reported 2014-11-28 02:07:46 PST
[EFL] libseccomp should pick up CC environment variable to be conform to other jhbuild modules
Attachments
Patch (2.27 KB, patch)
2014-11-28 02:08 PST, Csaba Osztrogonác 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Csaba Osztrogonác
Comment 1 2014-11-28 02:08:04 PST
Created attachment 242263 [details] Patch
Csaba Osztrogonác
Comment 2 2014-11-28 02:25:09 PST
To build with clang instead of gcc, we usually set CC=clang, CXX=clang++ and ICECC_VERSION=clang-toolchain.tar.gz. Unfortunately libseccomp doesn't respect CC environment variable and calling gcc (/usr/lib/icecc/bin/gcc) when ICECC_VERSION points to a clang toolchain causes strange error messages. I think the proper fix is to pick up CC as other jhbuild modules already do.
Gyuyoung Kim
Comment 3 2014-11-28 04:26:59 PST
Comment on attachment 242263 [details] Patch rs=me.
WebKit Commit Bot
Comment 4 2014-11-28 05:41:29 PST
Comment on attachment 242263 [details] Patch Clearing flags on attachment: 242263 Committed r176562: <http://trac.webkit.org/changeset/176562>
WebKit Commit Bot
Comment 5 2014-11-28 05:41:32 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.