[OE-core] [PATCH] image.bbclass: ensure makedevs does return true

2013-10-01 Thread Saul Wold
When doing an incremental image generation, ensure that makedevs returns true when it encounters a node that already exists, otherwise image generation will fail. [YOCTO #5288] Signed-off-by: Saul Wold s...@linux.intel.com --- meta/classes/image.bbclass | 2 +- 1 file changed, 1 insertion(+),

Re: [OE-core] [PATCH] image.bbclass: ensure makedevs does return true

2013-10-01 Thread Mark Hatle
On 10/1/13 10:40 AM, Saul Wold wrote: When doing an incremental image generation, ensure that makedevs returns true when it encounters a node that already exists, otherwise image generation will fail. [YOCTO #5288] Signed-off-by: Saul Wold s...@linux.intel.com --- meta/classes/image.bbclass

[OE-core] [PATCH] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Saul Wold
This ensures that makedevs will not cause image creation failures when it encounters a pipe (fifo) that exists from a previous image. The other file/device creation code either ignore this error or check for file type. [YOCTO #5288] Signed-off-by: Saul Wold s...@linux.intel.com ---

Re: [OE-core] [PATCH] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Phil Blundell
On Tue, 2013-10-01 at 09:51 -0700, Saul Wold wrote: static void add_new_fifo(char *name, char *path, unsigned long uid, unsigned long gid, unsigned long mode) { - if (mknod(path, mode, 0)) + if (mknod(path, mode, 0) errno != EEXIST)

Re: [OE-core] [PATCH] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Mark Hatle
On 10/1/13 11:51 AM, Saul Wold wrote: This ensures that makedevs will not cause image creation failures when it encounters a pipe (fifo) that exists from a previous image. The other file/device creation code either ignore this error or check for file type. [YOCTO #5288] Signed-off-by: Saul

Re: [OE-core] [PATCH] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Khem Raj
On Oct 1, 2013, at 9:56 AM, Phil Blundell p...@pbcl.net wrote: On Tue, 2013-10-01 at 09:51 -0700, Saul Wold wrote: static void add_new_fifo(char *name, char *path, unsigned long uid, unsigned long gid, unsigned long mode) { -if (mknod(path, mode, 0)) +

[OE-core] [PATCH] update-rc: Stop and remove service if updating package

2013-10-01 Thread eu
From: Felipe F. Tonello e...@felipetonello.com Since when updating packages packages managers do not remove the previous package, it just replaces new files and run pre/post install scripts. This causes not to update update-rc scripts if they were changed. This patch is useful in case the newer

Re: [OE-core] [PATCH 1/1] buildhistory.bbclass: always record PKG, PKGE, PKGV and PKGR

2013-10-01 Thread Mark Hatle
On 9/30/13 11:26 AM, Paul Eggleton wrote: On Thursday 26 September 2013 13:23:32 qi.c...@windriver.com wrote: From: Chen Qi qi.c...@windriver.com The buildhistory.bbclass always records PV instead of PKGV. However, the buildhistory-diff script treats PKGV as a monitored variable instead of PV.

[OE-core] [PATCH v2] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Saul Wold
This ensures that makedevs will not cause image creation failures when it encounters a pipe (fifo) that exists from a previous image. This handles mode changes and it will correctly fail for dangling symlinks. [YOCTO #5288] Signed-off-by: Saul Wold s...@linux.intel.com makedev Signed-off-by:

Re: [OE-core] [PATCH v2] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Phil Blundell
On Tue, 2013-10-01 at 12:15 -0700, Saul Wold wrote: + int status; + struct stat sb; + + memset(sb, 0, sizeof(struct stat)); + status = stat(path, sb); Don't you want lstat() there? Also, I think *stat() is guaranteed to fill in all of sb if it returns anything other than an

Re: [OE-core] [PATCH v2] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Saul Wold
On 10/01/2013 12:49 PM, Phil Blundell wrote: On Tue, 2013-10-01 at 12:15 -0700, Saul Wold wrote: + int status; + struct stat sb; + + memset(sb, 0, sizeof(struct stat)); + status = stat(path, sb); Don't you want lstat() there? Also, I think *stat() is guaranteed to

Re: [OE-core] [PATCH v2] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Phil Blundell
On Tue, 2013-10-01 at 12:58 -0700, Saul Wold wrote: On 10/01/2013 12:49 PM, Phil Blundell wrote: On Tue, 2013-10-01 at 12:15 -0700, Saul Wold wrote: + int status; + struct stat sb; + + memset(sb, 0, sizeof(struct stat)); + status = stat(path, sb); Don't you want lstat() there?

Re: [OE-core] [PATCH v2] makedevs: Do not return error if the fifo exisits

2013-10-01 Thread Saul Wold
On 10/01/2013 01:07 PM, Phil Blundell wrote: On Tue, 2013-10-01 at 12:58 -0700, Saul Wold wrote: On 10/01/2013 12:49 PM, Phil Blundell wrote: On Tue, 2013-10-01 at 12:15 -0700, Saul Wold wrote: + int status; + struct stat sb; + + memset(sb, 0, sizeof(struct stat)); +

[OE-core] [PATCH v2] kmod: ptest fixes

2013-10-01 Thread Tudor Florea
Make kmod-ptest able to compile with separated source and build dir. Since kmod test files contain kernel modules for many different architectures, strip and arch gets confused and throws errors. Signed-off-by: Tudor Florea tudor.flo...@enea.com Signed-off-by: Björn Stenberg b...@enea.com