Re: [oe] [meta-oe][PATCH] stop trying to remove folders that do not exist

2014-07-15 Thread Stephano Cetola 
Stephano Cetola  gmail.com> writes:

> 
> From: Stephano Cetola  reachtech.com>
> 
> The directory ${D}/${TMPDIR} does not exist. This patch removes code
> that was calling rm and rmdir on that location.
> 
> Signed-off-by: Stephano Cetola  reachtech.com>
> ---
>  meta-oe/recipes-devtools/php/php.inc |6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/meta-oe/recipes-devtools/php/php.inc b/meta-oe/recipes-
devtools/php/php.inc
> index b1bf79c..000316b 100644
> --- a/meta-oe/recipes-devtools/php/php.inc
> +++ b/meta-oe/recipes-devtools/php/php.inc
>  -90,6 +90,12  do_install_append_pn-php() {
>  sed -i 's:=/var:=${localstatedir}:g' ${B}/sapi/fpm/init.d.php-fpm
>  install -m 0755 ${B}/sapi/fpm/init.d.php-fpm 
${D}${sysconfdir}/init.d/php-fpm
>  TMP=`dirname ${D}/${TMPDIR}`
> +bbwarn "D"
> +bbwarn ${D}
> +bbwarn "TMPDIR"
> +bbwarn ${TMPDIR}
> +bbwarn "TMP"
> +bbwarn ${TMP}
>  while test ${TMP} != ${D}; do
>  rmdir ${TMP}
>  TMP=`dirname ${TMP}`;

Sorry for the mixup. I was ssh'd into my testing machine at work and thought 
I was committing from my local machine. I have resubmitted. Sorry for the 
confusion.

-Stephano


-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-oe][PATCH] stop trying to remove folders that do not exist

2014-07-15 Thread Martin Jansa 
On Tue, Jul 15, 2014 at 11:01:55AM -0700, Stephano Cetola wrote:
> From: Stephano Cetola 
> 
> The directory ${D}/${TMPDIR} does not exist. This patch removes code
> that was calling rm and rmdir on that location.

this doesn't match with the diff

> 
> Signed-off-by: Stephano Cetola 
> ---
>  meta-oe/recipes-devtools/php/php.inc |6 ++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/meta-oe/recipes-devtools/php/php.inc 
> b/meta-oe/recipes-devtools/php/php.inc
> index b1bf79c..000316b 100644
> --- a/meta-oe/recipes-devtools/php/php.inc
> +++ b/meta-oe/recipes-devtools/php/php.inc
> @@ -90,6 +90,12 @@ do_install_append_pn-php() {
>  sed -i 's:=/var:=${localstatedir}:g' ${B}/sapi/fpm/init.d.php-fpm
>  install -m 0755 ${B}/sapi/fpm/init.d.php-fpm 
> ${D}${sysconfdir}/init.d/php-fpm
>  TMP=`dirname ${D}/${TMPDIR}`
> +bbwarn "D"
> +bbwarn ${D}
> +bbwarn "TMPDIR"
> +bbwarn ${TMPDIR}
> +bbwarn "TMP"
> +bbwarn ${TMP}
>  while test ${TMP} != ${D}; do
>  rmdir ${TMP}
>  TMP=`dirname ${TMP}`;
> -- 
> 1.7.10.4
> 
> -- 
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa jabber: martin.ja...@gmail.com


signature.asc
Description: Digital signature
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-oe][PATCH] stop trying to remove folders that do not exist

2014-07-15 Thread Gary Thomas 

On 2014-07-15 12:01, Stephano Cetola wrote:

From: Stephano Cetola 

The directory ${D}/${TMPDIR} does not exist. This patch removes code
that was calling rm and rmdir on that location.

Signed-off-by: Stephano Cetola 
---
  meta-oe/recipes-devtools/php/php.inc |6 ++
  1 file changed, 6 insertions(+)

diff --git a/meta-oe/recipes-devtools/php/php.inc 
b/meta-oe/recipes-devtools/php/php.inc
index b1bf79c..000316b 100644
--- a/meta-oe/recipes-devtools/php/php.inc
+++ b/meta-oe/recipes-devtools/php/php.inc
@@ -90,6 +90,12 @@ do_install_append_pn-php() {
  sed -i 's:=/var:=${localstatedir}:g' ${B}/sapi/fpm/init.d.php-fpm
  install -m 0755 ${B}/sapi/fpm/init.d.php-fpm 
${D}${sysconfdir}/init.d/php-fpm
  TMP=`dirname ${D}/${TMPDIR}`
+bbwarn "D"
+bbwarn ${D}
+bbwarn "TMPDIR"
+bbwarn ${TMPDIR}
+bbwarn "TMP"
+bbwarn ${TMP}
  while test ${TMP} != ${D}; do
  rmdir ${TMP}
  TMP=`dirname ${TMP}`;



The actual patch does not do what the summary claims.  Are you
sure this is what you wanted to send/commit?

--

Gary Thomas |  Consulting for the
MLB Associates  |Embedded world

--
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [meta-oe][PATCH] stop trying to remove folders that do not exist

2014-07-15 Thread Stephano Cetola 
From: Stephano Cetola 

The directory ${D}/${TMPDIR} does not exist. This patch removes code
that was calling rm and rmdir on that location.

Signed-off-by: Stephano Cetola 
---
 meta-oe/recipes-devtools/php/php.inc |6 ++
 1 file changed, 6 insertions(+)

diff --git a/meta-oe/recipes-devtools/php/php.inc 
b/meta-oe/recipes-devtools/php/php.inc
index b1bf79c..000316b 100644
--- a/meta-oe/recipes-devtools/php/php.inc
+++ b/meta-oe/recipes-devtools/php/php.inc
@@ -90,6 +90,12 @@ do_install_append_pn-php() {
 sed -i 's:=/var:=${localstatedir}:g' ${B}/sapi/fpm/init.d.php-fpm
 install -m 0755 ${B}/sapi/fpm/init.d.php-fpm 
${D}${sysconfdir}/init.d/php-fpm
 TMP=`dirname ${D}/${TMPDIR}`
+bbwarn "D"
+bbwarn ${D}
+bbwarn "TMPDIR"
+bbwarn ${TMPDIR}
+bbwarn "TMP"
+bbwarn ${TMP}
 while test ${TMP} != ${D}; do
 rmdir ${TMP}
 TMP=`dirname ${TMP}`;
-- 
1.7.10.4

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel