Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Branko ibej
[EMAIL PROTECTED] wrote: striker 2003/09/27 16:51:46 Modified:misc/unix start.c memory/unix apr_pools.c Log: * misc/unix/start.c (apr_initialize): Remove atomics initialization from this function. Due to circular dependency, doing it here is too late.

RE: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Sander Striker
From: Branko Cibej [mailto:[EMAIL PROTECTED] Sent: Monday, September 29, 2003 2:56 AM [snip] I can't believe we don't have a Win32 implementation of the atomics. So right now, HEAD doesn't compile on Windows. :-( Sure we do. Take a look at include/apr_atomic.h Is the implementation in

Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Branko ibej
Sander Striker wrote: From: Branko Cibej [mailto:[EMAIL PROTECTED] Sent: Monday, September 29, 2003 2:56 AM [snip] I can't believe we don't have a Win32 implementation of the atomics. So right now, HEAD doesn't compile on Windows. :-( Sure we do. Take a look at

Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Branko ibej
Branko ibej wrote: Also, I'm being double stupid because it's not APR that doesn't compile, it's APR-iconv. Ho hum. Now how can that happen? /me grumbles and looks Huh, not surprising. apr_atomic.h does _not_ define apr_atomic_init on Win32, Netware, FreeBSD, Linux, MVS and djgpp; no wonder

RE: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Sander Striker
From: Branko Cibej [mailto:[EMAIL PROTECTED] Sent: Monday, September 29, 2003 3:22 AM Branko ibej wrote: Also, I'm being double stupid because it's not APR that doesn't compile, it's APR-iconv. Ho hum. Now how can that happen? /me grumbles and looks Huh, not surprising.

Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Branko ibej
Sander Striker wrote: From: Branko Cibej [mailto:[EMAIL PROTECTED] Sent: Monday, September 29, 2003 3:22 AM Branko ibej wrote: Also, I'm being double stupid because it's not APR that doesn't compile, it's APR-iconv. Ho hum. Now how can that happen? /me grumbles and looks

RE: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Sander Striker
From: Branko Cibej [mailto:[EMAIL PROTECTED] Sent: Monday, September 29, 2003 3:31 AM [...] Looks like a definition to me... Why doesn't this work? Uh -- I said HEAD, not the 0.9 branch. Doh. Since there's a whole bunch of atomics that do _not_ have a Win32-specific implementation

Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Cliff Woolley
On Mon, 29 Sep 2003, [UTF-8] Branko ^Libej wrote: Huh, not surprising. apr_atomic.h does _not_ define apr_atomic_init on Win32, Netware, FreeBSD, Linux, MVS and djgpp; no wonder there are dangling references to it in apr_pools.obj. Uh -- I said HEAD, not the 0.9 branch. As a data point, the

Re: cvs commit: apr/memory/unix apr_pools.c

2003-09-29 Thread Branko ibej
Cliff Woolley wrote: On Mon, 29 Sep 2003, [UTF-8] Branko ?^Libej wrote: Huh, not surprising. apr_atomic.h does _not_ define apr_atomic_init on Win32, Netware, FreeBSD, Linux, MVS and djgpp; no wonder there are dangling references to it in apr_pools.obj. Uh -- I said HEAD, not the

Re: problem building apr on darwin

2003-09-29 Thread Jeff Trawick
chiggsy wrote: Hello all, I am having a problem building apr on darwin. I am using : darwin 7.0.0 gcc 3.3 and here is the error i get : poll.c: In function `apr_poll': poll.c:126: error: storage size of `pollset' isn't known problem was caused by finding poll() but no header file with

Re: cvs commit: apr configure.in Makefile.in

2003-09-29 Thread Joe Orton
On Mon, Sep 29, 2003 at 02:20:02PM -, [EMAIL PROTECTED] wrote: ... $(TARGET_LIB): @for i in $(SUBDIRS); do objects=$$objects $$i/[EMAIL PROTECTED]@; done ; \ - tmpcmd=$(LINK) @lib_target@ @lib_target_libs@; \ + tmpcmd=$(LINK) @lib_target@ $(ALL_LIBS); \

RE: Tagged 2.0

2003-09-29 Thread Brad Nicholes
-1 on NetWare. With the new call to apr_atomic_init() that was added to apr_pool.c, apr_pool.c must now #include apr_atomic.h in order to pick up the NETWARE macro for apr_atomic_init. Without it, it doesn't compile. Brad Brad Nicholes Senior Software Engineer Novell, Inc., the leading

[PATCH] move APR_INADDR_NONE definition

2003-09-29 Thread Joe Orton
Is there any reason why APR_INADDR_NONE must be defined in apr.h and detected at configure-time which I'm missing? Index: configure.in === RCS file: /home/cvs/apr/configure.in,v retrieving revision 1.538 diff -u -r1.538 configure.in

Re: Tagged 2.0

2003-09-29 Thread The Doctor
+1 on BSD/OS!! On Mon, Sep 29, 2003 at 10:19:07AM -0600, Brad Nicholes wrote: -1 on NetWare. With the new call to apr_atomic_init() that was added to apr_pool.c, apr_pool.c must now #include apr_atomic.h in order to pick up the NETWARE macro for apr_atomic_init. Without it, it doesn't