RESOLVED FIXED 216205
Unified build fix for "redefinition of 'kCMVideoCodecType_VP9" 
https://bugs.webkit.org/show_bug.cgi?id=216205
Summary Unified build fix for "redefinition of 'kCMVideoCodecType_VP9"
Jer Noble
Reported 2020-09-04 17:12:58 PDT
Unified build fix for "redefinition of 'kCMVideoCodecType_VP9"
Attachments
Patch (11.96 KB, patch)
2020-09-04 17:20 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Patch (13.34 KB, patch)
2020-09-08 09:18 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Patch (12.79 KB, patch)
2020-09-08 09:30 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Patch (13.65 KB, patch)
2020-09-08 09:40 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Patch (13.24 KB, patch)
2020-09-08 16:18 PDT, Jer Noble 		no flags
DetailsFormatted DiffDiff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.
Jer Noble
Comment 1 2020-09-04 17:20:10 PDT
Created attachment 408049 [details] Patch
Darin Adler
Comment 2 2020-09-04 21:30:08 PDT
Comment on attachment 408049 [details] Patch Build failures so I won’t r+ yet.
Jer Noble
Comment 3 2020-09-08 09:18:48 PDT
Created attachment 408233 [details] Patch
Jer Noble
Comment 4 2020-09-08 09:30:47 PDT
Created attachment 408234 [details] Patch
Jer Noble
Comment 5 2020-09-08 09:40:04 PDT
Created attachment 408235 [details] Patch
Jer Noble
Comment 6 2020-09-08 16:18:18 PDT
Created attachment 408282 [details] Patch
Peng Liu
Comment 7 2020-09-08 17:03:50 PDT
Comment on attachment 408282 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=408282&action=review > Source/WebCore/ChangeLog:17 > + (): Deleted. Something is missing here?
Jer Noble
Comment 8 2020-09-08 18:32:15 PDT
(In reply to Peng Liu from comment #7) > Comment on attachment 408282 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=408282&action=review > > > Source/WebCore/ChangeLog:17 > > + (): Deleted. > > Something is missing here? Seems like a bug in the ChangeLog generation script.
Darin Adler
Comment 9 2020-09-08 18:44:25 PDT
(In reply to Jer Noble from comment #8) > (In reply to Peng Liu from comment #7) > > > Source/WebCore/ChangeLog:17 > > > + (): Deleted. > > > > Something is missing here? > > Seems like a bug in the ChangeLog generation script. Yes, definitely is. The script has a lot of bugs so I recommend looking over what it generated. It’s there to help you write the change log, not to write it for you. However, since a lot of people are not writing per-function change log I’m starting to wonder what we should do. Not sure that blob of text is doing a lot of good if the person who wrote the change log isn’t reading it.
youenn fablet
Comment 10 2020-09-10 23:58:04 PDT
Comment on attachment 408282 [details] Patch ENABLE_VP9 is Mac Big Sur only, RTC_ENABLE_VP9 is on in all cases. Maybe we should rationalise that.
Radar WebKit Bug Importer
Comment 11 2020-09-11 17:13:14 PDT
<rdar://problem/68740141>
EWS
Comment 12 2020-09-17 02:19:16 PDT
Committed r267185: <https://trac.webkit.org/changeset/267185> All reviewed patches have been landed. Closing bug and clearing flags on attachment 408282 [details].
Note You need to log in before you can comment on or make changes to this bug.