Re: [bug-gnulib] New GNULIB glob module?

2005-05-25 Thread Derek Price
Larry Jones wrote: >Derek Price writes: > > >>Larry, can you tell us if defining >>_POSIX_PTHREAD_SEMANTICS would work to get the POSIX version of >>getpwnam_r on Solaris? >> >> > >It looks like it. > > I've committed Paul's patch to the CVS CVS tree, as well as removing the associated gl

[bug-gnulib] Re: New getlogin_r module

2005-05-25 Thread Derek Price
Bruno Haible wrote: >Derek Price wrote: > > >>This is what I installed. >> >> > >Note that lib/ and m4/ have ChangeLogs of their own. Only the top-level >and modules/* modifications go into the top-level ChangeLog. Including >the update of MODULES.html.sh (adding one line for each new module

Re: [bug-gnulib] New getlogin_r module

2005-05-25 Thread Paul Eggert
Derek Price <[EMAIL PROTECTED]> writes: > It sets *and* returns errno since my local man pages make it sound > like errno will be set by getlogin_r and others may already have > code which checks errno and not getlogin_r's return value. But glibc getlogin_r doesn't always set errno to the returne

[bug-gnulib] new modules for C# interoperability

2005-05-25 Thread Bruno Haible
After the modules for Java, here come the modules for C# interoperability. C# is a language which lies between C++ and Java, nearer to Java than to C++. It is ECMA standardized and has at least 2 free implementations (pnet and mono). Like Java, it allows to write conceptually simple tasks - that wo

Re: [bug-gnulib] New GNULIB glob module?

2005-05-25 Thread Larry Jones
Derek Price writes: > > Larry, can you tell us if defining > _POSIX_PTHREAD_SEMANTICS would work to get the POSIX version of > getpwnam_r on Solaris? It looks like it. -Larry Jones I never get to do anything fun. -- Calvin ___ bug-gnulib mailing lis

[bug-gnulib] Re: New getlogin_r module

2005-05-25 Thread Bruno Haible
Derek Price wrote: > This is what I installed. Note that lib/ and m4/ have ChangeLogs of their own. Only the top-level and modules/* modifications go into the top-level ChangeLog. Including the update of MODULES.html.sh (adding one line for each new module). Three further remarks: - In the inclu

[bug-gnulib] Re: New getlogin_r module

2005-05-25 Thread Bruno Haible
Derek Price wrote: > How about something like the attached patch to the modules/memcpy file? That's a bit redundant. It's already mentioned in MODULES.html. Bruno ___ bug-gnulib mailing list bug-gnulib@gnu.org http://lists.gnu.org/mailman/listinfo/bu

Re: [bug-gnulib] New GNULIB glob module?

2005-05-25 Thread Derek Price
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Paul Eggert wrote: >Derek Price <[EMAIL PROTECTED]> writes: > >>+# ifdef HAVE___POSIX_GETPWNAM_R >>+ /* Solaris. */ >>+# define getpwnam_r(name, bufp, buf, len, res) \ >>+ __posix_getpwnam_r (name, bufp, buf, len, res) >>+# endif > > >I don't see why

Re: [bug-gnulib] [bug-gnulib] New getlogin_r module

2005-05-25 Thread Derek Price
Bruno Haible wrote: >Would you mind adding a bit of documentation? Something like > I've added the following to getlogin_r.h: /* Copies the user's login name to NAME. The array pointed to by NAME has room for SIZE bytes. Returns 0 if successful. Upon error, an error number is re

Re: [bug-gnulib] [bug-gnulib] New getlogin_r module

2005-05-25 Thread Derek Price
Bruno Haible wrote: >Derek Price wrote: > > >>+Depends-on: >>+memcpy >> >> > >Modules of the category "Support for systems lacking ANSI C 89" are not >listed among dependencies. We assume that projects needing support for K&R C >and SunOS 4 will collect these modules themselves. > > My ba

Re: [bug-gnulib] Re: [bug-gnulib] stat and lstat should define their replacements

2005-05-25 Thread Derek Price
Bruno Haible wrote: >Derek Price <[EMAIL PROTECTED]> writes: > > >>As near as I can tell, stat and lstat do not define names for their >>replacements as many of the other GNULIB modules do. >> >> > >Yes. The 'stat' and 'lstat' modules look incomplete. I think this should >be added to make th

Re: [bug-gnulib] Re: [bug-gnulib] stat and lstat should define their replacements

2005-05-25 Thread Bruno Haible
Derek Price <[EMAIL PROTECTED]> writes: > As near as I can tell, stat and lstat do not define names for their > replacements as many of the other GNULIB modules do. Yes. The 'stat' and 'lstat' modules look incomplete. I think this should be added to make them usable out-of-the-box. diff -c -3 -r1

Re: [bug-gnulib] [bug-gnulib] New getlogin_r module

2005-05-25 Thread Bruno Haible
Derek Price wrote: > +int > +getlogin_r (char *name, size_t size) > +{ > + char *n = getlogin (); > + if (n) > +{ > + size_t nlen = strlen (n); > + if (nlen < size) > +{ > + memcpy (name, n, nlen + 1); > + return 0; > +} > + errno = ERANGE; > +

Re: [bug-gnulib] [bug-gnulib] New getlogin_r module

2005-05-25 Thread Bruno Haible
Derek Price wrote: > +Depends-on: > +memcpy Modules of the category "Support for systems lacking ANSI C 89" are not listed among dependencies. We assume that projects needing support for K&R C and SunOS 4 will collect these modules themselves. Bruno

Re: [bug-gnulib] [bug-gnulib] New getlogin_r module

2005-05-25 Thread Bruno Haible
Derek Price wrote: > +#if !HAVE_DECL_GETLOGIN_R > +int getlogin_r (char *name, size_t size); > +#endif Would you mind adding a bit of documentation? Something like /* Copies the user's login name to NAME. The array pointed to by NAME has room for SIZE bytes. Returns 0 if successful. Upon er

Re: [bug-gnulib] stat and lstat should define their replacements

2005-05-25 Thread Paul Eggert
Derek Price <[EMAIL PROTECTED]> writes: > +/* Get the system versions when something else was defined by config.h. */ > +#undef lstat > +#undef stat A big worry here is hosts that use something like "#define stat stat64" when compiled in large-file mode. On such hosts, you can't #undef 'stat' a

[bug-gnulib] Re: New getlogin_r module

2005-05-25 Thread Paul Eggert
Derek Price <[EMAIL PROTECTED]> writes: > Per your suggestion, Paul, I've attached a getlogin_r module for > review. It compiles and works here as gl_GETLOGIN_R or > gl_GETLOGIN_R_SUBSTITUTE. That looks good to me. Can you please install it, along with appropriate ChangeLog entries? I added yo