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

add xauthority for X11 permission issues #53

Open
wants to merge 2 commits into
base: humble
Choose a base branch
from

Conversation

SubaruArai
Copy link
Contributor

@SubaruArai SubaruArai commented May 23, 2023

Under a freshly installed fedora38, this template didn't work due to X11 authentication issues.
While this PR should fix it, it needs to be tested with other configurations.

Although to be fair, this should work properly - it's basically the same logic as in osrf/rocker.
See: https://github.com/osrf/rocker/blob/2b8d5abb18c829d22f290679b739dc53765198ad/src/rocker/nvidia_extension.py#L36

@athackst
Copy link
Owner

athackst commented Jun 3, 2023

This doesn't quite work right for me.

I think because nmerge - is looking for a file that doesn't exist?

[3308 ms] Start: Run: /bin/sh -c if [ ! -f /tmp/.1pupkplp20mn9ec9445bfflqpf6b2eoceah3fpn8qv5nc4p0gcrb.xauth ]; then touch /tmp/.1pupkplp20mn9ec9445bfflqpf6b2eoceah3fpn8qv5nc4p0gcrb.xauth; fi && xauth nlist :0 | sed -e 's/^..../ffff/' | xauth -f /tmp/.1pupkplp20mn9ec9445bfflqpf6b2eoceah3fpn8qv5nc4p0gcrb.xauth nmerge -
xauth:  file /home/athackst/.Xauthority does not exist
xauth: (argv):1:  unable to read any entries from file "(stdin)"

You also might consider splitting this out into a shell script file and calling it instead of the command inline.

@SubaruArai
Copy link
Contributor Author

Thanks, I'll check for the error and split into a shell script file.

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