Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jhbuild: Backport webrtc patches from GStreamer main branch #46

Merged
merged 2 commits into from
Oct 22, 2024

Conversation

@philn philn requested a review from dpino September 16, 2024 09:27
@nikolaszimmermann
Copy link
Collaborator

Thanks @philn ! In the meanwhile we upgraded to gst 1.26. -- are those patches still relevant?

@philn
Copy link
Member Author

philn commented Sep 28, 2024

You upgraded to 1.24.8 which doesn't ship these patches.

@nikolaszimmermann
Copy link
Collaborator

You upgraded to 1.24.8 which doesn't ship these patches.

Oops - misremembered 😂 thanks for the clarification.

@philn
Copy link
Member Author

philn commented Oct 17, 2024

Error: identifier is not a container: preparing container for next step: creating build container: creating container: creating read-write layer with ID "4244d8e8bb5a3d50479680d92c5091a3b8f9d9138a512a124a9f0de95a44c90e": no space left on device

@dpino
Copy link
Member

dpino commented Oct 17, 2024

Error: identifier is not a container: preparing container for next step: creating build container: creating container: creating read-write layer with ID "4244d8e8bb5a3d50479680d92c5091a3b8f9d9138a512a124a9f0de95a44c90e": no space left on device

I will take a look.

@dpino
Copy link
Member

dpino commented Oct 17, 2024

@philn I freed up space in the host. Now it's failing with a different error:

*** Installing sysprof *** [13/15]
meson install --no-rebuild --quiet
/bin/sh /usr/local/share/jhbuild/triggers/desktop-database.trigger
/bin/sh /usr/local/share/jhbuild/triggers/gtk+.trigger
/bin/sh /usr/local/share/jhbuild/triggers/mime-database.trigger
*** module libspiel not built due to non buildable gstreamer *** [14/15]
*** module webkit-sdk-deps not built due to non buildable libwpe *** [15/15]
*** module webkit-sdk-deps not built due to non buildable wpebackend-fdo *** [15/15]
*** module webkit-sdk-deps not built due to non buildable gstreamer *** [15/15]
*** module webkit-sdk-deps not built due to non buildable libspiel *** [15/15]
*** the following modules were not built *** [15/15]
libwpe wpebackend-fdo gstreamer libspiel webkit-sdk-deps 
Error: building at STEP "RUN git clone https://gitlab.gnome.org/GNOME/jhbuild.git &&     cd jhbuild &&     ./autogen.sh --prefix=/usr/local &&     make &&     make install &&     export JHBUILD_RUN_AS_ROOT=1 WKDEV_IN_IMAGE_BUILD=1 &&     jhbuild --no-interact build &&     rm -r /var/tmp/jhbuild/build": while running runtime: exit status 1

[FATAL] Container image build failed - aborting with exit code 1.
Error: Process completed with exit code 1.

https://github.com/Igalia/webkit-container-sdk/actions/runs/11381867202/job/31672580197?pr=46

@philn
Copy link
Member Author

philn commented Oct 17, 2024

Looks like a new jhbuild bug.

@philn
Copy link
Member Author

philn commented Oct 18, 2024

Related to disk usage, the gst monorepo has a "medias" subproject which is quite big. For our use-case that subproject is useless, can it be ignored during the git clone?

@dpino
Copy link
Member

dpino commented Oct 18, 2024

Looks like a new jhbuild bug.

Indeed, I was hiting the same error on Playwright. I pushed a patch to fix it:

What's happening is that recently a patch to skip the configuration step for cmake modules was merged. The patch introduced a regression, and now all freshly pulled cmake modules skip the configuration step. Without running the configuration step is not possible to build.

The error you're seeing:

Cannot build: libwpe wpebackend-fdo gstreamer libspiel webkit-sdk-deps

happens because libwpe is a cmake module, which failed to build due to this regression. Everything after fails because there's a chain of dependencies: wpebackend-fdo depends on libwpe, gstreamer depends on wpebackend-fdo and libspiel depends on gstreamer.

Also, there's another bug in JHBuild that have been around for a long time and it's quite annoying. Whenever a module fails due to an error, there's not error message printed out in console. That makes it hard to figure out what has happened when a module fails. I also pushed a patch to fix this issue:

cc @TingPing

@philn philn merged commit 3532ae8 into main Oct 22, 2024
2 checks passed
@philn philn deleted the gst-webrtc-patches branch October 22, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants