Patchwork [1/1] sudo: fixed wrong `chmod` path

login
register
mail settings
Submitter Mihai Lindner
Date May 24, 2012, 5:19 p.m.
Message ID <6b907017a92c4178980b18fbe2fef47cbd2b1a34.1337519599.git.mihaix.lindner@linux.intel.com>
Download mbox | patch
Permalink /patch/28615/
State Accepted
Commit 70562306a051ac93001851213441c9d5d6234517
Headers show

Comments

Mihai Lindner - May 24, 2012, 5:19 p.m.
Placed $D between braces ${D} to be correctly expanded to the
workdir path, instead of a path relative to host rootfs.
Currently, bitbake sudo fails on host systems where sudo is not
installed.

Signed-off-by: Mihai Lindner <mihaix.lindner@linux.intel.com>

---
meta/recipes-extended/sudo/sudo_1.8.4p4.bb |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
Saul Wold - May 25, 2012, 4:41 p.m.
On 05/24/2012 10:19 AM, Mihai Lindner wrote:
> Placed $D between braces ${D} to be correctly expanded to the
> workdir path, instead of a path relative to host rootfs.
> Currently, bitbake sudo fails on host systems where sudo is not
> installed.
>
> Signed-off-by: Mihai Lindner<mihaix.lindner@linux.intel.com>
>
> ---
> meta/recipes-extended/sudo/sudo_1.8.4p4.bb |    6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/meta/recipes-extended/sudo/sudo_1.8.4p4.bb b/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
> index 2315322..20c4a14 100644
> --- a/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
> +++ b/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
> @@ -1,6 +1,6 @@
>   require sudo.inc
>
> -PR = "r0"
> +PR = "r1"
>
>   SRC_URI = "http://ftp.sudo.ws/sudo/dist/sudo-${PV}.tar.gz \
>              file://libtool.patch \
> @@ -24,6 +24,6 @@ do_install_append () {
>   		fi
>   	done
>
> -	chmod 4111 $D/usr/bin/sudo
> -	chmod 0440 $D/etc/sudoers
> +	chmod 4111 ${D}/usr/bin/sudo
> +	chmod 0440 ${D}/etc/sudoers
>   }
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
>
>

Merged into OE-Core

Thanks
	Sau!

Patch

diff --git a/meta/recipes-extended/sudo/sudo_1.8.4p4.bb b/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
index 2315322..20c4a14 100644
--- a/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
+++ b/meta/recipes-extended/sudo/sudo_1.8.4p4.bb
@@ -1,6 +1,6 @@ 
 require sudo.inc
 
-PR = "r0"
+PR = "r1"
 
 SRC_URI = "http://ftp.sudo.ws/sudo/dist/sudo-${PV}.tar.gz \
            file://libtool.patch \
@@ -24,6 +24,6 @@  do_install_append () {
 		fi
 	done
 
-	chmod 4111 $D/usr/bin/sudo
-	chmod 0440 $D/etc/sudoers
+	chmod 4111 ${D}/usr/bin/sudo
+	chmod 0440 ${D}/etc/sudoers
 }