WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
224632
Use kIOMainPortDefault where available
https://bugs.webkit.org/show_bug.cgi?id=224632
Summary
Use kIOMainPortDefault where available
Alex Christensen
Reported
2021-04-15 16:53:26 PDT
Replace deprecation pragmas with new port name
Attachments
Patch
(4.42 KB, patch)
2021-04-15 16:55 PDT
,
Alex Christensen
no flags
Details
Formatted Diff
Diff
Patch
(4.55 KB, patch)
2021-04-15 17:58 PDT
,
Alex Christensen
no flags
Details
Formatted Diff
Diff
Patch
(4.54 KB, patch)
2021-04-15 18:00 PDT
,
Alex Christensen
no flags
Details
Formatted Diff
Diff
Show Obsolete
(2)
View All
Add attachment
proposed patch, testcase, etc.
Alex Christensen
Comment 1
2021-04-15 16:55:45 PDT
Created
attachment 426157
[details]
Patch
EWS Watchlist
Comment 2
2021-04-15 16:57:20 PDT
Note that there are important steps to take when updating ANGLE. See
https://trac.webkit.org/wiki/UpdatingANGLE
Alex Christensen
Comment 3
2021-04-15 17:58:38 PDT
Created
attachment 426165
[details]
Patch
Alex Christensen
Comment 4
2021-04-15 18:00:03 PDT
Created
attachment 426166
[details]
Patch
Alexey Proskuryakov
Comment 5
2021-04-15 18:04:06 PDT
Comment on
attachment 426166
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=426166&action=review
I would probably centralize this in one place if I were doing this, but seems OK either way.
> Source/ThirdParty/ANGLE/src/gpu_info_util/SystemInfo_macos.mm:41 > #pragma clang diagnostic ignored "-Wdeprecated-declarations"
You can consider adding comments saying that the pragmas can be removed once
rdar://76706593
is fixed, but it's not terribly important.
EWS
Comment 6
2021-04-15 18:45:14 PDT
Committed
r276104
(
236604@main
): <
https://commits.webkit.org/236604@main
> All reviewed patches have been landed. Closing bug and clearing flags on
attachment 426166
[details]
.
Kimmo Kinnunen
Comment 7
2021-04-16 04:51:21 PDT
Not sure if I'm doing something wrong, but I'm seeing the inverse issue now with an older SDK. CompileC SystemInfo_macos.o /Users/kkinnunen/WebKit/OpenSource/Source/ThirdParty/ANGLE/src/gpu_info_util/SystemInfo_macos.mm:38:34: error: use of undeclared identifier 'kIOMainPortDefault' const mach_port_t mainPort = kIOMainPortDefault; ^ /Users/kkinnunen/WebKit/OpenSource/Source/ThirdParty/ANGLE/src/gpu_info_util/SystemInfo_macos.mm:109:38: error: use of undeclared identifier 'kIOMainPortDefault' const mach_port_t mainPort = kIOMainPortDefault; ^ /Users/kkinnunen/WebKit/OpenSource/Source/ThirdParty/ANGLE/src/gpu_info_util/SystemInfo_macos.mm:176:34: error: use of undeclared identifier 'kIOMainPortDefault' const mach_port_t mainPort = kIOMainPortDefault; ^ /Users/kkinnunen/WebKit/OpenSource/Source/ThirdParty/ANGLE/src/gpu_info_util/SystemInfo_macos.mm:285:38: error: use of undeclared identifier 'kIOMainPortDefault' const mach_port_t mainPort = kIOMainPortDefault; ^
Alex Christensen
Comment 8
2021-04-16 13:20:18 PDT
(In reply to Kimmo Kinnunen from
comment #7
)
> Not sure if I'm doing something wrong, but I'm seeing the inverse issue now > with an older SDK.
Time to update your SDK.
Radar WebKit Bug Importer
Comment 9
2021-04-24 15:10:42 PDT
<
rdar://problem/77110140
>
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