Update linux headless ssh docs with new findings #2094
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ReenigneArcher
requested changes
Feb 9, 2024
| @@ -206,6 +206,9 @@ Changing Resolution and Refresh Rate | |||
| | | Undo: ``xrandr --output HDMI-1 --mode 3840x2160 --rate 120`` | | |||
| +----------------------+------------------------------------------------------------------------------------------------------------------------------------+ | |||
|
|
|||
| .. warning:: | |||
There was a problem hiding this comment.
Suggested change
| .. warning:: | |
| .. attention:: |
A warning may be a bit extreme.
Comment on lines
+257
to
+259
| reference: | ||
| - https://wiki.archlinux.org/title/xorg | ||
| - https://winaero.com/find-change-screen-dpi-linux/ |
There was a problem hiding this comment.
Suggested change
| reference: | |
| - https://wiki.archlinux.org/title/xorg | |
| - https://winaero.com/find-change-screen-dpi-linux/ | |
| **reference:** | |
| - https://wiki.archlinux.org/title/xorg | |
| - https://winaero.com/find-change-screen-dpi-linux/ |
| @@ -12,7 +12,8 @@ This is a guide to setup remote SSH into host to startup X server and sunshine w | |||
| The virtual display is accelerated by the NVidia GPU using the TwinView configuration. | |||
|
|
|||
| .. attention:: | |||
| This guide is specific for Xorg and NVidia GPUs. I start the X server using the ``startx`` command. | |||
| This guide is specific for Xorg and NVidia GPUs. This guide assumes the host is not using a desktop environment and only a barebones windows manager. | |||
There was a problem hiding this comment.
What is a "barebones" window manager? Can you provide an example?
Comment on lines
+265
to
+266
| echo 'KERNEL=="uinput", SUBSYSTEM=="misc", MODE="0660", GROUP="input", OPTIONS+="static_node=uinput"' | \ | ||
| sudo tee /etc/udev/rules.d/85-sunshine.rules |
There was a problem hiding this comment.
Should we just update our rules so they work in this situation as well?
Here -> https://github.com/LizardByte/Sunshine/blob/nightly/src_assets/linux/misc/85-sunshine.rules
And anywhere it occurs in the docs?
| To enable root login over SSH edit your SSHD config, and add ``PermitRootLogin yes``, and restart the SSH server. | ||
|
|
||
| #. First make a backup of your ``/etc/sudoers.d/${USER}`` file. | ||
| #. Add the user running sunshine to the `input` group. |
There was a problem hiding this comment.
This used to be part of our instructions... not sure when or why it got removed.
| These changes allow the script to use sudo without being prompted with a password. | ||
|
|
||
| e.g. ``sudo $(pwd)/sunshine-setup.sh`` | ||
| sudo udevadm control --reload-rules && sudo udevadm trigger |
There was a problem hiding this comment.
Nice! I didn't know about these commands... always just rebooted.
| .. code-block:: bash | ||
|
|
||
| xrdb -merge ~/.Xresources | ||
| ... |
There was a problem hiding this comment.
Are the ... literal? If not someone will take them literally.
| .. code-block:: bash | ||
|
|
||
| #!/bin/bash | ||
|
|
||
| ssh_args="<user>@192.168.X.X" # Or use alias set in ~/.ssh/config |
There was a problem hiding this comment.
Why was the comment removed?
Comment on lines
+511
to
+513
| DPI References: | ||
| - https://wiki.archlinux.org/title/xorg | ||
| - https://winaero.com/find-change-screen-dpi-linux/ |
There was a problem hiding this comment.
Suggested change
| DPI References: | |
| - https://wiki.archlinux.org/title/xorg | |
| - https://winaero.com/find-change-screen-dpi-linux/ | |
| **DPI reference:** | |
| - https://wiki.archlinux.org/title/xorg | |
| - https://winaero.com/find-change-screen-dpi-linux/ |
| @@ -522,5 +522,5 @@ If you have any feedback and any suggestions, feel free to make a post on Discor | |||
|
|
|||
| .. seealso:: | |||
| Now that you have a virtual display, you may want to automate changing the resolution | |||
| and refresh rate prior to connecting to an app. See :ref:`Changing Resolution and Refresh Rate | |||
| and refresh rate on the host prior to connecting to an app. See :ref:`Changing Resolution and Refresh Rate | |||
There was a problem hiding this comment.
Suggested change
| and refresh rate on the host prior to connecting to an app. See :ref:`Changing Resolution and Refresh Rate | |
| and refresh rate, on the host, prior to connecting to an app. See :ref:`Changing Resolution and Refresh Rate |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Screenshot
Issues Fixed or Closed
Type of Change
.github/...)Checklist
Branch Updates
LizardByte requires that branches be up-to-date before merging. This means that after any PR is merged, this branch
must be updated before it can be merged. You must also
Allow edits from maintainers.