Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-15 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/#review137873 --- Ship it! Ship It! - Jie Yu On June 15, 2016, 10:42 p.m.,

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-15 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/ --- (Updated June 15, 2016, 3:42 p.m.) Review request for mesos, Artem

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-15 Thread Gilbert Song
> On June 15, 2016, 11 a.m., Jie Yu wrote: > > 3rdparty/stout/include/stout/os/posix/su.hpp, line 219 > > > > > > Per your TODO above, maxgroups will be 16 on OSX. If ngroups is greater > > than 16, what will

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-15 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/#review137784 --- 3rdparty/stout/include/stout/os/posix/su.hpp (line 219)

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-15 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/ --- (Updated June 15, 2016, 10:46 a.m.) Review request for mesos, Artem

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-14 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/ --- (Updated June 14, 2016, 4:28 p.m.) Review request for mesos, Artem

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-14 Thread Gilbert Song
> On June 13, 2016, 6:15 p.m., James Peach wrote: > > 3rdparty/stout/include/stout/os/posix/su.hpp, line 177 > > > > > > Bear in mind that ``SC_NGROUPS_MAX`` is not meaningful on Darwin other > > than it is the

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-14 Thread Gilbert Song
> On June 13, 2016, 6:15 p.m., James Peach wrote: > > 3rdparty/stout/include/stout/os/posix/su.hpp, line 177 > > > > > > Bear in mind that ``SC_NGROUPS_MAX`` is not meaningful on Darwin other > > than it is the

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-13 Thread James Peach
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/#review137435 --- 3rdparty/stout/include/stout/os/posix/su.hpp (line 177)

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-13 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/ --- (Updated June 13, 2016, 4:50 p.m.) Review request for mesos, Artem

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-06-10 Thread Jie Yu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/#review137083 --- 3rdparty/stout/include/stout/os/posix/su.hpp (line 177)

Re: Review Request 47667: Added stout functions to get and set supplementary groud ids.

2016-05-26 Thread Gilbert Song
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47667/ --- (Updated May 26, 2016, 1:55 p.m.) Review request for mesos, Artem Harutyunyan,