Skip to content
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

Updated alternative solution for Xorg's "max clients" issue (further distros). #264

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Goli4thus
Copy link
Contributor

Not as bad as I thought regarding how much "bulk" it would add to that section.

@robbert-vdh
Copy link
Owner

Do you happen to know if this information is documented externally? Maintaining this many distro-specific instructions in a troubleshooting section of the readme doesn't sound ideal, especially since most people won't need to use this and it adds quite a bit of bulk. (I'd put it in a <details> tag so it can be collapsed, but it's still a lot of information)

@Goli4thus
Copy link
Contributor Author

Ah sorry, maybe I got your previous question on the other pull request wrong.

No, I'm not aware of this information being anywhere else. At least not easily discoverable.
As I mentioned in other PR, the only reason I found this approach in the first place (re other PR) is via this link: https://unix.stackexchange.com/questions/498652/does-x-window-have-a-maximum-number-limit-on-clients

The information for this PR was literally something I came up by trial-and-error running each of those distros in a VM and trying to apply the original approach re other PR.

I agree that it's kinda out of place to go into this much detail in the README for something this specific for multiple distros.

So if you think putting it into a <details> tag makes it bearable to have it here, then feel free to do so.
If you think this just doesn't fit here, then let's just forget about this PR.
Apart from that, I'm not sure where else this could go. A random gist?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants