Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
mod_mirrorbrain victim of incompatibility between apr-util 1.2 and #4
Issue migrated (2015-06-05) from old issue tracker http://mirrorbrain.org/issues/issue7
msg17 (view) Author: poeml Date: 2009-10-06.22:28:51
Ah. Interesting one.
Apache successcully connects and talks to the database, and it does find the file on two
...something else seems wrong, and I'm quite puzzled by this one.
In fact, after reading mod_mirrorbrain.c back and forth I'm quite sure
The module correctly gets (and logs) the number of rows in the result,
I suspect that the Apache Portable Runtime version of Ubuntu 9.04, and
Yes, this is probably the reason. On my Ubuntu 9.04 system I see a
That would explain the error.
I can probably come up with a workaround. Sorry about the inconvenience...
msg19 (view) Author: poeml Date: 2009-10-06.23:19:52
The following patch seemingly makes mod_mirrorbrain work with the old libaprutil:
--- mod_mirrorbrain.c (revision 7798)
I think we can quite easily change mod_mirrorbrain to work with both APR versions,
I don't know whether this fix works well, I can't really promise that at the
msg22 (view) Author: poeml Date: 2009-10-08.07:52:36
I'm adding the proposed fix to the Ubuntu/Debian packages.
I have not learnt yet how to properly add the change as patch when packaging Debian packages in the openSUSE build
in the debian/rules file has the desired effect at the moment. Not nice, but for pragmatical reasons (also because
msg28 (view) Author: poeml Date: 2009-10-08.10:59:49
The packages with the fix are scheduled to be built in the openSUSE build service. They might
msg76 (view) Author: poeml Date: 2009-12-01.21:24:38
After sugarlabs is running with the fix for a month already, I think we can close
And since it is easy to do compile time detection of the APR version, I'll commit
msg77 (view) Author: poeml Date: 2009-12-01.21:32:27
Fixed in trunk:
Will be in the next release.
msg88 (view) Author: poeml Date: 2009-12-03.23:05:30
See also issue 29.
msg212 (view) Author: poeml Date: 2010-09-03.12:13:00
The check could be done at runtime, rather than at compile time.
That would protect the user in scenarios where a wrong package is installed (one
There is apr_version() which returns the runtime APR version. See apr_version.h.
Apache uses it:
httpd2 -V | grep "APR "
Server loaded: APR 1.4.2, APR-Util 1.3.9
It would also allow us to log the versions to the error log at Apache's start.
msg215 (view) Author: poeml Date: 2010-09-05.19:01:23
mod_asn got a fix for this now (r86).
msg217 (view) Author: poeml Date: 2010-09-05.20:10:07
mod_mirrorbrain now has the same fix (r8107).
msg218 (view) Author: poeml Date: 2010-09-05.23:36:10
There is one remaining compile-time if-branch, which is not about database access
if (APR_MAJOR_VERSION == 1 && APR_MINOR_VERSION == 2)
define DBD_LLD_FMT "d"
define DBD_LLD_FMT "lld"
I'm leaving that as is, since the consequences are much less relevant (if it
(end of migrated issue)