[Bug libc/1060] New: sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[Bug libc/1060] New: sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

As part of gnulib we've ported glibc sysdeps/generic/glob.c and posix/glob.h
to several other platforms, and fixed a couple of bugs.  We'd like to merge our
improvements and fixes into glibc.  The attached patch contains the first part
of our changes.  These are merely porting changes, useful when glob.c and glob.h
are employed outside of glibc; they should not affect glibc's behavior.

-- 
           Summary: sysdeps/generic/glob.c merge from gnulib (part 1 of 3)
           Product: glibc
           Version: 2.3.5
            Status: NEW
          Severity: normal
          Priority: P2
         Component: libc
        AssignedTo: gotom at debian dot or dot jp
        ReportedBy: eggert at gnu dot org
                CC: glibc-bugs at sources dot redhat dot com


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

------- Additional Comments From eggert at gnu dot org  2005-07-12 07:03 -------
Created an attachment (id=552)
 --> (http://sources.redhat.com/bugzilla/attachment.cgi?id=552&action=view)
b


-- 


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
    Attachment #552|b                           |sysdeps/generic/glob.c merge
        description|                            |from gnulib (part 1 of 3)


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |1062
              nThis|                            |


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[derek at ximbiot dot com]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |derek at ximbiot dot com


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[ldv at altlinux dot org]

------- Additional Comments From ldv at altlinux dot org  2005-08-24 23:31 -------
The LOGIN_NAME_MAX macro introduced by this patch is already defined with
another value by bits/local_lim.h

Could you please consider renaming LOGIN_NAME_MAX?

-- 


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[derek at ximbiot dot com]

------- Additional Comments From derek at ximbiot dot com  2005-08-25 01:52 -------
Created an attachment (id=617)
 --> (http://sources.redhat.com/bugzilla/attachment.cgi?id=617&action=view)
Revised patch.

How is this?  I replaced occurrances of LOGIN_NAME_MAX with GET_LOGIN_NAME_MAX.

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #552 is|0                           |1
           obsolete|                            |


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

------- Additional Comments From eggert at gnu dot org  2005-09-01 22:37 -------
Yes, renaming LOGIN_NAME_MAX is fine.  I've installed the revised
patch with GET_LOGIN_NAME_MAX into gnulib.  Thanks.

-- 


http://sources.redhat.com/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[roland at gnu dot org]

------- Additional Comments From roland at gnu dot org  2005-09-08 07:47 -------
Do prototypize the defns for external functions such as glob itself.
Using K&R defns ensures maximal warnings for missing decls.

The removal of old portability cruft in glob.c and replacement with gnulib cruft
is fine.  Send a patch just doing that.

The glob.h changes are way too much cruft in the installed header.
Users of gnulib glob can include whatever gnulib headers they need before glob.h.
 

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |SUSPENDED


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[derek at ximbiot dot com]

------- Additional Comments From derek at ximbiot dot com  2005-09-08 15:44 -------
Created an attachment (id=655)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=655&action=view)
Revised patch.

It looks like you are right, and most of our changes to glob.h could be moved
into a pre-included header.  If it turns out anything needs to stay in there,
I'll open another issue.

I've attached a revised patch containing only the changes to glob.c.

Thanks again,

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #617 is|0                           |1
           obsolete|                            |


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

------- Additional Comments From eggert at gnu dot org  2005-09-08 20:32 -------
Created an attachment (id=657)
 --> (http://sourceware.org/bugzilla/attachment.cgi?id=657&action=view)
improved patch to match current sources and use old-style defns

Roland asked that we use old-style function defns for external
functions, and p3.diff didn't do that.	Also, p3.diff didn't apply to
the current sources due to more-recent changes.  I fixed both problems
and attach a fixed patch.

Roland, can you please explain why it's nice to use old-style function
definitions here?  Doesn't gcc's -Wmissing-prototypes option suffice
to detect missing declarations?


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[eggert at gnu dot org]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #655 is|0                           |1
           obsolete|                            |


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[roland at gnu dot org]

------- Additional Comments From roland at gnu dot org  2005-09-08 23:32 -------
The log entry in that attachment still says "move GLOB_* definitions to glob.h",
though glob.h is unchanged.  Is having #include <glob.h> early without the
#undef GLOB_* ok for portability, given glob.h won't change to do any #undef's?

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[derek at ximbiot dot com]

------- Additional Comments From derek at ximbiot dot com  2005-09-09 01:42 -------
I think it will be fine.  Now that <glob.h> is included first, it's definitions
for the GLOB_* macros should take precedence, unless the headers that
"erroneously define" them do something extremely bad, like #undeffing them first
or failing to protect their definitions with #ifndef GLOB_WHATEVER, but I say
that bridge would best be crossed when we get to it.

The ChangeLog message was probably slightly inaccurate.  How about this
corercted one (I only changed the one sentence)?

2005-07-11  Derek Price  <derek@ximbiot.com>
        and Paul Eggert  <eggert@cs.ucla.edu>

    * sysdeps/generic/glob.c:  Update copyright.  Assume freestanding
    C89 compiler.  Simplify cruft that may be replaced with GNULIB
    modules.  Make no attempt to find 64-bit versions of file access
    functions directly when !_LIBC.  Avoid undef'ing GLOB_* macros by
    #including <glob.h> before potentially offending headers.
    (DIRENT_MUST_BE, DIRENT_MIGHT_BE_SYMLINK, DIRENT_MIGHT_BE_DIR): New
    macros to abstract dirent->d_type access.
    (GETPW_R_SIZE_MAX, GET_LOGIN_NAME_MAX): New macros to abstract sysconf
    access.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[drepper at redhat dot com]

------- Additional Comments From drepper at redhat dot com  2005-09-17 18:07 -------
Where is the equivalent of the _GNU_GLOB_INTERFACE_VERSION tests?  How will code
using this gnulib nonsense determine whether the system has an adequate glob
implementation which is usable?

And don't remove casts from malloc et.al calls.

-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[aj at suse dot de]

-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|gotom at debian dot or dot  |roland at gnu dot org
                   |jp                          |
             Status|SUSPENDED                   |ASSIGNED


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2006-01-11 05:30 -------
Subject: Bug 1060

CVSROOT:	/cvs/glibc
Module name:	libc
Changes by:	roland@sources.redhat.com	2006-01-11 05:30:03

Modified files:
	posix          : glob.c 

Log message:
	2006-01-10  Derek Price  <derek@ximbiot.com> Paul Eggert  <eggert@cs.ucla.edu>
	
	[BZ #1060]
	* posix/glob.c: Assume freestanding C89 compiler.  Simplify cruft that
	may be replaced with GNULIB modules.  Make no attempt to find 64-bit
	versions of file access functions directly when [!_LIBC].
	Don't define GLOB_* macros here.
	(DIRENT_MUST_BE, DIRENT_MIGHT_BE_SYMLINK, DIRENT_MIGHT_BE_DIR): New
	macros to abstract dirent->d_type access.
	(GETPW_R_SIZE_MAX, GET_LOGIN_NAME_MAX): New macros to abstract sysconf
	access.

Patches:
http://sources.redhat.com/cgi-bin/cvsweb.cgi/libc/posix/glob.c.diff?cvsroot=glibc&r1=1.68&r2=1.69



-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug libc/1060] sysdeps/generic/glob.c merge from gnulib (part 1 of 3)

[roland at gnu dot org]

------- Additional Comments From roland at gnu dot org  2006-01-11 05:31 -------
I've put in the patch, removing the changes we don't want.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


http://sourceware.org/bugzilla/show_bug.cgi?id=1060

------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.