-
Notifications
You must be signed in to change notification settings - Fork 37
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
Repo contains pre-built binaries, BUILDING.md is missing #2
Comments
Came here as I also got a 404 for the BUILDING.md file. Any info which which node version(s) are supported? |
I did some digging in the meantime and I think this repo is prebuilt binaries of https://github.com/signalapp/ringrtc plus some bindings. Possibly because the build instructions would become more complicated due to the dependency on rust. It'd be great if there'd be an option to build the content of this repo from source as well. :) |
I did some work on prebuilding for different architectures in signalapp/ringrtc#12 and now have a GitHub workflow set up for this: https://github.com/dennisameling/signal-native-deps/blob/34f9117a9f592665196b84afb37779023504391f/.github/workflows/build.yaml#L162-L282, which you can see in action here: https://github.com/dennisameling/signal-native-deps/actions/runs/871146189 Thing is, it takes ~1hr to build ringrtc because it needs to build Signal's WebRTC fork as well, so you really don't want to bother builders of Signal Desktop with this. Basically the build steps are:
|
hey!
Any chance
build/
can be dropped from the repo? The concern is that pre-built binaries are pulled into the signal-desktop build, for example the only difference between signal-desktop 1.35.1 and 1.35.2 seems to be 4ad4a1b, which is a little intransparent. Preferably the binary would be built during the regular signal-desktop build.There's a BUILDING.md linked in the README, but the link 404s unfortunately.
Thank you very much!
The text was updated successfully, but these errors were encountered: