Bug 9539 - Another case error preventing build
Summary: Another case error preventing build
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P1 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-06-22 00:48 PDT by Nicholas Shanks
Modified: 2006-06-22 21:45 PDT (History)
2 users (show)

See Also:


Attachments
Patch committed as r14969 (930 bytes, patch)
2006-06-22 03:35 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff
Document the Xcode 2.3 requirement (694 bytes, patch)
2006-06-22 09:10 PDT, Alexey Proskuryakov
darin: review+
Details | Formatted Diff | Diff
checkRequiredSystemConfig patch (1.00 KB, patch)
2006-06-22 09:46 PDT, Alexey Proskuryakov
ggaren: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Nicholas Shanks 2006-06-22 00:48:41 PDT
cc1plus: warnings being treated as errors
In file included from /Users/nicholas/Projects/Safari/JavaScriptCore/API/JSContextRef.cpp:27:
/Users/nicholas/Projects/Safari/JavaScriptCore/API/APICast.h:31: warning: mismatched case in filenames, wanted "UString.h" but found "ustring.h"
** BUILD FAILED **
Comment 1 David Kilzer (:ddkilzer) 2006-06-22 03:33:52 PDT
Obvious build fix.  Per Maciej, I checked this in without a review.

Committed revision 14969.

Comment 2 David Kilzer (:ddkilzer) 2006-06-22 03:35:02 PDT
Created attachment 8960 [details]
Patch committed as r14969
Comment 3 Nicholas Shanks 2006-06-22 06:52:27 PDT
Still doesn't build, next error pasted below.
Am I the only one who gets these errors?

cc1plus: warnings being treated as errors
/Users/nicholas/Projects/Safari/JavaScriptCore/API/JSObjectRef.cpp:31: warning: mismatched case in filenames, wanted "Identifier.h" but found "identifier.h"
** BUILD FAILED **
Comment 4 David Kilzer (:ddkilzer) 2006-06-22 07:55:29 PDT
(In reply to comment #3)
> Still doesn't build, next error pasted below.
> Am I the only one who gets these errors?

Yes, because you're on a case-sensitive filesystem.  Actually, the Linux porters run into this problem as well.

> cc1plus: warnings being treated as errors
> /Users/nicholas/Projects/Safari/JavaScriptCore/API/JSObjectRef.cpp:31: warning:
> mismatched case in filenames, wanted "Identifier.h" but found "identifier.h"
> ** BUILD FAILED **

Please go through and fix ALL of the errors until it compiles, then create a patch and post it to this bug.

Comment 5 Nicholas Shanks 2006-06-22 08:17:30 PDT
I'm not on a case sensitive file system! I've just got a bog standard HFS+ disk, exactly the same as you. I am also using a bog standard svn checkout, with no changes to the build system. The only thing I can think it might be is GCC. my version is:
powerpc-apple-darwin8-gcc-4.0.0 (GCC) 4.0.0 (Apple Computer, Inc. build 5026)
What are you using?
Comment 6 Alexey Proskuryakov 2006-06-22 09:10:29 PDT
Created attachment 8967 [details]
Document the Xcode 2.3 requirement

Shouldn't we just document that WebKit needs at least Xcode 2.3? With new features being used, such as DWARF debug symbols and better support for path quoting, it is not really compatible with 2.2 anyway.
Comment 7 Darin Adler 2006-06-22 09:30:20 PDT
Comment on attachment 8967 [details]
Document the Xcode 2.3 requirement

Looks good. But we should also change the checkRequiredSystemConfig function in webkitdirs.pm, so you'll get a good error message if you try to build.
Comment 8 Alexey Proskuryakov 2006-06-22 09:46:19 PDT
Created attachment 8968 [details]
checkRequiredSystemConfig patch
Comment 9 Geoffrey Garen 2006-06-22 12:39:00 PDT
Comment on attachment 8968 [details]
checkRequiredSystemConfig patch

r=me
Comment 10 Alexey Proskuryakov 2006-06-22 21:45:59 PDT
Looks like the option to warn about wrong filename case has disappeared from Xcode, that's why others are not seeing the problem.

I've committed a fix for Identifier.h; since it's the only remaining failure that was reported, I'm closing this bug.