Bug 179169 - [WPE][JHBuild] Update WPEBackend and WPEBackend-mesa
Summary: [WPE][JHBuild] Update WPEBackend and WPEBackend-mesa
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WPE WebKit (show other bugs)
Version: WebKit Local Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Carlos Alberto Lopez Perez
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-11-02 06:46 PDT by Carlos Alberto Lopez Perez
Modified: 2017-11-15 12:07 PST (History)
7 users (show)

See Also:


Attachments
Patch (1.50 KB, patch)
2017-11-02 07:00 PDT, Carlos Alberto Lopez Perez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Carlos Alberto Lopez Perez 2017-11-02 06:46:03 PDT
Update WPEBackend and WPEBackend-mesa to last master.


WPEBackend (master) $ git log --oneline 5e2a29d2ccc637f7122ba72c1d62ef669b42f05c..master
3013faa Properly find EGL and pass any flag defined on the EGL pkgconfig
0272534 Merge pull request #5 from JPEWdev/license
3cd82aa Add license file
baa960b Merge pull request #2 from blino/backend-load-error
afe28ad wpe_view_backend_create: fail early if backend can not be loaded
4aff0ea Merge pull request #1 from vertexodessa/cmake28
98e3006 CMake version 2.8 is enough for the build

WPEBackend-mesa (master) $ git log --oneline 0286aed84a09120de16c3ab08bdf83c91d4a1754..master
ce2a0ad Merge pull request #7 from clopez/xdg-shell-protocol-match
1a91ded Merge pull request #5 from clopez/FindWaylandEGL
26a019a Only try to create the xdg-surface once.
c7feee8 [WPEBackend-mesa] Correctly find libxkbcommon
07e5180 Find EGL and set definitions.
a0519c6 Add missing FindWaylandEGL cmake file.
Comment 1 Carlos Alberto Lopez Perez 2017-11-02 07:00:53 PDT
Created attachment 325714 [details]
Patch
Comment 2 Michael Catanzaro 2017-11-02 07:40:50 PDT
Comment on attachment 325714 [details]
Patch

I thought about switching these to use git, but I think the value of being able to blame issue to particular WebKit commits outweighs the convenience of not having to update the hashes here, so OK.

Thanks very much for working on this.
Comment 3 Carlos Alberto Lopez Perez 2017-11-02 07:46:25 PDT
(In reply to Michael Catanzaro from comment #2)
> Comment on attachment 325714 [details]
> Patch
> 
> I thought about switching these to use git, but I think the value of being
> able to blame issue to particular WebKit commits outweighs the convenience
> of not having to update the hashes here, so OK.
> 

I guess you meant here "switching these to _not_ use git" ? (Otherwise I don't understand what you mean)

In any case, either when using tarballs or when using git something has always to be updated on the moduleset (either be the hashsum or the git hashid).

I don't think we should be using vague references like pointing them always to "master", as that will risk introducing random breakage on the webkit build.
I think is a good practice to mandate that each update to this modules should be tested by at least one webkit developer. The best way of doing that is like now, manually updating the git hashids in a webkit commit that needs to be reviewed.

> Thanks very much for working on this.


Thanks you for the review :)
Comment 4 Carlos Alberto Lopez Perez 2017-11-02 07:48:12 PDT
Comment on attachment 325714 [details]
Patch

Clearing flags on attachment: 325714

Committed r224331: <https://trac.webkit.org/changeset/224331>
Comment 5 Michael Catanzaro 2017-11-02 07:58:23 PDT
(In reply to Carlos Alberto Lopez Perez from comment #3)
> I guess you meant here "switching these to _not_ use git" ? (Otherwise I
> don't understand what you mean)

No, I just read the commit too quickly. Anyway, I agree with your reasoning!
Comment 6 Michael Catanzaro 2017-11-02 08:54:19 PDT
Forgot to close it?