maltejur / discord-screenaudio

A custom discord client that supports streaming with audio on Linux.
1.09k stars 35 forks source link

Offline build support #3

Closed VortexAcherontic closed 2 years ago

VortexAcherontic commented 2 years ago

Hello there, while attempting to build this project on OBS (open build service) I noticed you do use git submodules and cloning some git release of rohrkabel directly into your discord-screeaudio source directory at build time.

This is generally a bad idea as most build services like those for openSUSE (OBS), Debian, Fedora, Ubuntu, flatpak and others do build their packages offline.

In order to allow for offline building I do the following:

-FetchContent_Declare(

The final source directory then looks as follows:

./discord-screenaudio-1.0.0-rc.5/
| assets/
| build/
| CMakeLists.txt
| LICENSE
| README.md
| resources.qrc
| rohrkabel/
| scripts/
| src/

It would be great to allow for build flags to swap between offline build and online build.

Kind regards,
V.

maltejur commented 2 years ago

Good point, I'll add something like this. Or I may even switch completely to a submodule.

VortexAcherontic commented 2 years ago

With version 1.0.0-rc6 I consider this as closed as there is no need to manually patch the CMakeList.txt anymore.
Many thanks for the fast work ❤️

maltejur commented 2 years ago

@VortexAcherontic just letting you know, for v1.0.0 I implemented a mechanism which automatically pulls the current version from git tags. If you build from a tarball, you have to provide it manually like in the PKGBUILD.

VortexAcherontic commented 2 years ago

@maltejur Thanks for letting me know. Already doing this since RC 5 😄