Patchwork icedtea6-native: fix problem on build hosts with linux kernel 3.x

login
register
mail settings
Submitter Steffen Sledz
Date Dec. 9, 2011, 5 p.m.
Message ID <1323450027-31964-1-git-send-email-sledz@dresearch-fe.de>
Download mbox | patch
Permalink /patch/16565/
State Accepted
Headers show

Comments

Steffen Sledz - Dec. 9, 2011, 5 p.m.
From: Steffen Sledz <sledz@dresearch.de>

see http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7072341

Signed-off-by: Steffen Sledz <sledz@dresearch.de>
---
 recipes/icedtea/icedtea6-native.inc |    9 ++++++++-
 1 files changed, 8 insertions(+), 1 deletions(-)
Khem Raj - Dec. 9, 2011, 7:59 p.m.
On Fri, Dec 9, 2011 at 9:00 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote:
>        done
> +
> +       for F in openjdk/hotspot/make/linux/Makefile openjdk-ecj/hotspot/make/linux/Makefile ;
> +       do
> +               sed -i \
> +                       -e"s|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 2.7%|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 3%|" \
> +                       $F
> +       done
>  }

why drop the 'dot' may be using 3. is better ?
Steffen Sledz - Dec. 10, 2011, 10:29 a.m.
Am 09.12.2011 20:59, schrieb Khem Raj:
> On Fri, Dec 9, 2011 at 9:00 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote:
>>        done
>> +
>> +       for F in openjdk/hotspot/make/linux/Makefile openjdk-ecj/hotspot/make/linux/Makefile ;
>> +       do
>> +               sed -i \
>> +                       -e"s|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 2.7%|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 3%|" \
>> +                       $F
>> +       done
>>  }
> 
> why drop the 'dot' may be using 3. is better ?

Just following the scheme, e.g. 2.6% is without a dot too.

Steffen
Steffen Sledz - Dec. 13, 2011, 8:12 a.m.
On 10.12.2011 11:29, Steffen Sledz wrote:
> Am 09.12.2011 20:59, schrieb Khem Raj:
>> On Fri, Dec 9, 2011 at 9:00 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote:
>>>        done
>>> +
>>> +       for F in openjdk/hotspot/make/linux/Makefile openjdk-ecj/hotspot/make/linux/Makefile ;
>>> +       do
>>> +               sed -i \
>>> +                       -e"s|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 2.7%|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 3%|" \
>>> +                       $F
>>> +       done
>>>  }
>>
>> why drop the 'dot' may be using 3. is better ?
> 
> Just following the scheme, e.g. 2.6% is without a dot too.

Ping!
Steffen Sledz - Dec. 14, 2011, 11:23 a.m.
On 13.12.2011 09:12, Steffen Sledz wrote:
> On 10.12.2011 11:29, Steffen Sledz wrote:
>> Am 09.12.2011 20:59, schrieb Khem Raj:
>>> On Fri, Dec 9, 2011 at 9:00 AM, Steffen Sledz <sledz@dresearch-fe.de> wrote:
>>>>        done
>>>> +
>>>> +       for F in openjdk/hotspot/make/linux/Makefile openjdk-ecj/hotspot/make/linux/Makefile ;
>>>> +       do
>>>> +               sed -i \
>>>> +                       -e"s|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 2.7%|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 3%|" \
>>>> +                       $F
>>>> +       done
>>>>  }
>>>
>>> why drop the 'dot' may be using 3. is better ?
>>
>> Just following the scheme, e.g. 2.6% is without a dot too.

pushed to oe-classic master

Patch

diff --git a/recipes/icedtea/icedtea6-native.inc b/recipes/icedtea/icedtea6-native.inc
index 06da7c0..91d6075 100644
--- a/recipes/icedtea/icedtea6-native.inc
+++ b/recipes/icedtea/icedtea6-native.inc
@@ -1,7 +1,7 @@ 
 DESCRIPTION = "Harness to build the source code from OpenJDK using Free Software build tools"
 HOMEPAGE = "http://icedtea.classpath.org"
 LICENSE  = "GPL with Classpath Exception"
-INC_PR = "r4"
+INC_PR = "r5"
 
 DEPENDS = "virtual/javac-native classpath-tools-native \
 	   virtual/java-native classpath-native \
@@ -108,6 +108,13 @@  do_configure_append() {
 			-e"s|g++\$(GCC_SUFFIX)|${CXX}|" \
 			$F
 	done
+
+	for F in openjdk/hotspot/make/linux/Makefile openjdk-ecj/hotspot/make/linux/Makefile ;
+	do
+		sed -i \
+			-e"s|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 2.7%|SUPPORTED_OS_VERSION = 2.4% 2.5% 2.6% 3%|" \
+			$F
+	done
 }
 
 EXTRA_OEMAKE = ' \