Re: [PATCH] Use getgrouplist where available.

2008-02-22 Thread Jim Meyering
James Youngman <[EMAIL PROTECTED]> wrote: > 2008-02-18 James Youngman <[EMAIL PROTECTED]> > > * gl/m4/mgetgroups.m4: Check for getgrouplist. > * gl/lib/mgetgroups.c (mgetgroups): Use getgrouplist, if > available. > * TODO: Remove the item about switching to getgrouplist. .

Re: [PATCH] Use getgrouplist where available.

2008-02-22 Thread James Youngman
On Fri, Feb 22, 2008 at 9:04 AM, Jim Meyering <[EMAIL PROTECTED]> wrote: > + while (1) > + { > + GETGROUPS_T *h; > + ng = getgrouplist (username, gid, g, &max_n_groups); > + if (0 <= ng) > + { > + *groups = g; > + return

Re: [PATCH] Use getgrouplist where available.

2008-02-22 Thread Jim Meyering
"James Youngman" <[EMAIL PROTECTED]> wrote: > On Fri, Feb 22, 2008 at 9:04 AM, Jim Meyering <[EMAIL PROTECTED]> wrote: ... >> + h = realloc (g, max_n_groups * sizeof *h); > > Shouldn't realloc here be xnrealloc? No. This function is intended to be usable from a library. I.e., no fair cal

Re: [PATCH] Use getgrouplist where available.

2008-02-22 Thread Jim Meyering
Jim Meyering <[EMAIL PROTECTED]> wrote: > "James Youngman" <[EMAIL PROTECTED]> wrote: >> On Fri, Feb 22, 2008 at 9:04 AM, Jim Meyering <[EMAIL PROTECTED]> wrote: > ... >>> + h = realloc (g, max_n_groups * sizeof *h); >> >> Shouldn't realloc here be xnrealloc? > > No. This function is int

Re: [PATCH] Use getgrouplist where available.

2008-02-22 Thread James Youngman
(I have moved the discussion to the gnulib mailing list; bug-coreutils is BCC'ed) On Fri, Feb 22, 2008 at 9:27 AM, Jim Meyering <[EMAIL PROTECTED]> wrote: > No. I note your subsequent mail, and yes, I was indeed thinking about oversized allocations. > This function is intended to be usable from

Re: rename("symlink-to-dir/", "name") behavior

2008-02-22 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Eric Blake on 1/28/2008 6:28 AM: | According to Geoff Clare on 1/28/2008 2:57 AM: | |> | |> My strict reading of the current wording in draft 4 does not permit | Linux' | |> behavior (even though it is more useful, in my opinion), since t

Re: rename("symlink-to-dir/", "name") behavior

2008-02-22 Thread Jim Meyering
Eric Blake <[EMAIL PROTECTED]> wrote: > I wrote the aardvark along those lines, and it was rejected in yesterday's > meeting of the Austin Group. They argued that Linux is allowed to fail to > follow symlink-to-dir/ in the rename and rmdir case, but only if it > returns a different errno than ENOT

Re: rename("symlink-to-dir/", "name") behavior

2008-02-22 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to Jim Meyering on 2/22/2008 6:09 AM: |> I wonder if we would have much luck proposing a patch to the Linux kernel |> folks to do just that? | | Do you see another errno symbol name that makes sense? | I think that ENOTDIR makes the most sen

nohup breaks logname and tty commands

2008-02-22 Thread ROTH, MATTHEW G (ATTSI)
As we are moving from RHEL 4.x to 5.x I noticed that some scripts started failing. When I researched this I found that RHEL 4.x used coreutils 5.2.1 and RHEL 5.x used coreutils 5.97. In RHEL 4 with coreutils 5.2.1 the following works: $ /usr/bin/nohup /usr/bin/logname /usr/bin/no

Re: nohup breaks logname and tty commands

2008-02-22 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to ROTH, MATTHEW G (ATTSI) on 2/22/2008 9:06 AM: | As we are moving from RHEL 4.x to 5.x I noticed that some scripts | started failing. You may want to consider further upgrades - the latest stable version of coreutils is 6.10. | In RHEL 5

Re: nohup breaks logname and tty commands

2008-02-22 Thread Andreas Schwab
Eric Blake <[EMAIL PROTECTED]> writes: > According to ROTH, MATTHEW G (ATTSI) on 2/22/2008 9:06 AM: > | In RHEL 5 with coreutils 5.97 the following does not work: > | > | $ /usr/bin/nohup /usr/bin/logname > | /usr/bin/nohup: appending output to `nohup.out' > | > | $ cat nohup.out > |

how to apply for adding a change or a patch in gnu-coreutils-6.9 and whom to contact for the same

2008-02-22 Thread nilesh kumar
My name is Nilesh Kumar studying in Vellore Institute Of Technology University, Vellore, Tamil Nadu. I would like to know as to how to apply for adding a change or a patch in gnu-coreutils-6.9 in more detail. Waiting in anticipation. Thanking you. Yours faithfully Nilesh Kumar ___

Re: how to apply for adding a change or a patch in gnu-coreutils-6.9 and whom to contact for the same

2008-02-22 Thread Eric Blake
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 According to nilesh kumar on 2/22/2008 12:23 PM: | I would like to know as to how to | apply for adding a change or a patch in gnu-coreutils-6.9 in more | detail. This is the right list. Feel free to post your patches here, although if they are non-

Re: rename("symlink-to-dir/", "name") behavior

2008-02-22 Thread James Youngman
On Fri, Feb 22, 2008 at 1:09 PM, Jim Meyering <[EMAIL PROTECTED]> wrote: > Do you see another errno symbol name that makes sense? ENOTSUP? James. ___ Bug-coreutils mailing list Bug-coreutils@gnu.org http://lists.gnu.org/mailman/listinfo/bug-coreutils

`cp -p` incorrectly sets g+s bit when fs supports acl

2008-02-22 Thread Mike Frysinger
i'm using coreutils-6.10 with acl-2.2.47 on linux-2.6.24. when using ext2 with acl support enabled, `cp -p` on a directory who does not have the g+s bit set but whose parent does have g+s set, the new destination directory will have the g+s bit set. if the filesystem is remounted with acl supp

Re: `cp -p` incorrectly sets g+s bit when fs supports acl

2008-02-22 Thread Paul Eggert
Mike Frysinger <[EMAIL PROTECTED]> writes: > i'm using coreutils-6.10 with acl-2.2.47 on linux-2.6.24. when using ext2 > with acl support enabled, `cp -p` on a directory who does not have the g+s > bit set but whose parent does have g+s set, the new destination directory > will have the g+s bi

Re: `cp -p` incorrectly sets g+s bit when fs supports acl

2008-02-22 Thread Mike Frysinger
On Saturday 23 February 2008, Paul Eggert wrote: > Mike Frysinger <[EMAIL PROTECTED]> writes: > > i'm using coreutils-6.10 with acl-2.2.47 on linux-2.6.24. when using > > ext2 with acl support enabled, `cp -p` on a directory who does not have > > the g+s bit set but whose parent does have g+s set,

Re: rename("symlink-to-dir/", "name") behavior

2008-02-22 Thread Jim Meyering
Eric Blake <[EMAIL PROTECTED]> wrote: > According to Jim Meyering on 2/22/2008 6:09 AM: > |> I wonder if we would have much luck proposing a patch to the Linux kernel > |> folks to do just that? > | > | Do you see another errno symbol name that makes sense? > | I think that ENOTDIR makes the most