Igalia / meta-webkit

Yocto / OpenEmbedded layer for WebKit based engines and browsers
MIT License
127 stars 71 forks source link

Backport recipes for Thud #272

Closed philn closed 3 years ago

psaavedra commented 3 years ago

Adding a new recipe for a new version has to come with the incorporation of a new CI job to test the build of the layer with that version. Something like what we did in this commit 4a2a32e but, in this case, avoiding the deletion of the job for the 2.28. In other case the CI will not check the build of the 2.32 recipes.

philn commented 3 years ago

I updated the CI, not sure how to test that though. :)

psaavedra commented 3 years ago

The CI should do it for you.

On Wed, May 19, 2021, 16:36 Philippe Normand @.***> wrote:

I updated the CI, not sure how to test that though. :)

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHub https://github.com/Igalia/meta-webkit/pull/272#issuecomment-844167035, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAC4ABSNKN5SMWNPNGWHVYLTOPEHTANCNFSM45EFUBCQ .

psaavedra commented 3 years ago

... here: https://gitlab.com/browsers/meta-webkit/-/jobs/1276717846

ERROR: Nothing PROVIDES 'vulkan-loader' (but /home/bot/yocto-webkit/sources/meta-gstreamer1.0/recipes-multimedia/gstreamer/gstreamer1.0-plugins-bad_1.16.2.bb DEPENDS on or otherwise requires it). Close matches:
195  vulkan-demos
philn commented 3 years ago

Hum, looks like a meta-gstreamer bug, poky/thud has no vulkan-loader recipe it seems.

philn commented 3 years ago

Should be fixed now: https://github.com/OSSystems/meta-gstreamer1.0/pull/58

philn commented 3 years ago

CI fails again but the error is not visible... Job's log exceeded limit of 4194304 bytes.

philn commented 3 years ago

It passed now? What happened? :)

psaavedra commented 3 years ago

It passed now? What happened? :)

A wizard. No jokes. It seems the machine was overloaded because a parallel build. I just did a retry.

philn commented 3 years ago

DEFAULT_PREFERENCE now set to "-1".

philn commented 3 years ago

CI broke, wpewebkit couldn't find recent enough libwpe. I've pushed again, attempting to fix this.