All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/8] [meta-oe] devtools/php: remove the failing ${D}/${TMPDIR} code
@ 2018-09-18  8:26 Max Kellermann
  2018-09-18  8:26 ` [PATCH 2/8] [meta-oe] vorbis-tools: add PACKAGECONFIG[ogg123] Max Kellermann
                   ` (8 more replies)
  0 siblings, 9 replies; 15+ messages in thread
From: Max Kellermann @ 2018-09-18  8:26 UTC (permalink / raw)
  To: openembedded-devel; +Cc: Max Kellermann

From: Max Kellermann <max.kellermann@gmail.com>

Appending ${TMPDIR} to ${D} doesn't make any sense, because both are
absolute paths.  And additionally, the code fails:

 rmdir: failed to remove '/usr/src/oe/tmp-musl/work/core2-64-oe-linux-musl/php/7.1.9-r0/image//usr': Directory not empty

Signed-off-by: Max Kellermann <max.kellermann@gmail.com>
---
 meta-oe/recipes-devtools/php/php.inc | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/meta-oe/recipes-devtools/php/php.inc b/meta-oe/recipes-devtools/php/php.inc
index 65dfd2d89..120f678d7 100644
--- a/meta-oe/recipes-devtools/php/php.inc
+++ b/meta-oe/recipes-devtools/php/php.inc
@@ -139,7 +139,6 @@ do_install_append_class-target() {
     if [ -d ${RECIPE_SYSROOT_NATIVE}${sysconfdir} ];then
          install -m 0644 ${RECIPE_SYSROOT_NATIVE}${sysconfdir}/pear.conf ${D}${sysconfdir}/
     fi
-    rm -rf ${D}/${TMPDIR}
     rm -rf ${D}/.registry
     rm -rf ${D}/.channels
     rm -rf ${D}/.[a-z]*
@@ -164,14 +163,6 @@ do_install_append_class-target() {
             ${D}${systemd_unitdir}/system/php-fpm.service
     fi
 
-    TMP=`dirname ${D}/${TMPDIR}`
-    while test ${TMP} != ${D}; do
-        if [ -d ${TMP} ]; then
-            rmdir ${TMP}
-        fi
-        TMP=`dirname ${TMP}`;
-    done
-
     if ${@bb.utils.contains('PACKAGECONFIG', 'apache2', 'true', 'false', d)}; then
         install -d ${D}${libdir}/apache2/modules
         install -d ${D}${sysconfdir}/apache2/modules.d
-- 
2.18.0



^ permalink raw reply related	[flat|nested] 15+ messages in thread
end of thread, other threads:[~2018-10-01 16:13 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-18  8:26 [PATCH 1/8] [meta-oe] devtools/php: remove the failing ${D}/${TMPDIR} code Max Kellermann
2018-09-18  8:26 ` [PATCH 2/8] [meta-oe] vorbis-tools: add PACKAGECONFIG[ogg123] Max Kellermann
2018-09-18  8:26 ` [PATCH 3/8] [meta-oe] nginx: add PACKAGECONFIG[ssl] Max Kellermann
2018-09-18  8:26 ` [PATCH 4/8] [meta-oe] mpd: update 0.20.21 Max Kellermann
2018-09-18  8:26 ` [PATCH 5/8] [meta-oe] mpd: sort PACKAGECONFIGs Max Kellermann
2018-09-18  8:26 ` [PATCH 6/8] [meta-oe] mpd: make lots of plugins optional Max Kellermann
2018-09-21  5:18   ` Khem Raj
2018-09-21  8:50     ` [PATCH v2] " Max Kellermann
2018-09-21  9:28     ` Max Kellermann
2018-09-18  8:26 ` [PATCH 7/8] [meta-oe] libmicrohttpd: add PACKAGECONFIG[https] Max Kellermann
2018-09-18  8:26 ` [PATCH 8/8] [meta-oe] libblockdev: make various features optional Max Kellermann
2018-09-21  9:07 ` ✗ patchtest: failure for mpd: make lots of plugins optional Patchwork
2018-09-24 18:48 ` [PATCH 1/8] [meta-oe] devtools/php: remove the failing ${D}/${TMPDIR} code Khem Raj
2018-09-27 12:54   ` Max Kellermann
2018-10-01 16:12     ` Khem Raj

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.