* [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