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

tmpfiles.d/xpra.conf references nonexistant group #1093

Closed
totaam opened this issue Jan 19, 2016 · 6 comments
Closed

tmpfiles.d/xpra.conf references nonexistant group #1093

totaam opened this issue Jan 19, 2016 · 6 comments

Comments

@totaam
Copy link
Collaborator

totaam commented Jan 19, 2016

Issue migrated from trac ticket # 1093

component: packaging | priority: minor | resolution: fixed

2016-01-19 12:56:44: tribut created the issue


On Debian jessie, systemd-tmpfiles-setup.service and systemd-tmpfiles-clean.service fail because /usr/lib/tmpfiles.d/xpra.conf references the group xpra, which does not exist:

systemd-tmpfiles[842]: [/usr/lib/tmpfiles.d/xpra.conf:3] Unknown group 'xpra'.

I believe the package should create the group at install time.

@totaam
Copy link
Collaborator Author

totaam commented Jan 20, 2016

2016-01-20 22:12:04: antoine changed owner from antoine to tribut

@totaam
Copy link
Collaborator Author

totaam commented Jan 20, 2016

2016-01-20 22:12:04: antoine commented


r11696 should do what you want, please try the beta packages and close this ticket if that works for you.

@totaam
Copy link
Collaborator Author

totaam commented Jan 21, 2016

2016-01-21 10:17:26: tribut changed status from new to closed

@totaam
Copy link
Collaborator Author

totaam commented Jan 21, 2016

2016-01-21 10:17:26: tribut set resolution to fixed

@totaam
Copy link
Collaborator Author

totaam commented Jan 21, 2016

2016-01-21 10:17:26: tribut commented


Thanks, that does it.

However, a slightly cleaner way to check for group existence would be

if ! getent group $SERVER_GROUP >/dev/null; then

@totaam totaam closed this as completed Jan 21, 2016
@totaam
Copy link
Collaborator Author

totaam commented Jan 21, 2016

2016-01-21 19:45:03: antoine commented


I've backported this change to v0.16.x in 11706 and will be included in 0.16.1

r11707 uses the syntax from comment:2 (trunk only).

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

1 participant