Bug 218434 - Update WebRTC libyuv to M87
Summary: Update WebRTC libyuv to M87
Status: REOPENED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebRTC (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: youenn fablet
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2020-11-02 03:43 PST by youenn fablet
Modified: 2022-09-13 05:28 PDT (History)
7 users (show)

See Also:

Attachments
Patch (15.68 KB, patch)
2020-11-02 03:48 PST, youenn fablet 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description youenn fablet 2020-11-02 03:43:41 PST 
Update WebRTC libyuv to M87
Comment 1 youenn fablet 2020-11-02 03:48:19 PST 
Created attachment 412900 [details]
Patch
Comment 2 Eric Carlson 2020-11-02 14:06:47 PST 
Comment on attachment 412900 [details]
Patch

rs=me
Comment 3 EWS 2020-11-03 03:23:41 PST 
Committed r269291: <https://trac.webkit.org/changeset/269291>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 412900 [details].
Comment 4 Radar WebKit Bug Importer 2020-11-03 03:24:21 PST 
<rdar://problem/70988211>
Comment 5 Carlos Alberto Lopez Perez 2020-11-03 04:05:20 PST 
(In reply to EWS from comment #3)
> Committed r269291: <https://trac.webkit.org/changeset/269291>
> 
> All reviewed patches have been landed. Closing bug and clearing flags on
> attachment 412900 [details].

It seems the commit-queue bot did something funny here.

As far as I can see the patch landed on r269291 misses all the changes to the ThirdParty directory :?
Comment 6 youenn fablet 2020-11-06 04:43:11 PST 
Reopening to investigate
Comment 7 Carlos Alberto Lopez Perez 2020-11-06 05:48:59 PST 
Looking at the diff a bit carefully i realize that there is no actually diff. The content as such of the files are the same.

The only difference seems to be the line-encoding endings. This 4 files on repo currently have windows-type line endings (\r\n) and the ones on the patch have unix-type line endings (\n)

So perhaps this is related with the SVN server configuration or something like that.
Comment 8 Olivier Blin 2022-09-13 05:28:31 PDT 
libyuv is now at M92