Source code

Revision control

Copy as Markdown

Other Tools

From: Jan Grulich <jgrulich@redhat.com>
Date: Mon, 24 Mar 2025 10:09:00 +0000
Subject: Bug 1954903 - WebRTC backport: PipeWire camera: drop redundant
<pipewire/core.h> include r=pehrsons,webrtc-reviewers
We already include main <pipewire/pipewire.h> header, which includes all
the PipeWire headers, making <pipewire/core.h> to be included twice.
This breaks Firefox builds, where using system libraries we have to list
all system headers, making Firefox to unnecessarily list "core.h" header
in the list, while "pipewire.h" should be enough.
This is a simple backport of an WebRTC upstream change.
Upstream commit: 537158885bc60ffe64c2c0c8a7c89d74cb7c105b
---
modules/video_capture/linux/pipewire_session.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/modules/video_capture/linux/pipewire_session.h b/modules/video_capture/linux/pipewire_session.h
index 1f3a00614f..aed3fbb1bf 100644
--- a/modules/video_capture/linux/pipewire_session.h
+++ b/modules/video_capture/linux/pipewire_session.h
@@ -11,7 +11,6 @@
#ifndef MODULES_VIDEO_CAPTURE_LINUX_PIPEWIRE_SESSION_H_
#define MODULES_VIDEO_CAPTURE_LINUX_PIPEWIRE_SESSION_H_
-#include <pipewire/core.h>
#include <pipewire/pipewire.h>
#include <deque>