Skip to content

Remove remnants of webrtc_dir caching#7

Merged
cole-miller merged 2 commits intomainfrom
missing-headers-v2
Jul 30, 2025
Merged

Remove remnants of webrtc_dir caching#7
cole-miller merged 2 commits intomainfrom
missing-headers-v2

Conversation

@cole-miller
Copy link
Copy Markdown
Member

Ensure that we download the webrtc release anew each time download_webrtc is called (from either the livekit-ffi build script or the webrtc-sys build script). This is another stab at fixing the build errors we've been seeing in Zed CI, or at least making them easier to understand.

Also reverts #6 since it seems the integrity of the downloaded zip was a red herring.

@cole-miller cole-miller merged commit a2954c1 into main Jul 30, 2025
4 checks passed
cole-miller added a commit that referenced this pull request Jul 30, 2025
cole-miller added a commit to zed-industries/zed that referenced this pull request Jul 31, 2025
kubkon pushed a commit that referenced this pull request Feb 25, 2026
* feat: expose DesktopCapturer

Implements screen sharing by exposing libwebrtc's
DesktopCapturer.

A few platform specific notes:
  - macos:
    * It is using screen capture kit and the system picker by
      default. If the system picker is disabled then get_sources
      returns an empty list when trying to capture a display. The
      display native id needs to be acquired using different means
      from the client.
  - linux:
    * With pipewire the only way to select window or display is via
      the system picker.

* use pkg-config to find Linux libraries (#2)

* Support x11 dependencies

* Modify example to work on x11

* Address comments

* Properly guard sck options

* Address review comments

In particular:
   - Adds support for the generic capturer.
   - Exposes the desktop_capturer module only on macos, linux and
     windows.
   - Updates the example

* Update license

* Set callback on start

* Make the callback FnMut

* cpp formatting

* Allow disabling system picker in example

* Address review comments

* Remove reference to temporary

* Run screen-sharing example only on supported platforms

* cargo fmt

* Update build runner with DesktopCapturer deps on linux

* Update runner

* Update tests runner

* fix Android build (#7)

* screensharing example: remove rustls-native feature from livekit

This was breaking the Android builds.

* improve error handling for File::open

* webrtc-sys-build: skip parsing desktop_capture.ninja on Android

* webrtc-sys-build: bump prebuilt library to webrtc-0001d84-2

* Enable TLS features for the screenshare example

---------

Co-authored-by: Be <be.0@gmx.com>
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.

1 participant