Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
should there be a problem with the sddm user, be more verbose instead of normal exit #397
Conversation
davidedmundson
reviewed
Apr 4, 2015
| - bail(2); | ||
| - if (chdir(pw->pw_dir) != 0) | ||
| - bail(2); | ||
| + //TODO: why do we need to fetch the info from the passwd file? |
|
Looks good. Make that one change I want and I'll merge this. Thanks. |
|
i removed that comment and fix a syntax error I had. |
|
Thanks very much. |
davidedmundson
closed this
Apr 4, 2015
davidedmundson
reopened this
Apr 4, 2015
added a commit
that referenced
this pull request
Apr 4, 2015
davidedmundson
merged commit 9fa9177
into
sddm:master
Apr 4, 2015
1 check was pending
continuous-integration/travis-ci/pr
The Travis CI build is in progress
Details
mathieujobin
deleted the
mathieujobin:helper_verbose_exit
branch
Apr 4, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
mathieujobin commentedApr 4, 2015
fix in relation to issue #371
print verbose messages in case there is a problem with sddm user.
also, do failure exit instead of normal exit
cheers,