1.13.0 coreutils/id.c calls libc getgrouplist()

[Hamish Moffatt]

On Mon, Nov 17, 2008 at 01:12:35AM +0100, Denys Vlasenko wrote:
> On Monday 17 November 2008 00:51, Hamish Moffatt wrote:
> > On Sun, Nov 16, 2008 at 04:04:06PM +0100, Denys Vlasenko wrote:
> > > On Sunday 16 November 2008 09:18, Tito wrote:
> > > > On Sunday 16 November 2008 04:21:10 Denys Vlasenko wrote:
> > > > Maybe we can add some warning like:
> > > > 
> > > > #if defined (__UCLIBC__) \
> > > > && (__UCLIBC_MAJOR__ == 0 && __UCLIBC_MINOR__ < 9) \
> > > > || (__UCLIBC_MINOR__ == 9 &&  __UCLIBC_SUBLEVEL__ < 30)
> > > > #warning "Sorry, you need at least uClibc version 0.9.30 for getgrouplist :("
> > > > #endif
> > > > 
> > > > Attached you will find a patch with the above check,
> > > > please apply if you like it.
> > > > The patch is only roughly tested as i don't use uClibc 
> > > > (but seems to work as expected).
> > > 
> > > Applied and tested it to work as intended with 0.9.29 and 0.9.30.
> > > Thanks!
> > 
> > Shouldn't it check that !defined(CONFIG_USE_BB_PWD_GRP) though? id works
> > if that is enabled.
> 
> Fixed, thanks for noticing.

Thanks, will you update the patch for 1.13.0?

Hamish
-- 
Hamish Moffatt VK3SB <hamish at debian.org> <hamish at cloud.net.au>