[for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Otavio Salvador]

This allows use of meta-fsl-arm without meta-oe by default allowing
Yocto autobuilder to succeed in testing danny branch.

The cons of this change is that people using udev with feed will need
to unmask udev and add an udev bbappend in their internal layer to
restore upgrade path.

Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
 README                                        | 2 --
 conf/layer.conf                               | 3 +++
 conf/machine/include/fsl-default-versions.inc | 2 --
 recipes-core/udev/udev_173.bbappend           | 8 --------
 4 files changed, 3 insertions(+), 12 deletions(-)
 delete mode 100644 recipes-core/udev/udev_173.bbappend

diff --git a/README b/README
index 1987424..63aef9f 100644
--- a/README
+++ b/README
@@ -17,8 +17,6 @@ revision: HEAD
 
 To avoid the meta-oe dependency add to local.conf:
 
-BBMASK = "udev_.*\.bbappend$"
-
 EULA
 ----
 
diff --git a/conf/layer.conf b/conf/layer.conf
index c4b9cd1..0f5b3f2 100644
--- a/conf/layer.conf
+++ b/conf/layer.conf
@@ -9,6 +9,9 @@ BBFILE_COLLECTIONS += "fsl-arm"
 BBFILE_PATTERN_fsl-arm := "^${LAYERDIR}/"
 BBFILE_PRIORITY_fsl-arm = "5"
 
+# Mask out meta-oe newer udev
+BBMASK = "meta-oe/recipes-core/udev/udev_(173|175|182)\.bb$"
+
 FSL_EULA_FILE = "${LAYERDIR}/EULA"
 
 FSL_MIRROR ?= "http://www.freescale.com/lgfiles/NMG/MAD/YOCTO/"
diff --git a/conf/machine/include/fsl-default-versions.inc b/conf/machine/include/fsl-default-versions.inc
index 5319f34..d2a4e93 100644
--- a/conf/machine/include/fsl-default-versions.inc
+++ b/conf/machine/include/fsl-default-versions.inc
@@ -1,3 +1 @@
 # Freescale BSP default versions
-
-PREFERRED_VERSION_udev = "173"
diff --git a/recipes-core/udev/udev_173.bbappend b/recipes-core/udev/udev_173.bbappend
deleted file mode 100644
index 29b20bd..0000000
--- a/recipes-core/udev/udev_173.bbappend
+++ /dev/null
@@ -1,8 +0,0 @@
-PRINC := "${@int(PRINC) + 2}"
-
-# Switch to git source
-# 173 tag
-SRCREV = "ad667dff51711fed763a23283d973486de3cd6b5"
-SRC_URI := "${@oe_filter_out('${KERNELORG_MIRROR}/linux/utils/kernel/hotplug/udev-${PV}.tar.bz2', '${SRC_URI}', d)}"
-SRC_URI_prepend = "git://git.kernel.org/pub/scm/linux/hotplug/udev.git;protocol=git "
-S = "${WORKDIR}/git"
-- 
1.8.1

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Eric Bénard]

Hi Otavio,

Le Wed,  6 Feb 2013 14:15:36 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :

> This allows use of meta-fsl-arm without meta-oe by default allowing
> Yocto autobuilder to succeed in testing danny branch.
> 
> The cons of this change is that people using udev with feed will need
> to unmask udev and add an udev bbappend in their internal layer to
> restore upgrade path.
> 
no, danny is a _stable_ branch so you can't break it this way now.
Please configure the autobuilder to have the right setting as stated in
the readme.

Eric

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Otavio Salvador]

On Wed, Feb 6, 2013 at 4:17 PM, Eric Bénard <eric@eukrea.com> wrote:
> Hi Otavio,
>
> Le Wed,  6 Feb 2013 14:15:36 -0200,
> Otavio Salvador <otavio@ossystems.com.br> a écrit :
>
>> This allows use of meta-fsl-arm without meta-oe by default allowing
>> Yocto autobuilder to succeed in testing danny branch.
>>
>> The cons of this change is that people using udev with feed will need
>> to unmask udev and add an udev bbappend in their internal layer to
>> restore upgrade path.
>>
> no, danny is a _stable_ branch so you can't break it this way now.
> Please configure the autobuilder to have the right setting as stated in
> the readme.

That's why I send it and I am waiting for comments; we cannot fix the
autobuilder as it tests the Yocto + <layer> so <layer> cannot depends
on meta-oe. In fact when Ross pinged me today to discuss it I asked if
it'd be possible but it seems not.

Most users won't see any difference with this change; the exception
here are those using feeds. Master is fixed in this regards and does
not depends on meta-oe however danny I see no way to make it work
without meta-oe without manual change in local.conf. Do you see any
alternative?

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Eric Bénard]

Le Wed, 6 Feb 2013 16:22:33 -0200,
Otavio Salvador <otavio@ossystems.com.br> a écrit :

> On Wed, Feb 6, 2013 at 4:17 PM, Eric Bénard <eric@eukrea.com> wrote:
> > Hi Otavio,
> >
> > Le Wed,  6 Feb 2013 14:15:36 -0200,
> > Otavio Salvador <otavio@ossystems.com.br> a écrit :
> >
> >> This allows use of meta-fsl-arm without meta-oe by default allowing
> >> Yocto autobuilder to succeed in testing danny branch.
> >>
> >> The cons of this change is that people using udev with feed will need
> >> to unmask udev and add an udev bbappend in their internal layer to
> >> restore upgrade path.
> >>
> > no, danny is a _stable_ branch so you can't break it this way now.
> > Please configure the autobuilder to have the right setting as stated in
> > the readme.
> 
> That's why I send it and I am waiting for comments; we cannot fix the
> autobuilder as it tests the Yocto + <layer> so <layer> cannot depends
> on meta-oe. In fact when Ross pinged me today to discuss it I asked if
> it'd be possible but it seems not.
> 
> Most users won't see any difference with this change; the exception
> here are those using feeds. Master is fixed in this regards and does
> not depends on meta-oe however danny I see no way to make it work
> without meta-oe without manual change in local.conf. Do you see any
> alternative?
> 
yes : removing meta-fsl-arm's danny branch from the autobuilder.

Eric

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Otavio Salvador]

On Wed, Feb 6, 2013 at 4:41 PM, Eric Bénard <eric@eukrea.com> wrote:
> Le Wed, 6 Feb 2013 16:22:33 -0200,
> Otavio Salvador <otavio@ossystems.com.br> a écrit :
>
>> On Wed, Feb 6, 2013 at 4:17 PM, Eric Bénard <eric@eukrea.com> wrote:
>> > Hi Otavio,
>> >
>> > Le Wed,  6 Feb 2013 14:15:36 -0200,
>> > Otavio Salvador <otavio@ossystems.com.br> a écrit :
>> >
>> >> This allows use of meta-fsl-arm without meta-oe by default allowing
>> >> Yocto autobuilder to succeed in testing danny branch.
>> >>
>> >> The cons of this change is that people using udev with feed will need
>> >> to unmask udev and add an udev bbappend in their internal layer to
>> >> restore upgrade path.
>> >>
>> > no, danny is a _stable_ branch so you can't break it this way now.
>> > Please configure the autobuilder to have the right setting as stated in
>> > the readme.
>>
>> That's why I send it and I am waiting for comments; we cannot fix the
>> autobuilder as it tests the Yocto + <layer> so <layer> cannot depends
>> on meta-oe. In fact when Ross pinged me today to discuss it I asked if
>> it'd be possible but it seems not.
>>
>> Most users won't see any difference with this change; the exception
>> here are those using feeds. Master is fixed in this regards and does
>> not depends on meta-oe however danny I see no way to make it work
>> without meta-oe without manual change in local.conf. Do you see any
>> alternative?
>>
> yes : removing meta-fsl-arm's danny branch from the autobuilder.

This is not an option as it push on me and other people the obligation
of daily build it. I'd prefer to keep it to autobuilder.

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Burton, Ross]

On 6 February 2013 18:17, Eric Bénard <eric@eukrea.com> wrote:
> no, danny is a _stable_ branch so you can't break it this way now.
> Please configure the autobuilder to have the right setting as stated in
> the readme.

I'll see what I can do, but I'm not convinced we'd be happy (or able)
to do that.

(I'm also surprised you can mask out a hanging bbappend, as I thought
masking happened after the parse)

Ross

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Burton, Ross]

On 6 February 2013 19:48, Burton, Ross <ross.burton@intel.com> wrote:
> On 6 February 2013 18:17, Eric Bénard <eric@eukrea.com> wrote:
>> no, danny is a _stable_ branch so you can't break it this way now.
>> Please configure the autobuilder to have the right setting as stated in
>> the readme.
>
> I'll see what I can do, but I'm not convinced we'd be happy (or able)
> to do that.

Beth said she'll see what she can do, I filed a bug:

https://bugzilla.yoctoproject.org/show_bug.cgi?id=3860

Ross

Re: [for-danny][PATCH] udev: Drop bbappend and mask out Meta-OpenEmbedded udev

[Otavio Salvador]

On Wed, Feb 6, 2013 at 5:53 PM, Burton, Ross <ross.burton@intel.com> wrote:
> On 6 February 2013 19:48, Burton, Ross <ross.burton@intel.com> wrote:
>> On 6 February 2013 18:17, Eric Bénard <eric@eukrea.com> wrote:
>>> no, danny is a _stable_ branch so you can't break it this way now.
>>> Please configure the autobuilder to have the right setting as stated in
>>> the readme.
>>
>> I'll see what I can do, but I'm not convinced we'd be happy (or able)
>> to do that.
>
> Beth said she'll see what she can do, I filed a bug:
>
> https://bugzilla.yoctoproject.org/show_bug.cgi?id=3860

Thanks Ross; thank Beth for me :-)

-- 
Otavio Salvador                             O.S. Systems
E-mail: otavio@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br