Changes to support pipewire broke moz.build code generation
Categories
(Core :: WebRTC: Audio/Video, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr78 | --- | unaffected |
firefox83 | --- | unaffected |
firefox84 | --- | wontfix |
firefox85 | --- | fixed |
People
(Reporter: dminor, Assigned: dminor)
References
(Blocks 1 open bug, Regression)
Details
(Keywords: regression)
Attachments
(7 files)
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details | |
(deleted),
text/x-phabricator-request
|
Details |
I missed this during review, but Martin was not aware that the moz.build files are code generated from BUILD.gn files and edited them directly. It is not possible to generate correct moz.build files from the BUILD.gn files for desktop_capture at the moment. This has broken the libwebrtc build for FreeBSD (Bug 1676134) and caused problems while doing the libwebrtc update (Bug 1654112).
Updated•4 years ago
|
Assignee | ||
Updated•4 years ago
|
Comment 1•4 years ago
|
||
Set release status flags based on info from the regressing bug 1672989
Assignee | ||
Comment 2•4 years ago
|
||
This adds write_mozbuild_variables to the GnProjectData that can be used to
configure code generation of moz.build files. This is done to support adding
CONFIG["TK_CFLAGS"] to INCLUDES for gtk builds, which is necessary to support
pipewire.
There is likely a more elegant way of handling this, but since we plan to
remove code generation of moz.build files in the near future anyway, I'm not
sure it is worth investing more time than a minimal amount of time in this.
Assignee | ||
Comment 3•4 years ago
|
||
This sets the write_mozbuild_vars for libwebrtc to include TK_CFLAGS for
desktop_capture.
Depends on D97427
Assignee | ||
Comment 4•4 years ago
|
||
This enables rtc_use_pipewire for libwebrtc as a whole. We should fallback
to X11 screen capture on non-Wayland systems.
Depends on D97428
Assignee | ||
Comment 5•4 years ago
|
||
Depends on D97429
Assignee | ||
Comment 6•4 years ago
|
||
Depends on D97430
Assignee | ||
Comment 7•4 years ago
|
||
Depends on D97431
Assignee | ||
Comment 8•4 years ago
|
||
Depends on D97432
Assignee | ||
Updated•4 years ago
|
Updated•4 years ago
|
Assignee | ||
Comment 9•4 years ago
|
||
Comment 10•4 years ago
|
||
Pushed by dminor@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0ab2dc45e3f1 Add a way of setting TK_CFLAGS in a gn generated moz.build file r=firefox-build-system-reviewers,mhentges,ng https://hg.mozilla.org/integration/autoland/rev/1da93acd5a21 Set write_mozbuild_vars for libwebrtc; r=ng https://hg.mozilla.org/integration/autoland/rev/96935994bcca Set rtc_use_pipewire flag in webrtc.gni; r=ng https://hg.mozilla.org/integration/autoland/rev/6fb2ffeb035b Fixup errors in desktop_capture BUILD.gn; r=ng https://hg.mozilla.org/integration/autoland/rev/f6baf18918aa Build libpipewire as a dependency of libwebrtc; r=ng https://hg.mozilla.org/integration/autoland/rev/085eb2aa64c8 Update gn generated json files for tier-1 platforms; r=ng https://hg.mozilla.org/integration/autoland/rev/2c1390ba3315 Regenerate moz.build files; r=ng
Comment 11•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/0ab2dc45e3f1
https://hg.mozilla.org/mozilla-central/rev/1da93acd5a21
https://hg.mozilla.org/mozilla-central/rev/96935994bcca
https://hg.mozilla.org/mozilla-central/rev/6fb2ffeb035b
https://hg.mozilla.org/mozilla-central/rev/f6baf18918aa
https://hg.mozilla.org/mozilla-central/rev/085eb2aa64c8
https://hg.mozilla.org/mozilla-central/rev/2c1390ba3315
Comment 12•4 years ago
|
||
is this a candidate for backport to 84, if not too late ?
Assignee | ||
Comment 13•4 years ago
|
||
Unfortunately, I think it is too late for a backport for 84. You might consider making a local patch for the bad moz.build file for 84, which you could then drop for 85. Sorry about this!
Updated•4 years ago
|
Description
•