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.

Sign up for GitHub

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

Jump to bottom

xvfb pidfile should be created under $XDG_RUNTIME_DIR #143

Closed
dniku opened this issue Feb 27, 2019 · 0 comments · Fixed by #208
Closed

xvfb pidfile should be created under $XDG_RUNTIME_DIR #143

dniku opened this issue Feb 27, 2019 · 0 comments · Fixed by #208
Assignees
@dniku
Labels
priority:low

Comments

@dniku
Copy link
Collaborator

dniku commented Feb 27, 2019 •
edited
Loading

Creating it in the current directory may cause issues if the script is run twice from different locations.

A workaround for environments where $XDG_RUNTIME_DIR is not defined is described here.

Relevant: https://superuser.com/a/1127720/390791
@dniku dniku self-assigned this Feb 27, 2019
@dniku dniku mentioned this issue Feb 27, 2019
Merged
dniku added a commit that referenced this issue Jun 6, 2019
@dniku
Improve xvfb script (#143)
7f5a518 
* Make it executable
* Add shebang
* Improve formatting
* Move pidfile to $XDG_RUNTIME_DIR

Fixes #143.
@dniku dniku mentioned this issue Jun 6, 2019
Merged
dniku added a commit that referenced this issue Jun 6, 2019
@dniku
Improve xvfb script (#143)
d0f0c28 
* Make it executable
* Add shebang
* Improve formatting
* Move pidfile to $XDG_RUNTIME_DIR

Fixes #143.
justheuristic added a commit that referenced this issue Jun 16, 2019
@justheuristic
Merge pull request #208 from yandexdataschool/spring19-xvfb
a00ade4 
Improve xvfb script (#143)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dniku dniku

Labels
priority:low
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Improve xvfb script (#143) yandexdataschool/Practical_RL
1 participant
@dniku