* [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
@ 2012-09-19 7:16 Martin Ertsaas
2012-09-19 15:02 ` Saul Wold
2012-09-24 10:11 ` Richard Purdie
0 siblings, 2 replies; 7+ messages in thread
From: Martin Ertsaas @ 2012-09-19 7:16 UTC (permalink / raw)
To: openembedded-core
bash-3.2.48 did not provide the linking from sh to bash, making it unusable.
Moving the license part out of the bash.inc file, and into bash_4.2.bb file makes
us able to use that file also for bash_3.2.48.bb, which makes maintaining both
at the same time a lot easier.
Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
---
meta/recipes-extended/bash/bash.inc | 3 --
meta/recipes-extended/bash/bash_3.2.48.bb | 31 +---------------------------
meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
3 files changed, 7 insertions(+), 33 deletions(-)
diff --git a/meta/recipes-extended/bash/bash.inc b/meta/recipes-extended/bash/bash.inc
index 3684191..ae4b681 100644
--- a/meta/recipes-extended/bash/bash.inc
+++ b/meta/recipes-extended/bash/bash.inc
@@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language interpreter."
HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
SECTION = "base/shell"
-# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
-LICENSE = "GPLv3+"
-LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
DEPENDS = "ncurses bison-native"
inherit autotools gettext update-alternatives
diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb b/meta/recipes-extended/bash/bash_3.2.48.bb
index 509d7a0..828d629 100644
--- a/meta/recipes-extended/bash/bash_3.2.48.bb
+++ b/meta/recipes-extended/bash/bash_3.2.48.bb
@@ -1,12 +1,9 @@
-DESCRIPTION = "An sh-compatible command language interpreter."
-HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
-SECTION = "base/shell"
+require bash.inc
LICENSE = "GPLv2+"
LIC_FILES_CHKSUM = "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
-DEPENDS = "ncurses"
-PR = "r10"
+PR = "r11"
SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001 \
@@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] = "354886097cd95b4def77028f32ee01e2e088d58a98184fede
SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
SRC_URI[sha256sum] = "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
-
-inherit autotools gettext
-
-PARALLEL_MAKE = ""
-
-bindir = "/bin"
-sbindir = "/sbin"
-
-EXTRA_OECONF = "--with-ncurses"
-export CC_FOR_BUILD = "${BUILD_CC}"
-
-export AUTOHEADER = "true"
-
-do_configure_prepend () {
- if [ ! -e acinclude.m4 ]; then
- cat aclocal.m4 > acinclude.m4
- fi
-}
-
-pkg_postinst_${PN} () {
- touch $D${sysconfdir}/shells
- grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >> $D${sysconfdir}/shells
- grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >> $D${sysconfdir}/shells
-}
diff --git a/meta/recipes-extended/bash/bash_4.2.bb b/meta/recipes-extended/bash/bash_4.2.bb
index 384eb62..5a0f015 100644
--- a/meta/recipes-extended/bash/bash_4.2.bb
+++ b/meta/recipes-extended/bash/bash_4.2.bb
@@ -1,6 +1,10 @@
require bash.inc
-PR = "r4"
+# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
+LICENSE = "GPLv3+"
+LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
+
+PR = "r5"
SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001 \
--
1.7.8.6
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-19 7:16 [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2 Martin Ertsaas
@ 2012-09-19 15:02 ` Saul Wold
2012-09-20 6:05 ` Martin Ertsås
2012-09-24 10:11 ` Richard Purdie
1 sibling, 1 reply; 7+ messages in thread
From: Saul Wold @ 2012-09-19 15:02 UTC (permalink / raw)
To: Martin Ertsaas; +Cc: openembedded-core
On 09/19/2012 12:16 AM, Martin Ertsaas wrote:
> bash-3.2.48 did not provide the linking from sh to bash, making it unusable.
> Moving the license part out of the bash.inc file, and into bash_4.2.bb file makes
> us able to use that file also for bash_3.2.48.bb, which makes maintaining both
> at the same time a lot easier.
>
This is good, but I forgot to ask you to include the size change in
here, it's helpful for tracking, I am still considering this change, but
we have constantly adding little change like this adds up to bigger
change.
Is there any other way to do the equivalent of what you are attempting
to do in the other patch without adding size to busy box?
Thanks
Sau!
> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
> ---
> meta/recipes-extended/bash/bash.inc | 3 --
> meta/recipes-extended/bash/bash_3.2.48.bb | 31 +---------------------------
> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
> 3 files changed, 7 insertions(+), 33 deletions(-)
>
> diff --git a/meta/recipes-extended/bash/bash.inc b/meta/recipes-extended/bash/bash.inc
> index 3684191..ae4b681 100644
> --- a/meta/recipes-extended/bash/bash.inc
> +++ b/meta/recipes-extended/bash/bash.inc
> @@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language interpreter."
> HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
> SECTION = "base/shell"
>
> -# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
> -LICENSE = "GPLv3+"
> -LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> DEPENDS = "ncurses bison-native"
>
> inherit autotools gettext update-alternatives
> diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb b/meta/recipes-extended/bash/bash_3.2.48.bb
> index 509d7a0..828d629 100644
> --- a/meta/recipes-extended/bash/bash_3.2.48.bb
> +++ b/meta/recipes-extended/bash/bash_3.2.48.bb
> @@ -1,12 +1,9 @@
> -DESCRIPTION = "An sh-compatible command language interpreter."
> -HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
> -SECTION = "base/shell"
> +require bash.inc
>
> LICENSE = "GPLv2+"
> LIC_FILES_CHKSUM = "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
> -DEPENDS = "ncurses"
>
> -PR = "r10"
> +PR = "r11"
>
> SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
> ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001 \
> @@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] = "354886097cd95b4def77028f32ee01e2e088d58a98184fede
>
> SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
> SRC_URI[sha256sum] = "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
> -
> -inherit autotools gettext
> -
> -PARALLEL_MAKE = ""
> -
> -bindir = "/bin"
> -sbindir = "/sbin"
> -
> -EXTRA_OECONF = "--with-ncurses"
> -export CC_FOR_BUILD = "${BUILD_CC}"
> -
> -export AUTOHEADER = "true"
> -
> -do_configure_prepend () {
> - if [ ! -e acinclude.m4 ]; then
> - cat aclocal.m4 > acinclude.m4
> - fi
> -}
> -
> -pkg_postinst_${PN} () {
> - touch $D${sysconfdir}/shells
> - grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >> $D${sysconfdir}/shells
> - grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >> $D${sysconfdir}/shells
> -}
> diff --git a/meta/recipes-extended/bash/bash_4.2.bb b/meta/recipes-extended/bash/bash_4.2.bb
> index 384eb62..5a0f015 100644
> --- a/meta/recipes-extended/bash/bash_4.2.bb
> +++ b/meta/recipes-extended/bash/bash_4.2.bb
> @@ -1,6 +1,10 @@
> require bash.inc
>
> -PR = "r4"
> +# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
> +LICENSE = "GPLv3+"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
> +
> +PR = "r5"
>
> SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
> ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001 \
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-19 15:02 ` Saul Wold
@ 2012-09-20 6:05 ` Martin Ertsås
2012-09-20 6:48 ` Saul Wold
0 siblings, 1 reply; 7+ messages in thread
From: Martin Ertsås @ 2012-09-20 6:05 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On 09/19/12 17:02, Saul Wold wrote:
> On 09/19/2012 12:16 AM, Martin Ertsaas wrote:
>> bash-3.2.48 did not provide the linking from sh to bash, making it
>> unusable.
>> Moving the license part out of the bash.inc file, and into
>> bash_4.2.bb file makes
>> us able to use that file also for bash_3.2.48.bb, which makes
>> maintaining both
>> at the same time a lot easier.
>>
> This is good, but I forgot to ask you to include the size change in
> here, it's helpful for tracking, I am still considering this change,
> but we have constantly adding little change like this adds up to bigger
> change.
>
> Is there any other way to do the equivalent of what you are attempting
> to do in the other patch without adding size to busy box?
>
> Thanks
>
> Sau!
What size change is that? I'm not sure what you are referring to here,
sorry. As far as I see I'm not adding any size to busybox either, but I
might be wrong as I don't know the internals of busybox. If you could
elaborate a bit more, I can see what I can do about your concerns.
- Martin
>
>> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
>> ---
>> meta/recipes-extended/bash/bash.inc | 3 --
>> meta/recipes-extended/bash/bash_3.2.48.bb | 31
>> +---------------------------
>> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
>> 3 files changed, 7 insertions(+), 33 deletions(-)
>>
>> diff --git a/meta/recipes-extended/bash/bash.inc
>> b/meta/recipes-extended/bash/bash.inc
>> index 3684191..ae4b681 100644
>> --- a/meta/recipes-extended/bash/bash.inc
>> +++ b/meta/recipes-extended/bash/bash.inc
>> @@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language
>> interpreter."
>> HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>> SECTION = "base/shell"
>>
>> -# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>> -LICENSE = "GPLv3+"
>> -LIC_FILES_CHKSUM =
>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>> DEPENDS = "ncurses bison-native"
>>
>> inherit autotools gettext update-alternatives
>> diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb
>> b/meta/recipes-extended/bash/bash_3.2.48.bb
>> index 509d7a0..828d629 100644
>> --- a/meta/recipes-extended/bash/bash_3.2.48.bb
>> +++ b/meta/recipes-extended/bash/bash_3.2.48.bb
>> @@ -1,12 +1,9 @@
>> -DESCRIPTION = "An sh-compatible command language interpreter."
>> -HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>> -SECTION = "base/shell"
>> +require bash.inc
>>
>> LICENSE = "GPLv2+"
>> LIC_FILES_CHKSUM =
>> "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
>> -DEPENDS = "ncurses"
>>
>> -PR = "r10"
>> +PR = "r11"
>>
>> SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
>>
>> ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001
>> \
>> @@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] =
>> "354886097cd95b4def77028f32ee01e2e088d58a98184fede
>>
>> SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
>> SRC_URI[sha256sum] =
>> "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
>> -
>> -inherit autotools gettext
>> -
>> -PARALLEL_MAKE = ""
>> -
>> -bindir = "/bin"
>> -sbindir = "/sbin"
>> -
>> -EXTRA_OECONF = "--with-ncurses"
>> -export CC_FOR_BUILD = "${BUILD_CC}"
>> -
>> -export AUTOHEADER = "true"
>> -
>> -do_configure_prepend () {
>> - if [ ! -e acinclude.m4 ]; then
>> - cat aclocal.m4 > acinclude.m4
>> - fi
>> -}
>> -
>> -pkg_postinst_${PN} () {
>> - touch $D${sysconfdir}/shells
>> - grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >>
>> $D${sysconfdir}/shells
>> - grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >>
>> $D${sysconfdir}/shells
>> -}
>> diff --git a/meta/recipes-extended/bash/bash_4.2.bb
>> b/meta/recipes-extended/bash/bash_4.2.bb
>> index 384eb62..5a0f015 100644
>> --- a/meta/recipes-extended/bash/bash_4.2.bb
>> +++ b/meta/recipes-extended/bash/bash_4.2.bb
>> @@ -1,6 +1,10 @@
>> require bash.inc
>>
>> -PR = "r4"
>> +# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>> +LICENSE = "GPLv3+"
>> +LIC_FILES_CHKSUM =
>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>> +
>> +PR = "r5"
>>
>> SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
>>
>> ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001
>> \
>>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-20 6:05 ` Martin Ertsås
@ 2012-09-20 6:48 ` Saul Wold
2012-09-20 7:14 ` Martin Ertsås
2012-09-24 6:18 ` Martin Ertsås
0 siblings, 2 replies; 7+ messages in thread
From: Saul Wold @ 2012-09-20 6:48 UTC (permalink / raw)
To: Martin Ertsås; +Cc: openembedded-core
On 09/19/2012 11:05 PM, Martin Ertsås wrote:
> On 09/19/12 17:02, Saul Wold wrote:
>> On 09/19/2012 12:16 AM, Martin Ertsaas wrote:
>>> bash-3.2.48 did not provide the linking from sh to bash, making it
>>> unusable.
>>> Moving the license part out of the bash.inc file, and into
>>> bash_4.2.bb file makes
>>> us able to use that file also for bash_3.2.48.bb, which makes
>>> maintaining both
>>> at the same time a lot easier.
>>>
>> This is good, but I forgot to ask you to include the size change in
>> here, it's helpful for tracking, I am still considering this change,
>> but we have constantly adding little change like this adds up to bigger
>> change.
>>
>> Is there any other way to do the equivalent of what you are attempting
>> to do in the other patch without adding size to busy box?
>>
>> Thanks
>>
>> Sau!
> What size change is that? I'm not sure what you are referring to here,
> sorry. As far as I see I'm not adding any size to busybox either, but I
> might be wrong as I don't know the internals of busybox. If you could
> elaborate a bit more, I can see what I can do about your concerns.
>
Sorry, today's been a bit crazy! I confused a change to busybox with
your change to bash! My bad, ignore me.
I will revisit this tomorrow.
Sau!
> - Martin
>>
>>> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
>>> ---
>>> meta/recipes-extended/bash/bash.inc | 3 --
>>> meta/recipes-extended/bash/bash_3.2.48.bb | 31
>>> +---------------------------
>>> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
>>> 3 files changed, 7 insertions(+), 33 deletions(-)
>>>
>>> diff --git a/meta/recipes-extended/bash/bash.inc
>>> b/meta/recipes-extended/bash/bash.inc
>>> index 3684191..ae4b681 100644
>>> --- a/meta/recipes-extended/bash/bash.inc
>>> +++ b/meta/recipes-extended/bash/bash.inc
>>> @@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language
>>> interpreter."
>>> HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>> SECTION = "base/shell"
>>>
>>> -# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>> -LICENSE = "GPLv3+"
>>> -LIC_FILES_CHKSUM =
>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>> DEPENDS = "ncurses bison-native"
>>>
>>> inherit autotools gettext update-alternatives
>>> diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb
>>> b/meta/recipes-extended/bash/bash_3.2.48.bb
>>> index 509d7a0..828d629 100644
>>> --- a/meta/recipes-extended/bash/bash_3.2.48.bb
>>> +++ b/meta/recipes-extended/bash/bash_3.2.48.bb
>>> @@ -1,12 +1,9 @@
>>> -DESCRIPTION = "An sh-compatible command language interpreter."
>>> -HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>> -SECTION = "base/shell"
>>> +require bash.inc
>>>
>>> LICENSE = "GPLv2+"
>>> LIC_FILES_CHKSUM =
>>> "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
>>> -DEPENDS = "ncurses"
>>>
>>> -PR = "r10"
>>> +PR = "r11"
>>>
>>> SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
>>>
>>> ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001
>>> \
>>> @@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] =
>>> "354886097cd95b4def77028f32ee01e2e088d58a98184fede
>>>
>>> SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
>>> SRC_URI[sha256sum] =
>>> "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
>>> -
>>> -inherit autotools gettext
>>> -
>>> -PARALLEL_MAKE = ""
>>> -
>>> -bindir = "/bin"
>>> -sbindir = "/sbin"
>>> -
>>> -EXTRA_OECONF = "--with-ncurses"
>>> -export CC_FOR_BUILD = "${BUILD_CC}"
>>> -
>>> -export AUTOHEADER = "true"
>>> -
>>> -do_configure_prepend () {
>>> - if [ ! -e acinclude.m4 ]; then
>>> - cat aclocal.m4 > acinclude.m4
>>> - fi
>>> -}
>>> -
>>> -pkg_postinst_${PN} () {
>>> - touch $D${sysconfdir}/shells
>>> - grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >>
>>> $D${sysconfdir}/shells
>>> - grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >>
>>> $D${sysconfdir}/shells
>>> -}
>>> diff --git a/meta/recipes-extended/bash/bash_4.2.bb
>>> b/meta/recipes-extended/bash/bash_4.2.bb
>>> index 384eb62..5a0f015 100644
>>> --- a/meta/recipes-extended/bash/bash_4.2.bb
>>> +++ b/meta/recipes-extended/bash/bash_4.2.bb
>>> @@ -1,6 +1,10 @@
>>> require bash.inc
>>>
>>> -PR = "r4"
>>> +# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>> +LICENSE = "GPLv3+"
>>> +LIC_FILES_CHKSUM =
>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>> +
>>> +PR = "r5"
>>>
>>> SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
>>>
>>> ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001
>>> \
>>>
>
>
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-20 6:48 ` Saul Wold
@ 2012-09-20 7:14 ` Martin Ertsås
2012-09-24 6:18 ` Martin Ertsås
1 sibling, 0 replies; 7+ messages in thread
From: Martin Ertsås @ 2012-09-20 7:14 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
On 09/20/12 08:48, Saul Wold wrote:
> On 09/19/2012 11:05 PM, Martin Ertsås wrote:
>> On 09/19/12 17:02, Saul Wold wrote:
>>> On 09/19/2012 12:16 AM, Martin Ertsaas wrote:
>>>> bash-3.2.48 did not provide the linking from sh to bash, making it
>>>> unusable.
>>>> Moving the license part out of the bash.inc file, and into
>>>> bash_4.2.bb file makes
>>>> us able to use that file also for bash_3.2.48.bb, which makes
>>>> maintaining both
>>>> at the same time a lot easier.
>>>>
>>> This is good, but I forgot to ask you to include the size change in
>>> here, it's helpful for tracking, I am still considering this change,
>>> but we have constantly adding little change like this adds up to bigger
>>> change.
>>>
>>> Is there any other way to do the equivalent of what you are attempting
>>> to do in the other patch without adding size to busy box?
>>>
>>> Thanks
>>>
>>> Sau!
>> What size change is that? I'm not sure what you are referring to here,
>> sorry. As far as I see I'm not adding any size to busybox either, but I
>> might be wrong as I don't know the internals of busybox. If you could
>> elaborate a bit more, I can see what I can do about your concerns.
>>
> Sorry, today's been a bit crazy! I confused a change to busybox with
> your change to bash! My bad, ignore me.
>
> I will revisit this tomorrow.
>
> Sau!
Ok. No wonder I was a bit confused then.
- Martin
>
>
>
>
>> - Martin
>>>
>>>> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
>>>> ---
>>>> meta/recipes-extended/bash/bash.inc | 3 --
>>>> meta/recipes-extended/bash/bash_3.2.48.bb | 31
>>>> +---------------------------
>>>> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
>>>> 3 files changed, 7 insertions(+), 33 deletions(-)
>>>>
>>>> diff --git a/meta/recipes-extended/bash/bash.inc
>>>> b/meta/recipes-extended/bash/bash.inc
>>>> index 3684191..ae4b681 100644
>>>> --- a/meta/recipes-extended/bash/bash.inc
>>>> +++ b/meta/recipes-extended/bash/bash.inc
>>>> @@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language
>>>> interpreter."
>>>> HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>>> SECTION = "base/shell"
>>>>
>>>> -# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>>> -LICENSE = "GPLv3+"
>>>> -LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>>> DEPENDS = "ncurses bison-native"
>>>>
>>>> inherit autotools gettext update-alternatives
>>>> diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> b/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> index 509d7a0..828d629 100644
>>>> --- a/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> +++ b/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> @@ -1,12 +1,9 @@
>>>> -DESCRIPTION = "An sh-compatible command language interpreter."
>>>> -HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>>> -SECTION = "base/shell"
>>>> +require bash.inc
>>>>
>>>> LICENSE = "GPLv2+"
>>>> LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
>>>> -DEPENDS = "ncurses"
>>>>
>>>> -PR = "r10"
>>>> +PR = "r11"
>>>>
>>>> SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
>>>>
>>>> ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001
>>>>
>>>> \
>>>> @@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] =
>>>> "354886097cd95b4def77028f32ee01e2e088d58a98184fede
>>>>
>>>> SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
>>>> SRC_URI[sha256sum] =
>>>> "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
>>>> -
>>>> -inherit autotools gettext
>>>> -
>>>> -PARALLEL_MAKE = ""
>>>> -
>>>> -bindir = "/bin"
>>>> -sbindir = "/sbin"
>>>> -
>>>> -EXTRA_OECONF = "--with-ncurses"
>>>> -export CC_FOR_BUILD = "${BUILD_CC}"
>>>> -
>>>> -export AUTOHEADER = "true"
>>>> -
>>>> -do_configure_prepend () {
>>>> - if [ ! -e acinclude.m4 ]; then
>>>> - cat aclocal.m4 > acinclude.m4
>>>> - fi
>>>> -}
>>>> -
>>>> -pkg_postinst_${PN} () {
>>>> - touch $D${sysconfdir}/shells
>>>> - grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >>
>>>> $D${sysconfdir}/shells
>>>> - grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >>
>>>> $D${sysconfdir}/shells
>>>> -}
>>>> diff --git a/meta/recipes-extended/bash/bash_4.2.bb
>>>> b/meta/recipes-extended/bash/bash_4.2.bb
>>>> index 384eb62..5a0f015 100644
>>>> --- a/meta/recipes-extended/bash/bash_4.2.bb
>>>> +++ b/meta/recipes-extended/bash/bash_4.2.bb
>>>> @@ -1,6 +1,10 @@
>>>> require bash.inc
>>>>
>>>> -PR = "r4"
>>>> +# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>>> +LICENSE = "GPLv3+"
>>>> +LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>>> +
>>>> +PR = "r5"
>>>>
>>>> SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
>>>>
>>>> ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001
>>>>
>>>> \
>>>>
>>
>>
>>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-20 6:48 ` Saul Wold
2012-09-20 7:14 ` Martin Ertsås
@ 2012-09-24 6:18 ` Martin Ertsås
1 sibling, 0 replies; 7+ messages in thread
From: Martin Ertsås @ 2012-09-24 6:18 UTC (permalink / raw)
To: Saul Wold; +Cc: openembedded-core
What is the status of this patch? Are there more changes needed for this
to make it in master, or are we just waiting until after 1.3?
- Martin
On 09/20/12 08:48, Saul Wold wrote:
> On 09/19/2012 11:05 PM, Martin Ertsås wrote:
>> On 09/19/12 17:02, Saul Wold wrote:
>>> On 09/19/2012 12:16 AM, Martin Ertsaas wrote:
>>>> bash-3.2.48 did not provide the linking from sh to bash, making it
>>>> unusable.
>>>> Moving the license part out of the bash.inc file, and into
>>>> bash_4.2.bb file makes
>>>> us able to use that file also for bash_3.2.48.bb, which makes
>>>> maintaining both
>>>> at the same time a lot easier.
>>>>
>>> This is good, but I forgot to ask you to include the size change in
>>> here, it's helpful for tracking, I am still considering this change,
>>> but we have constantly adding little change like this adds up to bigger
>>> change.
>>>
>>> Is there any other way to do the equivalent of what you are attempting
>>> to do in the other patch without adding size to busy box?
>>>
>>> Thanks
>>>
>>> Sau!
>> What size change is that? I'm not sure what you are referring to here,
>> sorry. As far as I see I'm not adding any size to busybox either, but I
>> might be wrong as I don't know the internals of busybox. If you could
>> elaborate a bit more, I can see what I can do about your concerns.
>>
> Sorry, today's been a bit crazy! I confused a change to busybox with
> your change to bash! My bad, ignore me.
>
> I will revisit this tomorrow.
>
> Sau!
>
>
>
>
>> - Martin
>>>
>>>> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
>>>> ---
>>>> meta/recipes-extended/bash/bash.inc | 3 --
>>>> meta/recipes-extended/bash/bash_3.2.48.bb | 31
>>>> +---------------------------
>>>> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
>>>> 3 files changed, 7 insertions(+), 33 deletions(-)
>>>>
>>>> diff --git a/meta/recipes-extended/bash/bash.inc
>>>> b/meta/recipes-extended/bash/bash.inc
>>>> index 3684191..ae4b681 100644
>>>> --- a/meta/recipes-extended/bash/bash.inc
>>>> +++ b/meta/recipes-extended/bash/bash.inc
>>>> @@ -2,9 +2,6 @@ DESCRIPTION = "An sh-compatible command language
>>>> interpreter."
>>>> HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>>> SECTION = "base/shell"
>>>>
>>>> -# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>>> -LICENSE = "GPLv3+"
>>>> -LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>>> DEPENDS = "ncurses bison-native"
>>>>
>>>> inherit autotools gettext update-alternatives
>>>> diff --git a/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> b/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> index 509d7a0..828d629 100644
>>>> --- a/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> +++ b/meta/recipes-extended/bash/bash_3.2.48.bb
>>>> @@ -1,12 +1,9 @@
>>>> -DESCRIPTION = "An sh-compatible command language interpreter."
>>>> -HOMEPAGE = "http://cnswww.cns.cwru.edu/~chet/bash/bashtop.html"
>>>> -SECTION = "base/shell"
>>>> +require bash.inc
>>>>
>>>> LICENSE = "GPLv2+"
>>>> LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=fd5d9bcabd8ed5a54a01ce8d183d592a"
>>>> -DEPENDS = "ncurses"
>>>>
>>>> -PR = "r10"
>>>> +PR = "r11"
>>>>
>>>> SRC_URI = "${GNU_MIRROR}/bash/bash-${PV}.tar.gz;name=tarball \
>>>>
>>>> ${GNU_MIRROR}/bash/bash-3.2-patches/bash32-049;apply=yes;striplevel=0;name=patch001
>>>>
>>>> \
>>>> @@ -24,27 +21,3 @@ SRC_URI[patch003.sha256sum] =
>>>> "354886097cd95b4def77028f32ee01e2e088d58a98184fede
>>>>
>>>> SRC_URI[md5sum] = "338dcf975a93640bb3eaa843ca42e3f8"
>>>> SRC_URI[sha256sum] =
>>>> "128d281bd5682ba5f6953122915da71976357d7a76490d266c9173b1d0426348"
>>>> -
>>>> -inherit autotools gettext
>>>> -
>>>> -PARALLEL_MAKE = ""
>>>> -
>>>> -bindir = "/bin"
>>>> -sbindir = "/sbin"
>>>> -
>>>> -EXTRA_OECONF = "--with-ncurses"
>>>> -export CC_FOR_BUILD = "${BUILD_CC}"
>>>> -
>>>> -export AUTOHEADER = "true"
>>>> -
>>>> -do_configure_prepend () {
>>>> - if [ ! -e acinclude.m4 ]; then
>>>> - cat aclocal.m4 > acinclude.m4
>>>> - fi
>>>> -}
>>>> -
>>>> -pkg_postinst_${PN} () {
>>>> - touch $D${sysconfdir}/shells
>>>> - grep -q "bin/bash" $D${sysconfdir}/shells || echo /bin/bash >>
>>>> $D${sysconfdir}/shells
>>>> - grep -q "bin/sh" $D${sysconfdir}/shells || echo /bin/sh >>
>>>> $D${sysconfdir}/shells
>>>> -}
>>>> diff --git a/meta/recipes-extended/bash/bash_4.2.bb
>>>> b/meta/recipes-extended/bash/bash_4.2.bb
>>>> index 384eb62..5a0f015 100644
>>>> --- a/meta/recipes-extended/bash/bash_4.2.bb
>>>> +++ b/meta/recipes-extended/bash/bash_4.2.bb
>>>> @@ -1,6 +1,10 @@
>>>> require bash.inc
>>>>
>>>> -PR = "r4"
>>>> +# GPLv2+ (< 4.0), GPLv3+ (>= 4.0)
>>>> +LICENSE = "GPLv3+"
>>>> +LIC_FILES_CHKSUM =
>>>> "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
>>>> +
>>>> +PR = "r5"
>>>>
>>>> SRC_URI = "${GNU_MIRROR}/bash/${BPN}-${PV}.tar.gz;name=tarball \
>>>>
>>>> ${GNU_MIRROR}/bash/bash-4.2-patches/bash42-001;apply=yes;striplevel=0;name=patch001
>>>>
>>>> \
>>>>
>>
>>
>>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2.
2012-09-19 7:16 [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2 Martin Ertsaas
2012-09-19 15:02 ` Saul Wold
@ 2012-09-24 10:11 ` Richard Purdie
1 sibling, 0 replies; 7+ messages in thread
From: Richard Purdie @ 2012-09-24 10:11 UTC (permalink / raw)
To: Martin Ertsaas; +Cc: openembedded-core
On Wed, 2012-09-19 at 09:16 +0200, Martin Ertsaas wrote:
> bash-3.2.48 did not provide the linking from sh to bash, making it unusable.
> Moving the license part out of the bash.inc file, and into bash_4.2.bb file makes
> us able to use that file also for bash_3.2.48.bb, which makes maintaining both
> at the same time a lot easier.
>
> Signed-off-by: Martin Ertsaas <mertsas@cisco.com>
> ---
> meta/recipes-extended/bash/bash.inc | 3 --
> meta/recipes-extended/bash/bash_3.2.48.bb | 31 +---------------------------
> meta/recipes-extended/bash/bash_4.2.bb | 6 ++++-
> 3 files changed, 7 insertions(+), 33 deletions(-)
Merged to master, thanks.
Richard
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-09-24 10:24 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-09-19 7:16 [PATCH v2] bash: Make it possible to run bash 3.2.48 instead of 4.2 Martin Ertsaas
2012-09-19 15:02 ` Saul Wold
2012-09-20 6:05 ` Martin Ertsås
2012-09-20 6:48 ` Saul Wold
2012-09-20 7:14 ` Martin Ertsås
2012-09-24 6:18 ` Martin Ertsås
2012-09-24 10:11 ` Richard Purdie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox