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

udev rule in README is broken #7

Open
foxcpp opened this issue May 18, 2019 · 1 comment
Open

udev rule in README is broken #7

foxcpp opened this issue May 18, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@foxcpp
Copy link

foxcpp commented May 18, 2019

RUN
...
Starting daemons or other long running processes is not appropriate for udev; the forked processes, detached or not, will be unconditionally killed after the event handling has finished.

@foxpy
Copy link
Owner

foxpy commented May 19, 2019

Uh oh, I have accidentally published the wrong udev rule. The one I have used was spawning serialjs process with systemd-run. It technically worked, yes, despite being an ugly and defective solution. I am going to add systemd service to repo when I have any USB device emulating COM port to actually debug it.

foxpy added a commit that referenced this issue May 19, 2019
Starting serialjs with systemd-run ensures it wouldn't be killed by udev
after a short time.
However, this solution makes it impossible to handle multiple gamepads
and boards at the same time since we are killing all serialjs instances
before spawning one. This is a temporary solution and I will fix it
later as described in #7.
@foxpy foxpy added the enhancement New feature or request label May 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants