* [PATCH 0/1] Remove cp -n option as discussed
@ 2011-08-23 10:11 Paul Eggleton
2011-08-23 10:11 ` [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command Paul Eggleton
0 siblings, 1 reply; 5+ messages in thread
From: Paul Eggleton @ 2011-08-23 10:11 UTC (permalink / raw)
To: openembedded-core
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: remove non-standard -n option from cp command
meta/classes/image.bbclass | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
--
1.7.4.1
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command
2011-08-23 10:11 [PATCH 0/1] Remove cp -n option as discussed Paul Eggleton
@ 2011-08-23 10:11 ` Paul Eggleton
2011-08-23 15:07 ` Koen Kooi
2011-08-23 15:12 ` Saul Wold
0 siblings, 2 replies; 5+ messages in thread
From: Paul Eggleton @ 2011-08-23 10:11 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 +-
1 files changed, 1 insertions(+), 1 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
--
1.7.4.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command
2011-08-23 10:11 ` [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command Paul Eggleton
@ 2011-08-23 15:07 ` Koen Kooi
2011-08-23 15:12 ` Saul Wold
1 sibling, 0 replies; 5+ messages in thread
From: Koen Kooi @ 2011-08-23 15:07 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer
Op 23 aug. 2011, om 12:11 heeft Paul Eggleton het volgende geschreven:
> -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>
Tested-by: Koen Kooi <koen@dominion.thruhere.net>
> ---
> meta/classes/image.bbclass | 2 +-
> 1 files changed, 1 insertions(+), 1 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
> --
> 1.7.4.1
>
>
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command
2011-08-23 10:11 ` [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command Paul Eggleton
2011-08-23 15:07 ` Koen Kooi
@ 2011-08-23 15:12 ` Saul Wold
2011-08-23 15:19 ` Paul Eggleton
1 sibling, 1 reply; 5+ messages in thread
From: Saul Wold @ 2011-08-23 15:12 UTC (permalink / raw)
To: Patches and discussions about the oe-core layer; +Cc: Paul Eggleton
On 08/23/2011 03:11 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>
> ---
> meta/classes/image.bbclass | 2 +-
> 1 files changed, 1 insertions(+), 1 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
Paul,
Is this in addition to the previous image.bbclass patch you provided?
If so, there are 2 usages of -n in original patch that is merged.
Sau!
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-08-23 15:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-08-23 10:11 [PATCH 0/1] Remove cp -n option as discussed Paul Eggleton
2011-08-23 10:11 ` [PATCH 1/1] image.bbclass: remove non-standard -n option from cp command Paul Eggleton
2011-08-23 15:07 ` Koen Kooi
2011-08-23 15:12 ` Saul Wold
2011-08-23 15:19 ` Paul Eggleton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox