mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Marc Kleine-Budde <mkl@pengutronix.de>
To: ptxdist@pengutronix.de
Cc: Oliver Graute <oliver.graute@neuhaus.de>
Subject: Re: [ptxdist] [PATCHv2 2/2] apr: added patches for apr-1.5.1
Date: Wed, 04 Mar 2015 11:04:51 +0100	[thread overview]
Message-ID: <54F6D8C3.9000004@pengutronix.de> (raw)
In-Reply-To: <1425463143-3905-2-git-send-email-oliver.graute@neuhaus.de>


[-- Attachment #1.1: Type: text/plain, Size: 5690 bytes --]

On 03/04/2015 10:59 AM, Oliver Graute wrote:
> patches to compile apr on arm platform
> 
> Signed-off-by: Oliver Graute <oliver.graute@neuhaus.de>
> ---
>  patches/apr-1.5.1/0001-apr-gen-test-char.patch |   69 +++++++++++++++++++
>  patches/apr-1.5.1/0002-ac-check-sizeof.patch   |   22 +++++++
>  patches/apr-1.5.1/0003-Makefile-in.patch       |   84 ++++++++++++++++++++++++
>  patches/apr-1.5.1/series                       |    2 +
>  4 files changed, 177 insertions(+)
>  create mode 100644 patches/apr-1.5.1/0001-apr-gen-test-char.patch
>  create mode 100644 patches/apr-1.5.1/0002-ac-check-sizeof.patch
>  create mode 100644 patches/apr-1.5.1/0003-Makefile-in.patch
>  create mode 100644 patches/apr-1.5.1/series
> 
> diff --git a/patches/apr-1.5.1/0001-apr-gen-test-char.patch b/patches/apr-1.5.1/0001-apr-gen-test-char.patch
> new file mode 100644
> index 0000000..12d47f6
> --- /dev/null
> +++ b/patches/apr-1.5.1/0001-apr-gen-test-char.patch
> @@ -0,0 +1,69 @@
> +diff -rupN AK/apr-config.in AL/apr-config.in
> +--- AK/apr-config.in	2015-02-12 11:36:28.000000000 +0100
> ++++ AL/apr-config.in	2015-02-12 11:44:24.515350580 +0100
> +@@ -200,7 +200,7 @@ while test $# -gt 0; do
> +         ### avoid using -L if libdir is a "standard" location like /usr/lib
> +         # Since the user is specifying they are linking with libtool, we
> +         # *know* that -R will be recognized by libtool.
> +-        flags="$flags -L$libdir -R$libdir -l${APR_LIBNAME}"
> ++        flags="$flags -L@SYSROOT@$libdir -l${APR_LIBNAME}"

I think this is done automatically during the install stage.

> +     else
> +         flags="$flags $LA_FILE"
> +     fi
> +diff -rupN AK/include/private/apr_escape_test_char.h AL/include/private/apr_escape_test_char.h
> +--- AK/include/private/apr_escape_test_char.h	1970-01-01 01:00:00.000000000 +0100
> ++++ AL/include/private/apr_escape_test_char.h	2015-02-12 13:53:10.195232200 +0100
> +@@ -0,0 +1,23 @@
> ++/* this file is automatically generated by gen_test_char, do not edit. "make include/private/apr_escape_test_char.h" to regenerate. */
> ++#define T_ESCAPE_SHELL_CMD     (1)
> ++#define T_ESCAPE_PATH_SEGMENT  (2)
> ++#define T_OS_ESCAPE_PATH       (4)
> ++#define T_ESCAPE_ECHO          (8)
> ++#define T_ESCAPE_URLENCODED    (16)
> ++#define T_ESCAPE_XML           (32)
> ++
> ++static const unsigned char test_char_table[256] = {
> ++    32,30,30,30,30,30,30,30,30,30,31,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,6,16,63,22,17,22,49,17,
> ++    17,17,1,16,16,0,0,18,0,0,0,0,0,0,0,0,0,0,16,23,
> ++    55,16,55,23,16,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
> ++    0,0,0,0,0,0,0,0,0,0,0,23,31,23,23,0,23,0,0,0,
> ++    0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,
> ++    0,0,0,23,23,23,17,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,
> ++    30,30,30,30,30,30,30,30,30,30,30,30,30,30,30,30 
> ++};
> +diff -rupN AK/Makefile.in AL/Makefile.in
> +--- AK/Makefile.in	2014-03-17 16:10:26.000000000 +0100
> ++++ AL/Makefile.in	2015-02-12 15:42:23.506233505 +0100
> +@@ -71,7 +71,7 @@ apr-config.out: $(APR_CONFIG)
> + 
> + # Create apr_rules.mk suitable for the install tree
> + build/apr_rules.out: build/apr_rules.mk
> +-	sed -e 's,^\(apr_build.*=\).*$$,\1$(installbuilddir),' -e 's,^\(top_build.*=\).*$$,\1$(installbuilddir),' < build/apr_rules.mk > $@
> ++	sed 's,^\(apr_build.*=\).*$$,\1@SYSROOT@$(installbuilddir),' < build/apr_rules.mk > $@
> + 
> + install: $(TARGETS)
> + 	$(APR_MKDIR) $(DESTDIR)$(libdir) $(DESTDIR)$(bindir) $(DESTDIR)$(installbuilddir) \
> +@@ -132,14 +132,9 @@ etags:
> + make_tools_dir:
> + 	$(APR_MKDIR) tools
> + 
> +-OBJECTS_gen_test_char = tools/gen_test_char.lo $(LOCAL_LIBS)
> +-tools/gen_test_char.lo: make_tools_dir
> +-tools/gen_test_char@EXEEXT@: $(OBJECTS_gen_test_char)
> +-	$(LINK_PROG) $(OBJECTS_gen_test_char) $(ALL_LIBS)
> +-
> +-include/private/apr_escape_test_char.h: tools/gen_test_char@EXEEXT@
> +-	$(APR_MKDIR) include/private
> +-	tools/gen_test_char@EXEEXT@ > $@
> ++#include/private/apr_escape_test_char.h: tools/gen_test_char@EXEEXT@
> ++#	$(APR_MKDIR) include/private
> ++#	tools/gen_test_char@EXEEXT@ > $@
> + 
> + LINK_PROG = $(LIBTOOL) $(LTFLAGS) --mode=link $(COMPILE) $(LT_LDFLAGS) \
> + 	    @LT_NO_INSTALL@ $(ALL_LDFLAGS) -o $@
> diff --git a/patches/apr-1.5.1/0002-ac-check-sizeof.patch b/patches/apr-1.5.1/0002-ac-check-sizeof.patch
> new file mode 100644
> index 0000000..225ed54
> --- /dev/null
> +++ b/patches/apr-1.5.1/0002-ac-check-sizeof.patch
> @@ -0,0 +1,22 @@
> +diff -rupN AV/configure.in AU/configure.in
> +--- AV/configure.in	2015-02-18 13:35:04.619502605 +0100
> ++++ AU/configure.in	2014-01-25 16:17:29.000000000 +0100
> +@@ -1693,8 +1693,7 @@ else
> +     socklen_t_value="int"
> + fi
> + 
> +-AC_CHECK_SIZEOF([#include <sys/types.h>], pid_t, 8)
> +-
> ++APR_CHECK_SIZEOF_EXTENDED([#include <sys/types.h>], pid_t, 8)

Why do you replace the standard AC_CHECK_SIZEOF by the non standard
APR_CHECK_SIZEOF_EXTENDED?

Marc

-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |


[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 48 bytes --]

-- 
ptxdist mailing list
ptxdist@pengutronix.de

  reply	other threads:[~2015-03-04 10:04 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-04  9:59 [ptxdist] [PATCHv2 1/2] apr: version updated to 1.5.1 Oliver Graute
2015-03-04  9:59 ` [ptxdist] [PATCHv2 2/2] apr: added patches for apr-1.5.1 Oliver Graute
2015-03-04 10:04   ` Marc Kleine-Budde [this message]
2015-03-04 10:02 ` [ptxdist] [PATCHv2 1/2] apr: version updated to 1.5.1 Marc Kleine-Budde

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=54F6D8C3.9000004@pengutronix.de \
    --to=mkl@pengutronix.de \
    --cc=oliver.graute@neuhaus.de \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox