Bug 48181 - Add missing include of MacErrors.h
Summary: Add missing include of MacErrors.h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac OS X 10.5
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on: 32993
Blocks:
  Show dependency tree / graph
 
Reported: 2010-10-23 05:19 PDT by David Kilzer (:ddkilzer)
Modified: 2010-10-23 13:53 PDT (History)
4 users (show)

See Also:

Attachments
Patch v1 (1.24 KB, patch)
2010-10-23 05:23 PDT, David Kilzer (:ddkilzer) 		no flags Details | Formatted Diff | Diff
Patch v2 (1.31 KB, patch)
2010-10-23 10:50 PDT, David Kilzer (:ddkilzer) 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2010-10-23 05:19:42 PDT 
Add missing include of MacErrors.h
Comment 1 David Kilzer (:ddkilzer) 2010-10-23 05:23:17 PDT 
Created attachment 71635 [details]
Patch v1
Comment 2 WebKit Commit Bot 2010-10-23 08:28:00 PDT 
Comment on attachment 71635 [details]
Patch v1

Rejecting patch 71635 from commit-queue.

Failed to run "['./WebKitTools/Scripts/webkit-patch', '--status-host=queues.webkit.org', 'build', '--no-clean', '--no-update', '--build-style=both']" exit_code: 2
Last 500 characters of output:
N_MINOR 0320
    setenv YACC /Developer/usr/bin/yacc
    /bin/sh -c /Projects/CommitQueue/WebKitBuild/WebCore.build/Debug/WebCore.build/Script-5DF50887116F3077005202AB.sh

** BUILD FAILED **


The following build commands failed:
WebCore:
	CompileC /Projects/CommitQueue/WebKitBuild/WebCore.build/Debug/WebCore.build/Objects-normal/x86_64/FormDataStreamMac.o /Projects/CommitQueue/WebCore/platform/network/mac/FormDataStreamMac.mm normal x86_64 objective-c++ com.apple.compilers.gcc.4_2
(1 failure)


Full output: http://queues.webkit.org/results/4675043
Comment 3 David Kilzer (:ddkilzer) 2010-10-23 08:28:37 PDT 
Comment on attachment 71635 [details]
Patch v1

Clearing flags.  Apparently fnfErr is defined elsewhere.
Comment 4 David Kilzer (:ddkilzer) 2010-10-23 10:50:43 PDT 
Created attachment 71647 [details]
Patch v2
Comment 5 Alexey Proskuryakov 2010-10-23 13:11:55 PDT 
Comment on attachment 71647 [details]
Patch v2

It took me some time to understand why Mac path also needed to be modified. Currently, Mac gets CoreServices through WebCorePrefix.h, and we want the code to build without it.
Comment 6 David Kilzer (:ddkilzer) 2010-10-23 13:33:03 PDT 
(In reply to comment #5)
> (From update of attachment 71647 [details])
> It took me some time to understand why Mac path also needed to be modified. Currently, Mac gets CoreServices through WebCorePrefix.h, and we want the code to build without it.

Thanks. Yes, MacErrors.h is part of CarbonCore.framework which is a sub-framework of CoreServices.framework.
Comment 7 WebKit Commit Bot 2010-10-23 13:53:33 PDT 
Comment on attachment 71647 [details]
Patch v2

Clearing flags on attachment: 71647

Committed r70399: <http://trac.webkit.org/changeset/70399>
Comment 8 WebKit Commit Bot 2010-10-23 13:53:38 PDT 
All reviewed patches have been landed.  Closing bug.