Hi,
I noticed in apr_dbd_escape that it uses PQescapeString instead of
PQescapeStringConn.
PQescapeString is deprecated and dangerous. The documentation says,
*it might give the wrong results*
http://www.postgresql.org/docs/9.1/static/libpq-exec.html
Is there any objection to changing the API
function described here,
http://msdn.microsoft.com/en-us/library/aa365779(VS.85).aspx
Thanks,
Bob Rossi
On Fri, Feb 05, 2010 at 10:09:47AM +1100, Bojan Smojver wrote:
> On Thu, 2010-02-04 at 18:06 -0500, Bob Rossi wrote:
> > If you are bent on preserving this interface, does a new interface
> > make sense, that allows the user to pass in the parameters,
> > without passing in ad
On Fri, Feb 05, 2010 at 09:51:59AM +1100, Bojan Smojver wrote:
> On Thu, 2010-02-04 at 17:44 -0500, Bob Rossi wrote:
> > Is this a design decision or a flaw?
>
> Design decision.
>
> Just to be clear, you are getting segfaults when you are NOT passing
> pointers in, corr
BTW, is anyone using this interface? It's not even tested in
test/dbd.c. I'm assuming that with the bugs I've found above, no
one has used this interface, without great pain.
What do you think? If you like my suggestions, i'll write up
a patch for inclusion.
Thanks,
Bob Rossi
On Thu, Jan 14, 2010 at 03:47:03PM -0500, Bob Rossi wrote:
> Hi,
>
> I've noticed in the global mutex code, what I consider to be a problem.
> I'm trying to get a global mutex, that multiple process's can use to
> ensure 1 access at a time.
>
> For APR_LOC
On Thu, Jan 14, 2010 at 03:47:03PM -0500, Bob Rossi wrote:
> Hi,
>
> I've noticed in the global mutex code, what I consider to be a problem.
> I'm trying to get a global mutex, that multiple process's can use to
> ensure 1 access at a time.
>
> For APR_LOC
Hi,
I've noticed in the global mutex code, what I consider to be a problem.
I'm trying to get a global mutex, that multiple process's can use to
ensure 1 access at a time.
For APR_LOCK_SYSVSEM, in proc_mutex_sysv_create, semget is called with
IPC_PRIVATE, instead of using ftok to convert the file
eople working on APR/APU a volunteers.
Sure, but what should we do, isn't one of the goals of apr to be
portable?
Should I do a binary-search build to see when it broke, and report the
patch that broke it?
Thanks,
Bob Rossi
On Thu, Aug 07, 2008 at 10:02:51PM +0200, Ruediger Pluem wrote:
> What obstacles are left between us and an 1.3.3 release of APR / APR-UTIL?
Getting apr/apr-util to port cleanly to mingw? I can help with testing.
Bob Rossi
On Thu, Jun 26, 2008 at 09:37:41PM -0400, Bob Rossi wrote:
> Hi,
>
> I tried compiling apr 1.3.2 on mingw just now.
>
> I got this compiler error,
>
> /bin/sh apr/apr-1.3.2/apr-1.3.2/libtool --silent --mode=compile gcc-sjlj
> -g -gstabs -Wall -DHAVE_CONFIG_H -D
ke: *** [all-recursive] Error 1
configure: error: "make failed with a non-zero exit code 2."
See `config.log' for more details.
Is this a known issue? I can give more details if not.
Thanks,
Bob Rossi
pgsql_error picks it up?
>
> Not exactly critical - just a thought.
Maybe a way to allow the user to get access to the messages too?
I did notice that when I do this on startup,
set client_min_messages = 'WARNING';
that the messages stop. Perhaps this is a good enough work around?
I'm not sure, I don't know entirely the philosophy of the apr group.
Bob Rossi
Hi,
One issue I've noticed with postgresql is that when you use it through
apr, and postgresql detects an error, it prints to stdout.
Do you think that apr should catch this, or do you think that my
application should get the handle, and catch it?
Thanks,
Bob Rossi
On Thu, May 22, 2008 at 05:43:27AM +1000, Bojan Smojver wrote:
> On Wed, 2008-05-21 at 10:13 -0400, Bob Rossi wrote:
>
> > OK, I have confirmed 1 bug. If you pass '0' to apr_dbd_get_row, with
> > postgresql as the driver, the function returns 0, instead of -1 as
On Wed, May 21, 2008 at 10:13:41AM -0400, Bob Rossi wrote:
> On Mon, May 19, 2008 at 09:32:07PM -0400, Bob Rossi wrote:
> > On Mon, May 19, 2008 at 08:58:32AM +1000, Bojan Smojver wrote:
> > > On Sat, 2008-05-17 at 09:07 -0400, Bob Rossi wrote:
> > >
> > > &g
On Mon, May 19, 2008 at 09:32:07PM -0400, Bob Rossi wrote:
> On Mon, May 19, 2008 at 08:58:32AM +1000, Bojan Smojver wrote:
> > On Sat, 2008-05-17 at 09:07 -0400, Bob Rossi wrote:
> >
> > > I've been using sqlite3 dbd for some time now. I find that the f
On Wed, May 21, 2008 at 11:36:55AM +1000, Bojan Smojver wrote:
> On Tue, 2008-05-20 at 20:26 -0400, Bob Rossi wrote:
> > That's because as you can see the -lpq went at the
> > end which is incorrect.
>
> The real problem is that we're adding libraries to LDFLAGS
On Wed, May 21, 2008 at 10:07:25AM +1000, Bojan Smojver wrote:
> On Tue, 2008-05-20 at 20:04 -0400, Bob Rossi wrote:
>
> > I noticed you changed everything but the AC_CHECK_LIB, do you believe
> > that is unnecessary, or are you thinking about it?
>
> Not required. We
On Wed, May 21, 2008 at 09:49:37AM +1000, Bojan Smojver wrote:
> On Tue, 2008-05-20 at 19:34 -0400, Bob Rossi wrote:
> > --- apr-util-1.3.x-2008-05-19-svn/build/dbd.m4 Tue May 20 17:15:38 2008
> > +++ apr-util-1.3.x-2008-05-19-svn.changed/build/dbd.m4 Tue May 20 19:18:23
> &
], [$pgsql_LDFLAGS]))
This is the key really. This is the only part of apr (that I know of)
that needs to link with these libraries. It's just the test program. Am
I wrong? If the --libs are not passed to AC_CHECK_LIB, the test will
fail on windows. This is also done in a couple places, and I only
modified the case that I was getting coverage on.
Is this a sufficient explanation?
Thanks,
Bob Rossi
On Wed, May 21, 2008 at 06:59:53AM +1000, Bojan Smojver wrote:
> On Tue, 2008-05-20 at 06:55 -0400, Bob Rossi wrote:
>
> > OK, I can try this, however, this patch still doesn't work on windows. I
> > get link errors on windows because the the libs aren't linked in.
&
ged for dynamic linking is that this
creates unnecessary package dependencies. But for static linking, it sounds
OK to me. I guess we have never really bothered to carefully specify the
static case.
Thanks,
Bob Rossi
On Mon, May 19, 2008 at 08:58:32AM +1000, Bojan Smojver wrote:
> On Sat, 2008-05-17 at 09:07 -0400, Bob Rossi wrote:
>
> > I've been using sqlite3 dbd for some time now. I find that the function
> > apr_dbd_get_row,
> >
> > http:/
r.
>
> BTW, if --libdir alone doesn't cut it for you for some reason, I'll add
> --libs back too. But, if it does, I prefer to keep it simple.
BAM,
#define APU_HAVE_PGSQL 1
Let me test windows now. Sorry, this takes quiet a bit longer. I'll get
back to you.
Bob Rossi
On Tue, May 20, 2008 at 10:11:38AM +1000, Bojan Smojver wrote:
> On Mon, 2008-05-19 at 20:05 -0400, Bob Rossi wrote:
>
> > If you make the change, I'll test out the latest snapshots and all your
> > fixes for me. Thanks so much Bojan!
>
> Done. And you are welcom
>
> OK, I'll do --libs then.
OK, I hope I'm making things better, and not worse. I just found out
that -lpq is normally enough. The reason I'm seeing the issue is because
I'm linking statically.
If you make the change, I'll test out the latest snapshots and all your
fixes for me. Thanks so much Bojan!
Bob Rossi
On Mon, May 19, 2008 at 04:22:18PM -0500, William A. Rowe, Jr. wrote:
> Bojan Smojver wrote:
>> On Mon, 2008-05-19 at 16:07 -0400, Bob Rossi wrote:
>>
>>> You know, I really think you should be using the --libs there. What
>>> could the downside be? It's not
On Tue, May 20, 2008 at 07:05:51AM +1000, Bojan Smojver wrote:
> On Mon, 2008-05-19 at 15:54 -0400, Bob Rossi wrote:
>
> > So basically I need -lSecur32 and ws2_32. What can I do about this?
>
> I don't have any Windows machines (nor development tools), so I cannot
>
On Fri, May 16, 2008 at 11:21:46AM +1000, Bojan Smojver wrote:
> On Thu, 2008-05-15 at 09:08 -0400, Bob Rossi wrote:
> > So,
> > I get a link error, because it needs to link against -lcrypt.
>
> What kind of system do you have? Where does the -lcrypt live on your
> sys
On Sat, May 17, 2008 at 10:23:59AM +1000, Bojan Smojver wrote:
> On Fri, 2008-05-16 at 06:26 -0400, Bob Rossi wrote:
>
> > configure:40279: gcc -o conftest -g -Wall -pthread -DLINUX=2
> > -D_REENTRANT -D_GNU_SOURCE conftest.c -lcrypt >&5
> > configure:4028
On Mon, May 19, 2008 at 08:58:32AM +1000, Bojan Smojver wrote:
> On Sat, 2008-05-17 at 09:07 -0400, Bob Rossi wrote:
>
> > I've been using sqlite3 dbd for some time now. I find that the function
> > apr_dbd_get_row,
> >
> > http:/
ren't you worried about how that will break all the existing programs
out there using this interface?
Bob Rossi
s in 1 and 2 to
apr_dbd_get_row. With postgresql driver, I have to pass in 0 and 1.
Is this something I should code up on my side?
Thanks,
Bob Rossi
On Sat, May 17, 2008 at 09:10:50PM +1000, Bojan Smojver wrote:
> On Sat, 2008-05-17 at 06:56 -0400, Bob Rossi wrote:
>
> > Huh, I build with --disable-shared --enable-static. Perhaps it doesn't
> > work in this configuration?
>
> Possibly. Have you tried with --di
On Sat, May 17, 2008 at 06:56:28AM -0400, Bob Rossi wrote:
> On Sat, May 17, 2008 at 01:08:59PM +1000, Bojan Smojver wrote:
> > On Fri, 2008-05-16 at 22:43 -0400, Bob Rossi wrote:
> >
> > > I just upgraded to the 1.3 svn snapshot, to test out the new
> > > func
On Sat, May 17, 2008 at 01:08:59PM +1000, Bojan Smojver wrote:
> On Fri, 2008-05-16 at 22:43 -0400, Bob Rossi wrote:
>
> > I just upgraded to the 1.3 svn snapshot, to test out the new
> > functionality that was recently added. After I ran my unit tests I
> > noticed that t
/* APU_DSO_BUILD */
is not in the translation unit anymore.
Any idea why this would have changed? Something I did wrong?
Thanks,
Bob Rossi
On Sat, May 17, 2008 at 10:23:59AM +1000, Bojan Smojver wrote:
> On Fri, 2008-05-16 at 06:26 -0400, Bob Rossi wrote:
>
> > configure:40279: gcc -o conftest -g -Wall -pthread -DLINUX=2
> > -D_REENTRANT -D_GNU_SOURCE conftest.c -lcrypt >&5
> > configure:4028
On Sat, May 17, 2008 at 08:28:33AM +1000, Bojan Smojver wrote:
> On Fri, 2008-05-16 at 16:32 -0400, Bob Rossi wrote:
>
> > I'm looking at dbd_pgsql_open.
>
> http://apr.apache.org/docs/apr-util/trunk/group___a_p_r___util___d_b_d.html#gbddb1fdcb2f8a5f5b83127485c78e8ae
Wow
ld be called bewtween the
PQstatus and the PQfinish. However, the api,
static apr_dbd_t *dbd_pgsql_open(apr_pool_t *pool, const char *params)
doesn't allow for a 'string' message to be returned.
What can we do? I find this interface useless, if I can't get an error
message when it fails.
Bob Rossi
On Fri, May 16, 2008 at 01:34:29PM +1000, Bojan Smojver wrote:
> On Thu, 2008-05-15 at 23:18 -0400, Bob Rossi wrote:
>
> > Ubuntu 7.04, /lib/libcrypt-2.5.so
>
> OK.
>
> > OK, perhaps we should just add -lcrypt to the pg line? I could check to
> > see if this wor
On Fri, May 16, 2008 at 11:21:46AM +1000, Bojan Smojver wrote:
> On Thu, 2008-05-15 at 09:08 -0400, Bob Rossi wrote:
> > So,
> > I get a link error, because it needs to link against -lcrypt.
>
> What kind of system do you have? Where does the -lcrypt live on your
> sy
get a link error, because it needs to link against -lcrypt.
Attached is the patch that fixes the problem, what do you think?
Bob Rossi
--- apr-util-2006-12-14-cvs-2/apr-util-2006-12-14-cvs/build/dbd.m4 2006-12-14 20:41:02.0 -0500
+++ apr-util-2006-12-14-cvs-3/apr-util-2006-12-14-cvs/build/dbd.
On Fri, Nov 30, 2007 at 12:12:16PM +1100, Bojan Smojver wrote:
> On Thu, 2007-11-29 at 17:34 -0500, Bob Rossi wrote:
>
> > OK, perhaps we could have some high level documentation on the webpage
> > for the dbd documentation that describes which databases it works with?
>
&g
On Fri, Nov 30, 2007 at 09:21:56AM +1100, Bojan Smojver wrote:
> On Thu, 2007-11-29 at 16:43 -0500, Bob Rossi wrote:
>
> > I'm a newb with database connections, so forgive me if this is a simple
> > question. I'm looking into using the dbd routines to open connect
pen connections from different sources using the apr API.
Thanks,
Bob Rossi
me other the big wigs respond to see if
this would be acceptable.
Bob Rossi
27;t think we should use APR_CHECK_SIZEOF_EXTENDED at all anymore.
If someone submitted a patch to convert these to
AC_CHECK_SIZEOF
would that be acceptable?
Thanks,
Bob Rossi
r. There may be others that use
it in the configuration you have. If so, I hope they pipe up and let us
know.
Bob Rossi
e to turn for an answer
I have never tried this configuration. Sorry. I've only built apr in a
msys window, and compiled my applicatoin there as well. Maybe someone
else has experience in this area...
Bob Rossi
On Mon, Jul 09, 2007 at 02:53:17PM -0400, Bob Rossi wrote:
> On Mon, Jul 09, 2007 at 12:59:08PM -0500, Myers, Gary - 43 wrote:
> > I'm attempting to use APR in a mingw C project that I am working on to
> > add a layer of abstraction so I can later port in to Linux.
> > I&
tion. (Which may be my problem, I simply may not be able to make it
> work using mingw gcc)
Why can't you get it to work with mingw? Take a look at
http://wiki.apache.org/logging-log4cxx/MSWindowsBuildInstructions
Bob Rossi
Hi,
I've noticed talk about making a tag on the svn branch. Does this
project usually have a call to make sure that a particular version
builds on most platforms before a release is made?
In other words, is there a release candidate?
Thanks,
Bob Rossi
ow to
build log4cxx with mingw here,
http://wiki.apache.org/logging-log4cxx/MSWindowsBuildInstructions.
I thought this information was also relevant for building apr.
Hope this is helpful,
Bob Rossi
_sizeof_ssize_t=4
> + ac_cv_sizeof_size_t=4
> + ;;
The above I don't think should be addeed. Instead we should no longer
use APR_CHECK_SIZEOF_EXTENDED and instaed use
http://www.gnu.org/software/autoconf/manual/autoconf-2.57/html_mono/autoconf.html#IDX472
which claims to work when cross compiling. That way, no constants are
needed.
Thanks,
Bob Rossi
On Mon, Mar 19, 2007 at 09:54:21PM +0200, Aizer Danny-BDA023 wrote:
> > -Original Message-
> > From: Bob Rossi:
> >
> > On Mon, Mar 19, 2007 at 04:17:38PM +0200, Aizer Danny-BDA023 wrote:
> > > I am cross-compiling Apache HTTPD 2.2.3 on a Red Hat Linux 9
of ppc_82xx) instead of running the cross-compiler to
>
> calculate the value.
I think you will have to modify configure.in to fit your purposes. If
you submit a patch back, maybe you can make these features allowed
to be passed in on the command line to configure?
Thanks,
Bob Rossi
y automatically link in when
*** you link to this library. But I can only do this if you have a
*** shared version of the library, which you do not appear to have.
libtool: link: warning: undefined symbols not allowed in i686-pc-mingw32 shared
libraries
Is this normal?
Thanks,
Bob Rossi
On Wed, Dec 13, 2006 at 09:20:43AM -0800, Paul Querna wrote:
> Bob Rossi wrote:
> >On Wed, Dec 13, 2006 at 10:41:24AM -0500, Bob Rossi wrote:
> >>On Wed, Dec 13, 2006 at 10:05:16AM -0500, Bob Rossi wrote:
> >>>Hi,
> >>>
> >>>I'm trying
On Wed, Dec 13, 2006 at 10:41:24AM -0500, Bob Rossi wrote:
> On Wed, Dec 13, 2006 at 10:05:16AM -0500, Bob Rossi wrote:
> > Hi,
> >
> > I'm trying to build apr-util with mingw so that I can attempt to build
> > log4cxx there.
> >
> > Attach
On Wed, Dec 13, 2006 at 10:05:16AM -0500, Bob Rossi wrote:
> Hi,
>
> I'm trying to build apr-util with mingw so that I can attempt to build
> log4cxx there.
>
> Attached is the output of ./configure, which fails.
>
> Has anyone built apr-util with mingw? I'm
Hi,
I'm trying to build apr-util with mingw so that I can attempt to build
log4cxx there.
Attached is the output of ./configure, which fails.
Has anyone built apr-util with mingw? I'm using svn trunk.
Thanks,
Bob Rossi
checking build system type... i686-pc-mingw32
checking host s
On Sun, Dec 10, 2006 at 08:42:29PM +1100, John Vandenberg wrote:
> On 12/9/06, Bob Rossi <[EMAIL PROTECTED]> wrote:
> >On Wed, Dec 06, 2006 at 09:07:20AM +1100, John Vandenberg wrote:
> >> We should first check whether the libtool developers intentionally
> >> a
tarballs of apr.
Was i supposed to do this?
Thanks,
Bob Rossi
eaving directory `/home/kelli/apr/apr/test'
Is this a regression in apr svn trunk?
Also, if I used cl to compile apr, would that make more tests pass? For
example, does cl somehow have more capabilities on windows than mingw?
Thanks,
Bob Rossi
On Wed, Dec 06, 2006 at 02:24:16PM +1100, John Vandenberg wrote:
> On 12/2/06, Bob Rossi <[EMAIL PROTECTED]> wrote:
> >Hi,
> >
> >...
> >
> >Secondly, the APR_CHECK_SIZEOF_EXTENDED isn't working for me because the
> >default for the cross compi
On Tue, Dec 05, 2006 at 03:35:21PM -0600, William A. Rowe, Jr. wrote:
> Bob Rossi wrote:
> > Hi,
> >
> > Well, here is the compile command.
> >
> > /bin/sh
> > /home/bobbybrasko/rcs/svn/vigilant/vigilant/builddir/vigilant-tools/apr/libtool
> > -
; http://marc2.theaimsgroup.com/?l=apr-dev&m=114506483217503&w=2
Intereseting. So everything appears to be correct? For some reason,
testsock works now, and it didn't before.
I'd love to see the sed command go into apr. I'm still wondering what
should be done about the m4 macros.
Thanks,
Bob Rossi
-tools/apr/include/arch/win32
-I./include/arch/unix
-I/home/bobbybrasko/rcs/svn/vigilant/vigilant/builddir/vigilant-tools/apr/include
-o dso/win32/dso.lo -c dso/win32/dso.c && touch dso/win32/dso.lo
And yes,
#define APR_HAS_DSO 1
Thanks,
Bob Rossi
On Tue, Dec 05, 2006 at 03
t causing it to throw up?
Thanks for the quick response. This is the output of my configure run.
Is something obviously wrong here?
Checking for DSO...
checking for NSLinkModule... no
checking for shl_load in -ldld... no
checking for dlopen... no
checking for dlopen in -ldl... yes
adding "-ldl" to LIBS
checking for dlsym... yes
Thanks,
Bob Rossi
On Tue, Dec 05, 2006 at 10:16:52AM +1100, John Vandenberg wrote:
> Hi Bob,
>
> On 12/2/06, Bob Rossi <[EMAIL PROTECTED]> wrote:
>
> >First, and oddly enough, the buildconf script fails for me because
> >build/buildcheck.sh doesn't recognize my libtool --versi
On Tue, Dec 05, 2006 at 12:34:36PM +, Joe Orton wrote:
> On Fri, Dec 01, 2006 at 10:10:39PM -0500, Bob Rossi wrote:
> > Secondly, the APR_CHECK_SIZEOF_EXTENDED isn't working for me because the
> > default for the cross compiled size is 8. However, ssize_t for me is 4.
&
On Fri, Dec 01, 2006 at 10:10:39PM -0500, Bob Rossi wrote:
> Hi,
>
> I've successfully cross compiled apr on cygwin to mingw. I've already
> had one patch go upstream to autoconf, which fixes a problem
> AC_CHECK_SIZEOF.
>
> I have two very small other problems
stalled, and it ran fine. It was simply a hello world
type program with apr stuff in it.
I would really appreciate if these thoughts could be considered, so I
could create a patch to send upstream.
Thanks,
Bob Rossi
On Thu, Nov 30, 2006 at 07:29:32AM -0800, Justin Erenkrantz wrote:
> On 11/30/06, Bob Rossi <[EMAIL PROTECTED]> wrote:
> >Does anyone know the status of building apr with mingw on windows?
>
> If you use trunk, MinGW should work fine. -- justin
Hi Justin,
I've also tr
On Thu, Nov 30, 2006 at 07:29:32AM -0800, Justin Erenkrantz wrote:
> On 11/30/06, Bob Rossi <[EMAIL PROTECTED]> wrote:
> >Does anyone know the status of building apr with mingw on windows?
>
> If you use trunk, MinGW should work fine. -- justin
Hi Justin,
Thank
nGW under Cygwin, and
> MinGW doesn't provide any of the shared memory header files (shm.h,
> sys/mmap.h, sys/mman.h, etc.) so the test fails.
I've run into the same problem, and was wondering if this can be fixed
somehow?
Thanks,
Bob Rossi
lem already. I'm sure it
would benefit a lot of people.
Thanks for the help,
Bob Rossi
79 matches
Mail list logo
