Re: [ptxdist] [PATCH 1/2] valgrind: Also patch configure.in, not only configure

2011-08-08 Thread Robert Schwebel
On Mon, Aug 08, 2011 at 09:29:57AM +0200, Michael Olbrich wrote:
> On Sun, Aug 07, 2011 at 04:17:30PM +0200, Bernhard Walle wrote:
> > While it's a good idea to avoid running autoreconf just because two
> > letters that have changed, it makes sense to add the change in the
> > configure.in file, so that autoreconf doesn't break the build.
>
> I've added a patch to _really_ fix the kernel version. This adds a cache
> variable that can be set, so autoreconf is needed anyways.
> All patches only change configure.in and it should work with 3.x for build
> system and target.

The patch series should really be pushed upstream.

rsc
-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


Re: [ptxdist] [PATCH 1/2] valgrind: Also patch configure.in, not only configure

2011-08-08 Thread Michael Olbrich
On Sun, Aug 07, 2011 at 04:17:30PM +0200, Bernhard Walle wrote:
> While it's a good idea to avoid running autoreconf just because two
> letters that have changed, it makes sense to add the change in the
> configure.in file, so that autoreconf doesn't break the build.

I've added a patch to _really_ fix the kernel version. This adds a cache
variable that can be set, so autoreconf is needed anyways.
All patches only change configure.in and it should work with 3.x for build
system and target.

Michael

> Signed-off-by: Bernhard Walle 
> ---
>  ...ure-allow-all-arm-don-t-restrict-to-armv7.patch |   21 +--
>  1 files changed, 14 insertions(+), 7 deletions(-)
> 
> diff --git 
> a/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
>  
> b/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
> index b2fe6c2..63447b7 100644
> --- 
> a/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
> +++ 
> b/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
> @@ -4,11 +4,10 @@ Subject: [PATCH] configure: allow all arm, don't restrict 
> to armv7
>  
>  Signed-off-by: Marc Kleine-Budde 
>  ---
> - configure |2 +-
> - 1 files changed, 1 insertions(+), 1 deletions(-)
> + configure|2 +-
> + configure.in |2 +-
> + 2 files changed, 2 insertions(+), 2 deletions(-)
>  
> -diff --git a/configure b/configure
> -index 699137b..1179122 100755
>  --- a/configure
>  +++ b/configure
>  @@ -5262,7 +5262,7 @@ $as_echo "ok (${host_cpu})" >&6; }
> @@ -20,6 +19,14 @@ index 699137b..1179122 100755
>   { $as_echo "$as_me:${as_lineno-$LINENO}: result: ok (${host_cpu})" >&5
>   $as_echo "ok (${host_cpu})" >&6; }
>   ARCH_MAX="arm"
> --- 
> -1.7.4.1
> -
> +--- a/configure.in
>  b/configure.in
> +@@ -158,7 +158,7 @@ case "${host_cpu}" in
> + esac
> + ;;
> + 
> +- armv7*)
> ++ arm*)
> + AC_MSG_RESULT([ok (${host_cpu})])
> + ARCH_MAX="arm"
> + ;;
> -- 
> 1.7.6
> 
> 
> -- 
> ptxdist mailing list
> ptxdist@pengutronix.de
> 

-- 
Pengutronix e.K.   | |
Industrial Linux Solutions | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0|
Amtsgericht Hildesheim, HRA 2686   | Fax:   +49-5121-206917- |

-- 
ptxdist mailing list
ptxdist@pengutronix.de


[ptxdist] [PATCH 1/2] valgrind: Also patch configure.in, not only configure

2011-08-07 Thread Bernhard Walle
While it's a good idea to avoid running autoreconf just because two
letters that have changed, it makes sense to add the change in the
configure.in file, so that autoreconf doesn't break the build.

Signed-off-by: Bernhard Walle 
---
 ...ure-allow-all-arm-don-t-restrict-to-armv7.patch |   21 +--
 1 files changed, 14 insertions(+), 7 deletions(-)

diff --git 
a/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
 
b/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
index b2fe6c2..63447b7 100644
--- 
a/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
+++ 
b/patches/valgrind-3.6.1/0001-configure-allow-all-arm-don-t-restrict-to-armv7.patch
@@ -4,11 +4,10 @@ Subject: [PATCH] configure: allow all arm, don't restrict to 
armv7
 
 Signed-off-by: Marc Kleine-Budde 
 ---
- configure |2 +-
- 1 files changed, 1 insertions(+), 1 deletions(-)
+ configure|2 +-
+ configure.in |2 +-
+ 2 files changed, 2 insertions(+), 2 deletions(-)
 
-diff --git a/configure b/configure
-index 699137b..1179122 100755
 --- a/configure
 +++ b/configure
 @@ -5262,7 +5262,7 @@ $as_echo "ok (${host_cpu})" >&6; }
@@ -20,6 +19,14 @@ index 699137b..1179122 100755
{ $as_echo "$as_me:${as_lineno-$LINENO}: result: ok (${host_cpu})" >&5
  $as_echo "ok (${host_cpu})" >&6; }
ARCH_MAX="arm"
--- 
-1.7.4.1
-
+--- a/configure.in
 b/configure.in
+@@ -158,7 +158,7 @@ case "${host_cpu}" in
+ esac
+ ;;
+ 
+- armv7*)
++ arm*)
+   AC_MSG_RESULT([ok (${host_cpu})])
+   ARCH_MAX="arm"
+   ;;
-- 
1.7.6


-- 
ptxdist mailing list
ptxdist@pengutronix.de