qyriad/lix
1
0
Fork 0
forked from lix-project/lix
Code Pull requests Activity

Fix Darwin build

Browse source 
Turns out getgrouplist() is not POSIX.

http://hydra.nixos.org/build/23881243
This commit is contained in:
Eelco Dolstra 2015-07-21 14:45:24 +02:00
parent 0a2bee307b
commit 1993b10d11
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/libstore/build.cc
View file

@ -543,6 +543,7 @@ void UserLock::acquire()
throw Error(format("the Nix user should not be a member of ‘%1%’")
% settings.buildUsersGroup);
#if __linux__
/* Get the list of supplementary groups of this build user. This
is usually either empty or contains a group such as "kvm". */
supplementaryGIDs.resize(10);
@ -553,6 +554,7 @@ void UserLock::acquire()
throw Error(format("failed to get list of supplementary groups for ‘%1%’") % pw->pw_name);
supplementaryGIDs.resize(ngroups);
#endif
return;
}
@ -2372,7 +2374,8 @@ void DerivationGoal::runChild()
if (buildUser.enabled()) {
/* Preserve supplementary groups of the build user, to allow
admins to specify groups such as "kvm". */
if (setgroups(buildUser.getSupplementaryGIDs().size(),
if (!buildUser.getSupplementaryGIDs().empty() &&
setgroups(buildUser.getSupplementaryGIDs().size(),
buildUser.getSupplementaryGIDs().data()) == -1)
throw SysError("cannot set supplementary groups of build user");