New issue
Advanced search Search tips

Issue 622477 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Jun 2016
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

host package uses the wrong clang compiler

Project Member Reported by yunlian@chromium.org, Jun 22 2016

Issue description

USE="clang" sudo emerge btsocket


 creating build/temp.linux-x86_64-2.7/src
btsocket-0.0.1-r11: x86_64-pc-linux-gnu-clang -O2 -pipe -fPIC -I/usr/include/python2.7 -c src/btsocket.c -o build/temp.linux-x86_64-2.7/src/btsocket.o
btsocket-0.0.1-r11: unable to execute 'x86_64-pc-linux-gnu-clang': No such file or directory
btsocket-0.0.1-r11: error: command 'x86_64-pc-linux-gnu-clang' failed with exit status 1

This is because the function clang-setup-env will set the CC to
"${CHOST}-clang". For host, there is no such wrapper, so the error happens.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 23 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/b96171736c728a8fedee249523c9b6a99ba5c488

commit b96171736c728a8fedee249523c9b6a99ba5c488
Author: Yunlian Jiang <yunlian@google.com>
Date: Wed Jun 22 22:10:49 2016

toolchain-funcs.eclass: do not accept "USE=clang" for host package.

When building host package with USE=clang, it will set the compiler
to x86_64-pc-linux-gnu-clang. There is no such wrapper and the build
fails. This CL stops to set CC for host package even USE=clang is set.

BUG= chromium:622477 
TEST=sudo USE="clang" emerge btsocket passes.

Change-Id: I5c7621cd6e81fde9d1581a37b1b7c4c8800ee99b
Reviewed-on: https://chromium-review.googlesource.com/355171
Commit-Ready: Yunlian Jiang <yunlian@chromium.org>
Tested-by: Yunlian Jiang <yunlian@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Yunlian Jiang <yunlian@chromium.org>

[modify] https://crrev.com/b96171736c728a8fedee249523c9b6a99ba5c488/eclass/toolchain-funcs.eclass

Status: Fixed (was: Untriaged)
Status: Verified (was: Fixed)
Bulk verified

Sign in to add a comment