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

Allow service definition without explicit group #78

Closed
wants to merge 1 commit into from
Closed

Allow service definition without explicit group #78

wants to merge 1 commit into from

Conversation

smoeding
Copy link
Contributor

@smoeding smoeding commented Jul 4, 2016

The group setting may be omitted in a service configuration to use the users primary group. This config is currently not possible because setting the group parameter to undef will use the os specific default group instead. This patch adds an additional parameter use_default_group to control this behaviour. Using the default true keeps the old behaviour. Setting the parameter to false and leaving the group unset will omit the group setting from the service config file.

@smoeding
Copy link
Contributor Author

smoeding commented Jul 4, 2016

Travis CI failure seems to be caused by gem requirement Ruby version >= 2.0.

The group setting may be omitted in a service configuration to use the
users primary group. This config is currently not possible because
setting the group parameter to 'undef' will use the os specific default
group instead. This patch adds an additional parameter
'use_default_group' to control this behaviour. Using the default 'true'
keeps the old behaviour. Setting the parameter to 'false' and leaving
the group unset will omit the group setting from the service config
file.
@smoeding
Copy link
Contributor Author

Tests are successful after modulesync update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants