Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(718)

Issue 281753002: Bundle libc++ headers with clang. (Closed)

Created:
6 years, 7 months ago by Nico
Modified:
6 years, 4 months ago
Reviewers:
hans
CC:
chromium-reviews, eugenis+clang_chromium.org, glider+clang_chromium.org, dmikurube+clang_chromium.org, ukai+watch_chromium.org
Visibility:
Public.

Description

Bundle libc++ headers with clang. That way, -stdlib=libc++ works better, at least on OS X. BUG=371034 TEST=`echo '#include <vector>" > test.cc ; bin/clang -c test.cc -stdlib=libc++ -isysroot $(xcrun -show-sdk-path)` no longer complains about not finding the vector header on OS X. NOTRY=true Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=270095

Patch Set 1 #

Patch Set 2 : just use bootstrap #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
M tools/clang/scripts/package.sh View 1 1 chunk +4 lines, -0 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Nico
Two options: 1.) Add a "make install" for the non-bootstrap build and copy the headers ...
6 years, 7 months ago (2014-05-13 12:08:22 UTC) #1
hans
On 2014/05/13 12:08:22, Nico (traveling this week) wrote: > Two options: > 1.) Add a ...
6 years, 7 months ago (2014-05-13 14:50:34 UTC) #2
Nico
The CQ bit was checked by thakis@chromium.org
6 years, 7 months ago (2014-05-13 14:56:12 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/thakis@chromium.org/281753002/20001
6 years, 7 months ago (2014-05-13 14:56:37 UTC) #4
commit-bot: I haz the power
Change committed as 270095
6 years, 7 months ago (2014-05-13 14:58:24 UTC) #5
Nico
6 years, 4 months ago (2014-08-05 18:18:38 UTC) #6
Message was sent while issue was closed.
On 2014/05/13 12:08:22, Nico (away) wrote:
> Two options:
> 1.) Add a "make install" for the non-bootstrap build and copy the headers from
> there
> 2.) Bootstrap already does a "make install", and the libc++ headers are
> identical between bootstrap and regular build, so just copy those (package.sh
> always does a bootstrap build)
> 
> Patch set 1 does 1, patch set 2 does 2. 1 seems a bit clearer, 2 is shorter
and
> just as good in practice. I favor 2 a bit.

Since I had forgotten that I've done this already, I came up with a 3rd way when
I accidentally reimplemented this today: Like 1, but use libc++'s
"installheaders" target to only install the libc++ headers:
https://codereview.chromium.org/440143003/

If 2. isn't enough at some point, we can switch to that. Seems better than 1.

Powered by Google App Engine
This is Rietveld 408576698