docs: improve flatpak start commands

[caveat-makes-software]

Description

Corrected documentation of startup calls for Linux+Flatpak: I have removed $(pactl info | awk '/Server String/{print$3}') and replaced it with $(id -u $whoami). This should allow Sunshine to find the Pulse audio server also on Linux installations that are not English.

Issues Fixed or Closed

• Fixes Documentation: Startup call for Linux+Flatpak (sudo -i PULSE_SERVER...) is incorrect #3261
• Fixes Documentation: Fix the instruction for KMS_CAPTURE #2840
• Fixes Flatpak version: without audio or without video #2839

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Dependency update (updates to dependencies)
• Documentation update (changes to documentation)
• Repository update (changes to repository files, e.g. .github/...)

Checklist

• My code follows the style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have added or updated the in code docstring/documentation-blocks for new or existing methods/components

[ReenigneArcher]

@caveat-makes-software Do you think we should include anything else from the issues mentioned above? or #2839 ?

Also does this also solve #2839?

[caveat-makes-software]

Hello again,

yes, the fix for this issue also fixes #2839. I didn't find this issue earlier because it was already closed, but the two issues seem to be identical. The startup call for flatpak that I have provided should make the whole procedure independent of what the language is (I have a Portuguese/BR system as well), and I got it working this way. I think there is nothing else that needs to be included related to this.

Regarding #2840: This issue boils down to the command
which sunshine
not finding anything. I dont know why this is, but I can confirm that it is not working from my side (Fedora 40). Thus: I think the suggestion provided by cjuniorfox should be implemented.

[ReenigneArcher]

@cjuniorfox any thoughts on this? Are you okay with your suggestion in the issue you made being implemented by this PR?

[sonarcloud]

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 SonarCloud

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 11.12%. Comparing base (d80fe79) to head (2686259).
Report is 1 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #3264   +/-   ##
=======================================
  Coverage   11.12%   11.12%           
=======================================
  Files          99       99           
  Lines       17211    17211           
  Branches     8023     8023           
=======================================
  Hits         1915     1915           
  Misses      12594    12594           
  Partials     2702     2702           

Flag	Coverage Δ
Linux	8.42% <ø> (ø)
Windows	5.19% <ø> (ø)
macOS-13	13.60% <ø> (ø)
macOS-14	12.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.