Bug 178748 - Allow OjbC Weak References when building TestAPI
Summary: Allow OjbC Weak References when building TestAPI
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: Other
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Michael Saboff
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-10-24 14:02 PDT by Michael Saboff
Modified: 2017-10-24 15:59 PDT (History)
8 users (show)

See Also:


Attachments
Patch (1.87 KB, patch)
2017-10-24 14:12 PDT, Michael Saboff
no flags Details | Formatted Diff | Diff
Updated Patch (2.53 KB, patch)
2017-10-24 15:21 PDT, Michael Saboff
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Saboff 2017-10-24 14:02:34 PDT
TestAPI is built using manual reference counting.  Some version of the compiler will complain when building with default build flags.
Comment 1 Radar WebKit Bug Importer 2017-10-24 14:03:44 PDT
<rdar://problem/35158752>
Comment 2 Michael Saboff 2017-10-24 14:12:01 PDT
Created attachment 324718 [details]
Patch
Comment 3 WebKit Commit Bot 2017-10-24 14:31:05 PDT
Comment on attachment 324718 [details]
Patch

Clearing flags on attachment: 324718

Committed r223923: <https://trac.webkit.org/changeset/223923>
Comment 4 WebKit Commit Bot 2017-10-24 14:31:06 PDT
All reviewed patches have been landed.  Closing bug.
Comment 5 mitz 2017-10-24 14:55:40 PDT
Comment on attachment 324718 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=324718&action=review

> Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj:10386
> +				CLANG_ENABLE_OBJC_WEAK = YES;

We shouldn’t specify build settings in the project file. This can probably go in ToolExecutable.xcconfig or if, for some reason, it can’t apply to all tools, in a new .xcconfig file.
Comment 6 Michael Saboff 2017-10-24 15:05:20 PDT
(In reply to mitz from comment #5)
> Comment on attachment 324718 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=324718&action=review
> 
> > Source/JavaScriptCore/JavaScriptCore.xcodeproj/project.pbxproj:10386
> > +				CLANG_ENABLE_OBJC_WEAK = YES;
> 
> We shouldn’t specify build settings in the project file. This can probably
> go in ToolExecutable.xcconfig or if, for some reason, it can’t apply to all
> tools, in a new .xcconfig file.

Reopening this bug to move the build setting.
Comment 7 Michael Saboff 2017-10-24 15:21:59 PDT
Created attachment 324737 [details]
Updated Patch
Comment 8 WebKit Commit Bot 2017-10-24 15:59:54 PDT
Comment on attachment 324737 [details]
Updated Patch

Clearing flags on attachment: 324737

Committed r223937: <https://trac.webkit.org/changeset/223937>
Comment 9 WebKit Commit Bot 2017-10-24 15:59:56 PDT
All reviewed patches have been landed.  Closing bug.