* [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage
@ 2014-11-04 13:40 Koen Kooi
2014-11-04 13:40 ` [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes Koen Kooi
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Koen Kooi @ 2014-11-04 13:40 UTC (permalink / raw)
To: openembedded-core; +Cc: Koen Kooi
LSB can't cope with a "make clean" so disable that
newly enabled functionality.
Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/meta/recipes-extended/lsb/lsb_4.1.bb b/meta/recipes-extended/lsb/lsb_4.1.bb
index d265731..34a70d0 100644
--- a/meta/recipes-extended/lsb/lsb_4.1.bb
+++ b/meta/recipes-extended/lsb/lsb_4.1.bb
@@ -21,6 +21,12 @@ SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172"
S = "${WORKDIR}/lsb-release-1.4"
+# 'make clean' isn't supported
+
+do_configure() {
+ :
+}
+
do_install(){
oe_runmake install prefix=${D} mandir=${D}/${datadir}/man/ DESTDIR=${D}
--
1.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes
2014-11-04 13:40 [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Koen Kooi
@ 2014-11-04 13:40 ` Koen Kooi
2014-11-04 14:52 ` Mark Hatle
2014-11-04 13:40 ` [PATCH 3/3] lsb 4.1: fix indentation Koen Kooi
2014-11-05 10:48 ` [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Richard Purdie
2 siblings, 1 reply; 5+ messages in thread
From: Koen Kooi @ 2014-11-04 13:40 UTC (permalink / raw)
To: openembedded-core; +Cc: Koen Kooi
Also collapse do_install and do_install_append.
Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/meta/recipes-extended/lsb/lsb_4.1.bb b/meta/recipes-extended/lsb/lsb_4.1.bb
index 34a70d0..897520a 100644
--- a/meta/recipes-extended/lsb/lsb_4.1.bb
+++ b/meta/recipes-extended/lsb/lsb_4.1.bb
@@ -72,9 +72,7 @@ do_install(){
touch ${D}${sysconfdir}/lsb-release.d/graphics-${PV}-ppc64
touch ${D}${sysconfdir}/lsb-release.d/desktop-${PV}-ppc64
fi
-}
-do_install_append(){
install -d ${D}${sysconfdir}/core-lsb
install -d ${D}/${baselib}/lsb
for i in lsb_killproc lsb_log_message lsb_pidofproc lsb_start_daemon
@@ -84,9 +82,9 @@ do_install_append(){
install -m 0755 ${WORKDIR}/init-functions ${D}/${baselib}/lsb
# creat links for LSB test
- install -d ${D}/usr/lib/lsb
- ln -sf ${sbindir}/chkconfig ${D}/usr/lib/lsb/install_initd
- ln -sf ${sbindir}/chkconfig ${D}/usr/lib/lsb/remove_initd
+ install -d ${D}${libdir}/lsb
+ ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/install_initd
+ ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/remove_initd
install -d ${D}/${libdir}
ln -sf ${sbindir}/sendmail ${D}/${libdir}/sendmail
@@ -120,8 +118,9 @@ do_install_append(){
ln -sf ld.so.1 ld-lsb-ppc32.so.3
fi
}
+
FILES_${PN} += "/lib64 \
- /usr/lib/lsb \
+ ${libdir}/lsb \
${base_libdir}/lsb/* \
${libdir}/sendmail \
"
--
1.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes
2014-11-04 13:40 ` [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes Koen Kooi
@ 2014-11-04 14:52 ` Mark Hatle
0 siblings, 0 replies; 5+ messages in thread
From: Mark Hatle @ 2014-11-04 14:52 UTC (permalink / raw)
To: openembedded-core
On 11/4/14, 7:40 AM, Koen Kooi wrote:
> Also collapse do_install and do_install_append.
/usr/lib is hard coded for a reason. The LSB specification says the files must
be installed in /usr/lib/lsb. (If this has changed, great, but it was true
through at least early last year.)
--Mark
> Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
> ---
> meta/recipes-extended/lsb/lsb_4.1.bb | 11 +++++------
> 1 file changed, 5 insertions(+), 6 deletions(-)
>
> diff --git a/meta/recipes-extended/lsb/lsb_4.1.bb b/meta/recipes-extended/lsb/lsb_4.1.bb
> index 34a70d0..897520a 100644
> --- a/meta/recipes-extended/lsb/lsb_4.1.bb
> +++ b/meta/recipes-extended/lsb/lsb_4.1.bb
> @@ -72,9 +72,7 @@ do_install(){
> touch ${D}${sysconfdir}/lsb-release.d/graphics-${PV}-ppc64
> touch ${D}${sysconfdir}/lsb-release.d/desktop-${PV}-ppc64
> fi
> -}
>
> -do_install_append(){
> install -d ${D}${sysconfdir}/core-lsb
> install -d ${D}/${baselib}/lsb
> for i in lsb_killproc lsb_log_message lsb_pidofproc lsb_start_daemon
> @@ -84,9 +82,9 @@ do_install_append(){
> install -m 0755 ${WORKDIR}/init-functions ${D}/${baselib}/lsb
>
> # creat links for LSB test
> - install -d ${D}/usr/lib/lsb
> - ln -sf ${sbindir}/chkconfig ${D}/usr/lib/lsb/install_initd
> - ln -sf ${sbindir}/chkconfig ${D}/usr/lib/lsb/remove_initd
> + install -d ${D}${libdir}/lsb
> + ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/install_initd
> + ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/remove_initd
> install -d ${D}/${libdir}
> ln -sf ${sbindir}/sendmail ${D}/${libdir}/sendmail
>
> @@ -120,8 +118,9 @@ do_install_append(){
> ln -sf ld.so.1 ld-lsb-ppc32.so.3
> fi
> }
> +
> FILES_${PN} += "/lib64 \
> - /usr/lib/lsb \
> + ${libdir}/lsb \
> ${base_libdir}/lsb/* \
> ${libdir}/sendmail \
> "
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 3/3] lsb 4.1: fix indentation
2014-11-04 13:40 [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Koen Kooi
2014-11-04 13:40 ` [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes Koen Kooi
@ 2014-11-04 13:40 ` Koen Kooi
2014-11-05 10:48 ` [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Richard Purdie
2 siblings, 0 replies; 5+ messages in thread
From: Koen Kooi @ 2014-11-04 13:40 UTC (permalink / raw)
To: openembedded-core; +Cc: Koen Kooi
Shell methods use tabs, variables use spaces.
Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 96 ++++++++++++++++++------------------
1 file changed, 48 insertions(+), 48 deletions(-)
diff --git a/meta/recipes-extended/lsb/lsb_4.1.bb b/meta/recipes-extended/lsb/lsb_4.1.bb
index 897520a..b35796d 100644
--- a/meta/recipes-extended/lsb/lsb_4.1.bb
+++ b/meta/recipes-extended/lsb/lsb_4.1.bb
@@ -15,7 +15,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
file://lsb_log_message \
file://lsb_pidofproc \
file://lsb_start_daemon \
- "
+ "
SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172"
@@ -73,54 +73,54 @@ do_install(){
touch ${D}${sysconfdir}/lsb-release.d/desktop-${PV}-ppc64
fi
- install -d ${D}${sysconfdir}/core-lsb
- install -d ${D}/${baselib}/lsb
- for i in lsb_killproc lsb_log_message lsb_pidofproc lsb_start_daemon
- do
- install -m 0755 ${WORKDIR}/${i} ${D}${sysconfdir}/core-lsb
- done
- install -m 0755 ${WORKDIR}/init-functions ${D}/${baselib}/lsb
-
- # creat links for LSB test
- install -d ${D}${libdir}/lsb
- ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/install_initd
- ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/remove_initd
- install -d ${D}/${libdir}
- ln -sf ${sbindir}/sendmail ${D}/${libdir}/sendmail
-
- if [ "${TARGET_ARCH}" = "x86_64" ];then
- cd ${D}
- if [ "${baselib}" != "lib64" ]; then
- ln -sf ${baselib} lib64
- fi
- cd ${D}/${baselib}
- ln -sf ld-linux-x86-64.so.2 ld-lsb-x86-64.so.2
- ln -sf ld-linux-x86-64.so.2 ld-lsb-x86-64.so.3
- fi
- if [ "${TARGET_ARCH}" = "i586" ];then
- cd ${D}/${baselib}
- ln -sf ld-linux.so.2 ld-lsb.so.2
- ln -sf ld-linux.so.2 ld-lsb.so.3
- fi
-
- if [ "${TARGET_ARCH}" = "powerpc64" ];then
- cd ${D}
- if [ "${baselib}" != "lib64" ]; then
- ln -sf ${baselib} lib64
- fi
- cd ${D}/${baselib}
- ln -sf ld64.so.1 ld-lsb-ppc64.so.2
- ln -sf ld64.so.1 ld-lsb-ppc64.so.3
- fi
- if [ "${TARGET_ARCH}" = "powerpc" ];then
- cd ${D}/${baselib}
- ln -sf ld.so.1 ld-lsb-ppc32.so.2
- ln -sf ld.so.1 ld-lsb-ppc32.so.3
- fi
+ install -d ${D}${sysconfdir}/core-lsb
+ install -d ${D}/${baselib}/lsb
+ for i in lsb_killproc lsb_log_message lsb_pidofproc lsb_start_daemon
+ do
+ install -m 0755 ${WORKDIR}/${i} ${D}${sysconfdir}/core-lsb
+ done
+ install -m 0755 ${WORKDIR}/init-functions ${D}/${baselib}/lsb
+
+ # creat links for LSB test
+ install -d ${D}${libdir}/lsb
+ ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/install_initd
+ ln -sf ${sbindir}/chkconfig ${D}${libdir}/lsb/remove_initd
+ install -d ${D}/${libdir}
+ ln -sf ${sbindir}/sendmail ${D}/${libdir}/sendmail
+
+ if [ "${TARGET_ARCH}" = "x86_64" ];then
+ cd ${D}
+ if [ "${baselib}" != "lib64" ]; then
+ ln -sf ${baselib} lib64
+ fi
+ cd ${D}/${baselib}
+ ln -sf ld-linux-x86-64.so.2 ld-lsb-x86-64.so.2
+ ln -sf ld-linux-x86-64.so.2 ld-lsb-x86-64.so.3
+ fi
+ if [ "${TARGET_ARCH}" = "i586" ];then
+ cd ${D}/${baselib}
+ ln -sf ld-linux.so.2 ld-lsb.so.2
+ ln -sf ld-linux.so.2 ld-lsb.so.3
+ fi
+
+ if [ "${TARGET_ARCH}" = "powerpc64" ];then
+ cd ${D}
+ if [ "${baselib}" != "lib64" ]; then
+ ln -sf ${baselib} lib64
+ fi
+ cd ${D}/${baselib}
+ ln -sf ld64.so.1 ld-lsb-ppc64.so.2
+ ln -sf ld64.so.1 ld-lsb-ppc64.so.3
+ fi
+ if [ "${TARGET_ARCH}" = "powerpc" ];then
+ cd ${D}/${baselib}
+ ln -sf ld.so.1 ld-lsb-ppc32.so.2
+ ln -sf ld.so.1 ld-lsb-ppc32.so.3
+ fi
}
FILES_${PN} += "/lib64 \
- ${libdir}/lsb \
+ ${libdir}/lsb \
${base_libdir}/lsb/* \
- ${libdir}/sendmail \
- "
+ ${libdir}/sendmail \
+ "
--
1.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage
2014-11-04 13:40 [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Koen Kooi
2014-11-04 13:40 ` [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes Koen Kooi
2014-11-04 13:40 ` [PATCH 3/3] lsb 4.1: fix indentation Koen Kooi
@ 2014-11-05 10:48 ` Richard Purdie
2 siblings, 0 replies; 5+ messages in thread
From: Richard Purdie @ 2014-11-05 10:48 UTC (permalink / raw)
To: Koen Kooi; +Cc: openembedded-core
On Tue, 2014-11-04 at 14:40 +0100, Koen Kooi wrote:
> LSB can't cope with a "make clean" so disable that
> newly enabled functionality.
>
> Signed-off-by: Koen Kooi <koen.kooi@linaro.org>
> ---
> meta/recipes-extended/lsb/lsb_4.1.bb | 6 ++++++
> 1 file changed, 6 insertions(+)
In the interests of being clear and transparent, there are issues with
the clean change and I think we need a better mechanism for controlling
this. I've a patch in progress which introduces a CLEANBROKEN variable
which can be set in recipes where "make clean" is broken.
I have a list of several other recipes with the issue and am working on
pulling together a list/patch.
Cheers,
Richard
> diff --git a/meta/recipes-extended/lsb/lsb_4.1.bb b/meta/recipes-extended/lsb/lsb_4.1.bb
> index d265731..34a70d0 100644
> --- a/meta/recipes-extended/lsb/lsb_4.1.bb
> +++ b/meta/recipes-extended/lsb/lsb_4.1.bb
> @@ -21,6 +21,12 @@ SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
> SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172"
> S = "${WORKDIR}/lsb-release-1.4"
>
> +# 'make clean' isn't supported
> +
> +do_configure() {
> + :
> +}
> +
> do_install(){
> oe_runmake install prefix=${D} mandir=${D}/${datadir}/man/ DESTDIR=${D}
>
> --
> 1.9.3
>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-11-05 10:48 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-04 13:40 [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Koen Kooi
2014-11-04 13:40 ` [PATCH 2/3] lsb 4.1: fix /usr/lib/ hardcodes Koen Kooi
2014-11-04 14:52 ` Mark Hatle
2014-11-04 13:40 ` [PATCH 3/3] lsb 4.1: fix indentation Koen Kooi
2014-11-05 10:48 ` [PATCH 1/3] lsb 4.1: Deal with "make clean" breakage Richard Purdie
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.