Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

The script executed as expected after running it. I was able to build and compile sunshine just perfectly and install it using ninja.

sudo dnf install libstdc++-static
cmake -B build -G Ninja -S . -DSUNSHINE_ENABLE_CUDA=OFF
ninja -C build
sudo ninja install -C build/

I wanted to check with the maintainers whether they think making the libstdc++-static part of the linux build script, as it's required to be built but was missing after running the script.

Note: I disabled CUDA because I'm an AMD user by the way.

Can I also kindly request the maintainer of the Linux packaging to enable support for Fedora 43? I can't do this now as the packaging is done through Copr, and I obviously can not administrate that.

Can I also kindly request the maintainer of the Linux packaging to enable support for Fedora 43? I can't do this now as the packaging is done through Copr, and I obviously can not administrate that.

It's enabled in the beta copr. It will be enabled in the stable the next time we create a release... But we are not going to align releases with any distro as that's too demanding on a small project.

The script executed as expected after running it. I was able to build and compile sunshine just perfectly and install it using ninja.

sudo dnf install libstdc++-static
cmake -B build -G Ninja -S . -DSUNSHINE_ENABLE_CUDA=OFF
ninja -C build
sudo ninja install -C build/

I wanted to check with the maintainers whether they think making the libstdc++-static part of the linux build script, as it's required to be built but was missing after running the script.

Note: I disabled CUDA because I'm an AMD user by the way.

If you're asking if the package should be installed, I think it should.

We don't use this script anymore since we started using copr, so stuff may get missed from time to time.

The script executed as expected after running it. I was able to build and compile sunshine just perfectly and install it using ninja.

sudo dnf install libstdc++-static
cmake -B build -G Ninja -S . -DSUNSHINE_ENABLE_CUDA=OFF
ninja -C build
sudo ninja install -C build/

I wanted to check with the maintainers whether they think making the libstdc++-static part of the linux build script, as it's required to be built but was missing after running the script.
Note: I disabled CUDA because I'm an AMD user by the way.

If you're asking if the package should be installed, I think it should.

We don't use this script anymore since we started using copr, so stuff may get missed from time to time.

I can confirm the package does need to be installed, but I assumed the purpose of the script is to make the environment ready for building, meaning that package should probably be included.

But since you're not using the script anymore I'm not sure if you would want me to make it part of this PR, or have this PR exist in the first place.

The reason I went and changed the script is because there's no way for me to install Sunshine right now with the packages provided by the release on GitHub as that is missing a Fedora 43 build and the Fedora 42 build doesn't work.

The reason I went and changed the script is because there's no way for me to install Sunshine right now with the packages provided by the release on GitHub as that is missing a Fedora 43 build and the Fedora 42 build doesn't work.

The PR is fine, we accept changes for the Fedora portion because some people self compile using this script.

For Fedora 43 support, you'll want to use our beta copr instead of our stable copr. I need to improve the docs to mention why this is, but they do tell you how to enable the beta copr. https://docs.lizardbyte.dev/projects/sunshine/latest/md_docs_2getting__started.html#fedora