[WPE] Add some missing jhbuild dependencies on Fedora
Created attachment 337420 [details] Patch
Created attachment 337421 [details] Patch
Created attachment 337422 [details] Patch
Comment on attachment 337422 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=337422&action=review Please make some attempt to update Debian and Arch as well > Tools/wpe/install-dependencies:294 > + libXrandr-devel \ xorg-xrandr for Arch, libxrandr-dev for Debian > Tools/wpe/install-dependencies:299 > + mesa-libEGL-devel \ libegl1-mesa-dev for Debian. I guess in Arch it's just included in mesa > Tools/wpe/install-dependencies:305 > + yasm" yasm in all three distros.
(In reply to Michael Catanzaro from comment #4) > Comment on attachment 337422 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=337422&action=review > > Please make some attempt to update Debian and Arch as well > > > Tools/wpe/install-dependencies:294 > > + libXrandr-devel \ > > xorg-xrandr for Arch, libxrandr-dev for Debian > > > Tools/wpe/install-dependencies:299 > > + mesa-libEGL-devel \ > > libegl1-mesa-dev for Debian. I guess in Arch it's just included in mesa Yes, I can confirm this. In Arch there are no “*-devel” packages; the “normal” package includes also the files needed for development.
Created attachment 337488 [details] Patch
Comment on attachment 337488 [details] Patch Informal r+ here :-) Works fine on Arch Linux with the patch applied (I actually tested), and the Debian/Fedora parts of the changes LGTM as well (I am not very familiar with Fedora, though).
Comment on attachment 337488 [details] Patch JHBuild will always be a best-effort, works-for-me thing... we'll just make it as reliable as we can.
Comment on attachment 337488 [details] Patch Rejecting attachment 337488 [details] from commit-queue. Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-01', 'validate-changelog', '--check-oops', '--non-interactive', 337488, '--port=mac']" exit_code: 1 cwd: /Volumes/Data/EWS/WebKit ChangeLog entry in Tools/ChangeLog contains OOPS!. Full output: http://webkit-queues.webkit.org/results/7256352
(In reply to WebKit Commit Bot from comment #9) > ChangeLog entry in Tools/ChangeLog contains OOPS!
(In reply to Michael Catanzaro from comment #10) > (In reply to WebKit Commit Bot from comment #9) > > ChangeLog entry in Tools/ChangeLog contains OOPS! and what is wrong with that? didn't you gave an r+ with the cq+?
Comment on attachment 337488 [details] Patch Um, nope, because there was no r?. Trying again.
Comment on attachment 337488 [details] Patch Clearing flags on attachment: 337488 Committed r230460: <https://trac.webkit.org/changeset/230460>
All reviewed patches have been landed. Closing bug.