WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
142252
Remove WK_AVAILABLE and related macros when we cannot determine iOS/Mac version
https://bugs.webkit.org/show_bug.cgi?id=142252
Summary
Remove WK_AVAILABLE and related macros when we cannot determine iOS/Mac version
Joseph Pecoraro
Reported
2015-03-03 18:11:23 PST
* SUMMARY Better build fix when PLATFORM_NAME is unknown/missing. In such cases, we cannot determine the iOS/Mac version numbers, so just remove WK_AVAILABLE / WK_DEPRECATED macros that may contain WK_MAC_TBA/WK_IOS_TBA in the postprocess build step so we don't output a broken NS macro.
Attachments
[PATCH] Proposed Fix
(3.22 KB, patch)
2015-03-03 18:15 PST
,
Joseph Pecoraro
ddkilzer
: review+
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Joseph Pecoraro
Comment 1
2015-03-03 18:15:06 PST
Created
attachment 247823
[details]
[PATCH] Proposed Fix
David Kilzer (:ddkilzer)
Comment 2
2015-03-04 09:43:55 PST
Comment on
attachment 247823
[details]
[PATCH] Proposed Fix View in context:
https://bugs.webkit.org/attachment.cgi?id=247823&action=review
r=me
> Source/WebKit2/mac/postprocess-framework-headers.sh:50 > + if [[ -n $OSX_VERSION ]] && [[ -n $IOS_VERSION ]]; then
Should the variables be double-quoted in case they're not set? if [[ -n "$OSX_VERSION" ]] && [[ -n "$IOS_VERSION" ]]; then
Joseph Pecoraro
Comment 3
2015-03-04 10:49:38 PST
http://trac.webkit.org/changeset/181004
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug