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

keyword: add missing gname keyword functions #129

Merged
merged 2 commits into from
Jun 15, 2017
Merged

keyword: add missing gname keyword functions #129

merged 2 commits into from
Jun 15, 2017

Conversation

vbatts
Copy link
Owner

@vbatts vbatts commented Jun 15, 2017

Fixes: #128

Reported-by: Lennart Poettering lennart@poettering.net
Signed-off-by: Vincent Batts vbatts@hashbangbash.com

Fixes: #128

Reported-by: Lennart Poettering <lennart@poettering.net>
Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts
Copy link
Owner Author

vbatts commented Jun 15, 2017

dang. the os/user.LookupGroupId() was introduced in go1.7, so go1.6 does not have a clear path for gname. Presumably why I "forgot" to add it in the first place.

@vbatts vbatts force-pushed the fix_gname branch 3 times, most recently from a096b8b to 4a19c87 Compare June 15, 2017 17:32
Signed-off-by: Vincent Batts <vbatts@hashbangbash.com>
@vbatts vbatts merged commit abdee6f into master Jun 15, 2017
@vbatts vbatts deleted the fix_gname branch June 15, 2017 17:43
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

Successfully merging this pull request may close these issues.

1 participant