108089 2013-01-28 09:37:22 -0800 "clang: warning: not using the clang compiler for C++ inputs" due to hard-coding of /usr/bin/clang in WebCore.gyp 2013-01-28 21:23:45 -0800 1 1 1 Unclassified WebKit New Bugs 528+ (Nightly build) Mac OS X 10.6 RESOLVED FIXED P2 Normal --- 1 mike tony abarth laszlo.gombos thakis tony webkit.review.bot oldest_to_newest 817812 0 mike 2013-01-28 09:37:22 -0800 http://trac.webkit.org/changeset/140925 for bug 107984 changed Source/WebCore/WebCore.gyp/WebCore.gyp so that it now hardcodes "/usr/bin/clang" as an argument for --preprocessor switch that's fed to the part of the build that preprocesses the bindings idl files. In my OSX 10.6.8 environment, that change causes the build to generate hundreds of "clang: warning: not using the clang compiler for C++ inputs" messages. Why not just have "clang" there instead of "/usr/bin/clang"? That way you could just have this part of the build using the clang that the rest of the build uses; e.g., if I have my environment set up for building Chromium with clang in $CHROME/src/third_party/llvm-build/Release+Asserts/bin the idl preprocessing would use that clang too. 817821 1 thakis 2013-01-28 09:49:07 -0800 Install Xcode 4.2 and install the command line tools. Then you should be all set. 817830 2 mike 2013-01-28 09:58:32 -0800 (In reply to comment #1) > Install Xcode 4.2 and install the command line tools. Then you should be all set. If Xcode 4.2 is the latest, it's not possible to install it in a OSX 10.6.8. I tried it before and got a message saying as much. Anyway, I'm able to build Chromium fine in my 10.6 without needing Xcode 4.2, and I think it's reasonable to expect that dependencies on Xcode 4.2 shouldn't be introduced unless they're really necessary. 817855 3 tony 2013-01-28 10:30:23 -0800 Nico: Maybe I should have OSX use /usr/bin/gcc? That would actually be more like the Apple Mac build. When /usr/bin/gcc goes away, we'll have to update both builds. 817971 4 thakis 2013-01-28 12:17:22 -0800 Fine with me, but I'm not sure how much longer we're going to support buildling with Xcode 3.2.6 anyhow. And I believe Xcode 4.2 includes a capable clang and is available for 10.6 (that's what I have installed on my 10.6 box). 817985 5 185034 tony 2013-01-28 12:32:26 -0800 Created attachment 185034 Patch 818000 6 thakis 2013-01-28 12:38:59 -0800 lgtm / meh. 818082 7 185034 webkit.review.bot 2013-01-28 14:07:32 -0800 Comment on attachment 185034 Patch Clearing flags on attachment: 185034 Committed r141007: <http://trac.webkit.org/changeset/141007> 818083 8 webkit.review.bot 2013-01-28 14:07:36 -0800 All reviewed patches have been landed. Closing bug. 818471 9 mike 2013-01-28 21:23:45 -0800 (In reply to comment #4) > Fine with me, but I'm not sure how much longer we're going to support buildling with Xcode 3.2.6 anyhow. And I believe Xcode 4.2 includes a capable clang and is available for 10.6 (that's what I have installed on my 10.6 box). Not to beat this into the ground, but I went to https://developer.apple.com/downloads/index.action?name=Xcode to try to download it but the only Xcode 4.2 download there is specifically labeled "Xcode 4.2 for Lion". (The Xcode 4.1 and 4.3 downloads are labeled that way too.) So if you're using it with 10.6, I assume that despite how it's labeled it's actually possible to install it and run it under 10.6, so I'll give it a try. Also, I get the main point and will make some time soon to get my OS updated to Mountain Lion. Anyway, thanks for the responses and fix on this bug. 185034 2013-01-28 12:32:26 -0800 2013-01-28 14:07:32 -0800 Patch bug-108089-20130128122913.patch text/plain 2101 tony U3VidmVyc2lvbiBSZXZpc2lvbjogMTQwOTg4CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggOTk2MTI5OGQwODM3MzFj OTlhMDIzMDYyNWMzZGRlZjQwYTRiOWFiMC4uYmIxODliY2IyNjZjMmQzYTdiYjAwZmYyMzQ1M2Ix ZTIzZmJhMTIzYyAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEzLTAxLTI4ICBUb255 IENoYW5nICA8dG9ueUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgImNsYW5nOiB3YXJuaW5nOiBu b3QgdXNpbmcgdGhlIGNsYW5nIGNvbXBpbGVyIGZvciBDKysgaW5wdXRzIiBkdWUgdG8gaGFyZC1j b2Rpbmcgb2YgL3Vzci9iaW4vY2xhbmcgaW4gV2ViQ29yZS5neXAKKyAgICAgICAgaHR0cHM6Ly9i dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEwODA4OQorCisgICAgICAgIFJldmlld2Vk IGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFVzZSBnY2Mgc2luY2Ugb2xkZXIgdmVyc2lv bnMgb2YgY2xhbmcgKHdpdGggWGNvZGUgMy4yKSB3YXJuIHRoYXQgdGhleSBhcmUgZ29pbmcgdG8g dXNlIGdjYyBhbnl3YXkuCisgICAgICAgIFRoZXNlIHdhcm5pbmdzIGFyZSBzaG93aW5nIHVwIG9u IHRoZSBtYWluIENocm9taXVtIHdhdGVyZmFsbCB0b286CisgICAgICAgIGh0dHA6Ly9idWlsZC5j aHJvbWl1bS5vcmcvcC9jaHJvbWl1bS9idWlsZGVycy9NYWMvYnVpbGRzLzE5MTEzL3N0ZXBzL2Nv bXBpbGUvbG9ncy9zdGRpbworCisgICAgICAgIE5vIG5ldyB0ZXN0cywgdGhpcyBpcyBhIGJ1aWxk IGNoYW5nZS4KKworICAgICAgICAqIFdlYkNvcmUuZ3lwL1dlYkNvcmUuZ3lwOgorCiAyMDEzLTAx LTI4ICBUb255IEdlbnRpbGNvcmUgIDx0b255Z0BjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgRG9u J3QgdXNlIHRoZSB0aHJlYWRlZCBIVE1MIHBhcnNlciBmb3IgamF2YXNjcmlwdDogVVJMcwpkaWZm IC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvV2ViQ29yZS5neXAvV2ViQ29yZS5neXAgYi9Tb3VyY2Uv V2ViQ29yZS9XZWJDb3JlLmd5cC9XZWJDb3JlLmd5cAppbmRleCA4YmQwYTMzOTA1NzkzMTNlNTEx MmRiYzQ1N2EzMTgyYmJjZjEzNTIwLi5mMWNiZTNmMGVjMTY2ZTE3MTZiOTk1Y2E1YzI5N2NlN2Uy MjQyNTYwIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9XZWJDb3JlLmd5cC9XZWJDb3JlLmd5 cAorKysgYi9Tb3VyY2UvV2ViQ29yZS9XZWJDb3JlLmd5cC9XZWJDb3JlLmd5cApAQCAtMjQwLDE0 ICsyNDAsOSBAQAogICAgICAgICAnZ3BlcmZfZXhlJzogJ2dwZXJmJywKICAgICAgICAgJ2Jpc29u X2V4ZSc6ICdiaXNvbicsCiAKLSAgICAgICAgJ2NvbmRpdGlvbnMnOiBbCi0gICAgICAgICAgIyBX ZSBzcGVjaWZ5IGEgcHJlcHJvY2VzcyBzbyBpdCBoYXBwZW5zIGxvY2FsbHkgYW5kIHdvbid0IGdl dCBkaXN0cmlidXRlZCB0byBnb21hLgotICAgICAgICAgIFsnT1M9PSJtYWMiIG9yIE9TPT0iaW9z IicsIHsKLSAgICAgICAgICAgICdwcmVwcm9jZXNzb3InOiAnLS1wcmVwcm9jZXNzb3IgIi91c3Iv YmluL2NsYW5nIC1FIC1QIC14IGMrKyInCi0gICAgICAgICAgfSx7Ci0gICAgICAgICAgICAncHJl cHJvY2Vzc29yJzogJy0tcHJlcHJvY2Vzc29yICIvdXNyL2Jpbi9nY2MgLUUgLVAgLXggYysrIicK LSAgICAgICAgICB9XQotICAgICAgICBdLAorICAgICAgICAjIFdlIHNwZWNpZnkgYSBwcmVwcm9j ZXNzIHNvIGl0IGhhcHBlbnMgbG9jYWxseSBhbmQgd29uJ3QgZ2V0IGRpc3RyaWJ1dGVkIHRvIGdv bWEuCisgICAgICAgICMgRklYTUU6IC91c3IvYmluL2djYyB3b24ndCBleGlzdCBvbiBPU1ggZm9y ZXZlci4gV2Ugd2FudCB0byB1c2UgL3Vzci9iaW4vY2xhbmcgb25jZSB3ZSByZXF1aXJlIFhjb2Rl IDQueC4KKyAgICAgICAgJ3ByZXByb2Nlc3Nvcic6ICctLXByZXByb2Nlc3NvciAiL3Vzci9iaW4v Z2NjIC1FIC1QIC14IGMrKyInCiAgICAgICB9XSwKICAgICAgIFsndXNlX3gxMT09MSBvciBPUz09 ImFuZHJvaWQiJywgewogICAgICAgICAnd2ViY29yZV9pbmNsdWRlX2RpcnMnOiBbCg==