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

Detect X server exit status instead of doing timeouts #18

Open
midenok opened this issue Jul 28, 2019 · 4 comments
Open

Detect X server exit status instead of doing timeouts #18

midenok opened this issue Jul 28, 2019 · 4 comments

Comments

@midenok
Copy link

midenok commented Jul 28, 2019

There are 3 reasons why server exited:

  1. Crash;
  2. Configuration error;
  3. User initiative (including Ctrl + Alt + Backspace).

Usually they are differentiated by exit code (it also includes signal which caused termination). That's what nodm must do correspondingly:

  1. Restart and if crashed again in short timeout exit without further restarts;
  2. Exit immediately;
  3. Restart (without timeouts) or exit depending on configuration.

So timeout should be accounted only for crash and only one time (who needs X server crashing more frequently than working).

@sunweaver
Copy link
Collaborator

sunweaver commented Jul 28, 2019 via email

@sunweaver
Copy link
Collaborator

sunweaver commented Jul 28, 2019 via email

@mckaygerhard
Copy link

just for this issue maybe this help: softlab@f1a0f09

@sunweaver
Copy link
Collaborator

just for this issue maybe this help: softlab@f1a0f09

I just dropped @softlab a note via his nodm fork on Github. The nodm display manager is in deep need for another upstream maintainer. I pinged @softlab if he'd be willing to step up for this job. It needs someone who uses nodm on a regular basis and who encounters errors when they pop up.

Neither @spanezz nor myself (@sunweaver) are maintaining nodm actively anymore. I have also stopped using it (because I lack the customer use cases these days). But, still, I think that nodm fills a gap in the display manager universe.

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

No branches or pull requests

3 participants