Re: [PATCH] New sol10priv module

2009-05-03 Thread Jim Meyering
David Bartley wrote: ... > Updated the gnulib patch accordingly. > >> Jim, I leave it to you to commit this for David, since I don't want to >> interfere with your coreutils release. Thanks to both of you. I've just pushed that to gnulib, and the following to coreutils: >From 6249eb0596dfe50e1f95

Re: [PATCH] New sol10priv module

2009-05-03 Thread Bruno Haible
David Bartley wrote: > Updated the gnulib patch accordingly. Thanks. From my point of view, it's perfect now. I leave it to Jim to decide when to commit it. Bruno

Re: [PATCH] New sol10priv module

2009-05-01 Thread David Bartley
On Fri, May 1, 2009 at 4:35 AM, Bruno Haible wrote: > David Bartley wrote: >> I've attached updated patches that hopefully >> address your and Bruno's concerns. > > Looks quite fine. Only a couple of minor points: > >  - m4/priv-set.m4 should do an AC_REQUIRE([AC_C_INLINE]) >    because the .h fil

Re: [PATCH] New sol10priv module

2009-05-01 Thread Bruno Haible
David Bartley wrote: > I've attached updated patches that hopefully > address your and Bruno's concerns. Looks quite fine. Only a couple of minor points: - m4/priv-set.m4 should do an AC_REQUIRE([AC_C_INLINE]) because the .h file uses 'static inline'. - In priv-set.c, maybe add comments

Re: [PATCH] New sol10priv module

2009-04-30 Thread David Bartley
On Wed, Apr 29, 2009 at 1:57 AM, Jim Meyering wrote: > David Bartley wrote: >> The following patches for coreutils and gnulib add a new Solaris 10 >> privilege module as previously suggested [1]. > > Hi David, > > Thanks for following through. > Are you up to the task of making a few more changes?

Re: [PATCH] New sol10priv module

2009-04-29 Thread Jim Meyering
Jim Meyering wrote: > David Bartley wrote: >> The following patches for coreutils and gnulib add a new Solaris 10 >> privilege module as previously suggested [1]. > > Hi David, > > Thanks for following through. > Are you up to the task of making a few more changes? > >

Re: [PATCH] New sol10priv module

2009-04-29 Thread Bruno Haible
Hello David, Thank you for your module, and your patience regarding the paperwork. David Bartley wrote: > The following patches for coreutils and gnulib add a new Solaris 10 > privilege module Five small remarks about the module: 1) In the module description: > +lib/sol10priv.h > +lib/sol10pri

Re: [PATCH] New sol10priv module

2009-04-29 Thread Jim Meyering
David Bartley wrote: > The following patches for coreutils and gnulib add a new Solaris 10 > privilege module as previously suggested [1]. Hi David, Thanks for following through. Are you up to the task of making a few more changes? - n

[PATCH] New sol10priv module

2009-04-29 Thread David Bartley
Hi Jim, The following patches for coreutils and gnulib add a new Solaris 10 privilege module as previously suggested [1]. -- David [1] http://lists.gnu.org/archive/html/bug-gnulib/2009-03/msg00123.html From e67022be8d312fec0fa7619c65b9070171041c4c Mon Sep 17 00:00:00 2001 From: David Bartley Da