]> code.ossystems Code Review - openembedded-core.git/commit
Revert "subversion: fix "svnadmin create" fail on x86"
authorDengke Du <dengke.du@windriver.com>
Mon, 19 Dec 2016 05:40:14 +0000 (13:40 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Thu, 22 Dec 2016 08:46:34 +0000 (08:46 +0000)
commitfbdfb39c011676fe61a4d58b62226126e0e9ec62
treea0f701f6dbc2038f7fb87912d07ffaa897435ac4
parent99d1199fd0961f94732a1a533d66472ca17cf6f5
Revert "subversion: fix "svnadmin create" fail on x86"

This reverts commit cfe6f3e251240c9d9a70354be0501600357f0b87.

This is because the apr configure wrong, when the apr configure meets the
cross compiling, it pass 8 bytes to "off_t", in apr source code configure.in,
it was hardcoded:

APR_CHECK_SIZEOF_EXTENDED([#include <sys/types.h>], off_t, 8)

The macro "APR_CHECK_SIZEOF_EXTENDED" was defined in build/apr_common.m4,
it use the "AC_TRY_RUN" macro, this macro let the off_t to 8, when cross
compiling enable.

But in glibc on the x86 or multilib target the "off_t" was 4 bytes, so this
cases dismatch for softwares which use the apr.h, such as subversion, run this:

svnadmin create test

It failed because the "APR_OFF_T_FMT" was "lld" in apr.h when apr configure,
but the "apr_off_t" was 4 bytes, in the apr source code: apr_snprintf.c

i_quad = va_arg(ap, apr_int64_t);

When the function apr_vformatter meets "lld", it would use the above to parse,
but the above read 8 bytes, so the follow-up data go to wrong.

So we should configure the apr correct when cross compiling. I do this on the
following patchs.

Signed-off-by: Dengke Du <dengke.du@windriver.com>
Signed-off-by: Ross Burton <ross.burton@intel.com>
meta/recipes-devtools/subversion/subversion/0001-fix-svnadmin-create-fail-on-x86.patch [deleted file]
meta/recipes-devtools/subversion/subversion_1.9.5.bb