WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
198675
Open up xcfilelist processing to more platforms
https://bugs.webkit.org/show_bug.cgi?id=198675
Summary
Open up xcfilelist processing to more platforms
Keith Rollin
Reported
2019-06-07 13:28:43 PDT
Now that it's been tested, add AppleTV{OS,Simulator} to the set of platforms on which to perform xcfilelist generation/updating.
Attachments
Patch
(2.58 KB, patch)
2019-06-07 13:30 PDT
,
Keith Rollin
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2019-06-07 13:28:58 PDT
<
rdar://problem/51533238
>
Keith Rollin
Comment 2
2019-06-07 13:30:14 PDT
Created
attachment 371605
[details]
Patch
Jonathan Bedard
Comment 3
2019-06-07 13:56:32 PDT
Comment on
attachment 371605
[details]
Patch View in context:
https://bugs.webkit.org/attachment.cgi?id=371605&action=review
> Tools/ChangeLog:3 > + Open up xcfilelist processing to more platforms
Is this a multi-part change? Or is tvOS the end of it?
Keith Rollin
Comment 4
2019-06-07 14:04:47 PDT
I'm not sure what you're getting at. There will be a similar change in another, internal repository. And we'll be adding support for any new platforms that Apple creates in the future. If you're asking about iPadOS, there's no need for any changes along those lines.
Keith Rollin
Comment 5
2019-06-10 15:07:45 PDT
Jonathan, any concerns with this patch?
Keith Rollin
Comment 6
2019-06-11 09:44:58 PDT
Jonathan, could you please look at this?
Jonathan Bedard
Comment 7
2019-06-11 11:16:23 PDT
(In reply to Keith Rollin from
comment #6
)
> Jonathan, could you please look at this?
No, looks good. Sorry for the delay, had slipped my task queue.
WebKit Commit Bot
Comment 8
2019-06-11 12:40:30 PDT
Comment on
attachment 371605
[details]
Patch Clearing flags on attachment: 371605 Committed
r246329
: <
https://trac.webkit.org/changeset/246329
>
WebKit Commit Bot
Comment 9
2019-06-11 12:40:32 PDT
All reviewed patches have been landed. Closing bug.
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