* [PATCH 0/1] Remove cp -n option as discussed v2
@ 2011-08-23 15:22 Paul Eggleton
2011-08-23 15:22 ` [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command Paul Eggleton
0 siblings, 1 reply; 4+ messages in thread
From: Paul Eggleton @ 2011-08-23 15:22 UTC (permalink / raw)
To: openembedded-core
Since v1, add a fix for kernel.bbclass as well.
The following changes since commit 9134800548a03bfe3a7e351a15f2129bcb3cc680:
xcb/libpthreads: Add BBCLASSEXTEND to resolve missing providers in world builds (2011-08-19 16:27:29 -0700)
are available in the git repository at:
git://git.openembedded.org/openembedded-core-contrib paule/sanity3a
http://cgit.openembedded.org/cgit.cgi/openembedded-core-contrib/log/?h=paule/sanity3a
Paul Eggleton (1):
image.bbclass, kernel.bbclass: remove non-standard -n option from cp
command
meta/classes/image.bbclass | 2 +-
meta/classes/kernel.bbclass | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
1.7.4.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command
2011-08-23 15:22 [PATCH 0/1] Remove cp -n option as discussed v2 Paul Eggleton
@ 2011-08-23 15:22 ` Paul Eggleton
2011-08-23 18:26 ` Darren Hart
2011-08-24 1:28 ` Richard Purdie
0 siblings, 2 replies; 4+ messages in thread
From: Paul Eggleton @ 2011-08-23 15:22 UTC (permalink / raw)
To: openembedded-core
-n is not a POSIX option and does not work on some systems (e.g. Debian
Lenny); in any case it is not strictly necessary here, so remove it.
Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
---
meta/classes/image.bbclass | 2 +-
meta/classes/kernel.bbclass | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
index 286ae01..bf8b73a 100644
--- a/meta/classes/image.bbclass
+++ b/meta/classes/image.bbclass
@@ -138,7 +138,7 @@ fakeroot do_rootfs () {
mkdir -p ${IMAGE_ROOTFS}
mkdir -p ${DEPLOY_DIR_IMAGE}
- cp -n ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOY_DIR_IMAGE}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
+ cp ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOY_DIR_IMAGE}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
if [ "${USE_DEVFS}" != "1" ]; then
for devtable in ${@get_devtable_list(d)}; do
diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
index a2b10f2..c855268 100644
--- a/meta/classes/kernel.bbclass
+++ b/meta/classes/kernel.bbclass
@@ -501,7 +501,7 @@ kernel_do_deploy() {
rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.bin
ln -sf ${KERNEL_IMAGE_BASE_NAME}.bin ${KERNEL_IMAGE_SYMLINK_NAME}.bin
- cp -n ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
+ cp ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
}
do_deploy[dirs] = "${DEPLOYDIR} ${B}"
--
1.7.4.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command
2011-08-23 15:22 ` [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command Paul Eggleton
@ 2011-08-23 18:26 ` Darren Hart
2011-08-24 1:28 ` Richard Purdie
1 sibling, 0 replies; 4+ messages in thread
From: Darren Hart @ 2011-08-23 18:26 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Paul Eggleton
On 08/23/2011 08:22 AM, Paul Eggleton wrote:
> -n is not a POSIX option and does not work on some systems (e.g. Debian
> Lenny); in any case it is not strictly necessary here, so remove it.
>
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
Acked-by: Darren Hart <dvhart@linux.intel.com>
> ---
> meta/classes/image.bbclass | 2 +-
> meta/classes/kernel.bbclass | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/classes/image.bbclass b/meta/classes/image.bbclass
> index 286ae01..bf8b73a 100644
> --- a/meta/classes/image.bbclass
> +++ b/meta/classes/image.bbclass
> @@ -138,7 +138,7 @@ fakeroot do_rootfs () {
> mkdir -p ${IMAGE_ROOTFS}
> mkdir -p ${DEPLOY_DIR_IMAGE}
>
> - cp -n ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOY_DIR_IMAGE}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
> + cp ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOY_DIR_IMAGE}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
>
> if [ "${USE_DEVFS}" != "1" ]; then
> for devtable in ${@get_devtable_list(d)}; do
> diff --git a/meta/classes/kernel.bbclass b/meta/classes/kernel.bbclass
> index a2b10f2..c855268 100644
> --- a/meta/classes/kernel.bbclass
> +++ b/meta/classes/kernel.bbclass
> @@ -501,7 +501,7 @@ kernel_do_deploy() {
> rm -f ${KERNEL_IMAGE_SYMLINK_NAME}.bin
> ln -sf ${KERNEL_IMAGE_BASE_NAME}.bin ${KERNEL_IMAGE_SYMLINK_NAME}.bin
>
> - cp -n ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
> + cp ${COREBASE}/meta/files/deploydir_readme.txt ${DEPLOYDIR}/README_-_DO_NOT_DELETE_FILES_IN_THIS_DIRECTORY.txt
> }
> do_deploy[dirs] = "${DEPLOYDIR} ${B}"
>
--
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command
2011-08-23 15:22 ` [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command Paul Eggleton
2011-08-23 18:26 ` Darren Hart
@ 2011-08-24 1:28 ` Richard Purdie
1 sibling, 0 replies; 4+ messages in thread
From: Richard Purdie @ 2011-08-24 1:28 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
On Tue, 2011-08-23 at 16:22 +0100, Paul Eggleton wrote:
> -n is not a POSIX option and does not work on some systems (e.g. Debian
> Lenny); in any case it is not strictly necessary here, so remove it.
>
> Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
> ---
> meta/classes/image.bbclass | 2 +-
> meta/classes/kernel.bbclass | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
Merged to master, thanks.
Richard
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-08-24 1:33 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-23 15:22 [PATCH 0/1] Remove cp -n option as discussed v2 Paul Eggleton
2011-08-23 15:22 ` [PATCH 1/1] image.bbclass, kernel.bbclass: remove non-standard -n option from cp command Paul Eggleton
2011-08-23 18:26 ` Darren Hart
2011-08-24 1:28 ` 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.