[meta-fsl-ppc] recent breakage

[meta-fsl-ppc] recent breakage

[Gary Thomas]

This commit breaks a build when an image contains udev-extraconf

commit d7df92b6f5c1a673341d54d6b064ad2fd7b16d74
Author: Ting Liu <ting.liu@freescale.com>
Date:   Wed Jan 7 17:54:59 2015 +0800

     udev-extraconf: replace bbappend with own recipe

     * introduce variable RULE for DPAA FMan ethernet ports name rule.
     * use 72-fsl-dpaa-persistent-networking.rules for t1024, same as e6500.

     Signed-off-by: Ting Liu <ting.liu@freescale.com>

Since this change, I get this error when building:
  * check_data_file_clashes: Package udev-rules-qoriq wants to install file 
/local/diskstation-cutting-edge_2015-01-08/tmp/work/p1022ds-amltd-linux-gnuspe/amanda-server-image/1.0-r0/rootfs/etc/udev/mount.blacklist
         But that file is already provided by package  * udev-extraconf

I don't see why this was changed from using .bbappend for
the udev-extraconf - it seems wrong headed to me.

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: [meta-fsl-ppc] recent breakage

[Bob Cochran]

On 01/15/2015 12:46 PM, Gary Thomas wrote:
> This commit breaks a build when an image contains udev-extraconf
>
> commit d7df92b6f5c1a673341d54d6b064ad2fd7b16d74
> Author: Ting Liu <ting.liu@freescale.com>
> Date:   Wed Jan 7 17:54:59 2015 +0800
>
>      udev-extraconf: replace bbappend with own recipe
>
>      * introduce variable RULE for DPAA FMan ethernet ports name rule.
>      * use 72-fsl-dpaa-persistent-networking.rules for t1024, same as
> e6500.
>
>      Signed-off-by: Ting Liu <ting.liu@freescale.com>
>
> Since this change, I get this error when building:
>   * check_data_file_clashes: Package udev-rules-qoriq wants to install
> file
> /local/diskstation-cutting-edge_2015-01-08/tmp/work/p1022ds-amltd-linux-gnuspe/amanda-server-image/1.0-r0/rootfs/etc/udev/mount.blacklist
>
>          But that file is already provided by package  * udev-extraconf


I'm wondering where the root cause is located.  The recipe isn't 
installing the blacklist file.  It's just appending to it (if it already 
exists), so why is there a clash?  Does the recipe need modification or 
is the problem in the framework?

As an aside, should the udev-rules-qoriq recipe see if the blacklists 
already exist before issuing the append (echo) statements?

Lastly, I think I like having the separate recipe rather than the 
udev-extraconf append since I can easily just leave it out of my image, 
which I'm currently doing.

>
> I don't see why this was changed from using .bbappend for
> the udev-extraconf - it seems wrong headed to me.
>

Re: [meta-fsl-ppc] recent breakage

[Gary Thomas]

On 2015-01-15 12:38, Bob Cochran wrote:
> On 01/15/2015 12:46 PM, Gary Thomas wrote:
>> This commit breaks a build when an image contains udev-extraconf
>>
>> commit d7df92b6f5c1a673341d54d6b064ad2fd7b16d74
>> Author: Ting Liu <ting.liu@freescale.com>
>> Date:   Wed Jan 7 17:54:59 2015 +0800
>>
>>      udev-extraconf: replace bbappend with own recipe
>>
>>      * introduce variable RULE for DPAA FMan ethernet ports name rule.
>>      * use 72-fsl-dpaa-persistent-networking.rules for t1024, same as
>> e6500.
>>
>>      Signed-off-by: Ting Liu <ting.liu@freescale.com>
>>
>> Since this change, I get this error when building:
>>   * check_data_file_clashes: Package udev-rules-qoriq wants to install
>> file
>> /local/diskstation-cutting-edge_2015-01-08/tmp/work/p1022ds-amltd-linux-gnuspe/amanda-server-image/1.0-r0/rootfs/etc/udev/mount.blacklist
>>
>>          But that file is already provided by package  * udev-extraconf
>
>
> I'm wondering where the root cause is located.  The recipe isn't installing the blacklist file.  It's just appending to it (if it already exists), so why is there a clash?  Does
> the recipe need modification or is the problem in the framework?

One recipe doesn't get to modify the results of a different recipe
so the do_install() step is really creating a second copy of the
blacklist file (which if you don't have udev-extraconf installed
isn't used anyway)

>
> As an aside, should the udev-rules-qoriq recipe see if the blacklists already exist before issuing the append (echo) statements?
>
> Lastly, I think I like having the separate recipe rather than the udev-extraconf append since I can easily just leave it out of my image, which I'm currently doing.

Well I would like to have udev-extraconf in my image (that's
what choices are all about) and this version of udev-rules-qoriq
is making that impossible :-(

>
>>
>> I don't see why this was changed from using .bbappend for
>> the udev-extraconf - it seems wrong headed to me.
>>

-- 
------------------------------------------------------------
Gary Thomas                 |  Consulting for the
MLB Associates              |    Embedded world
------------------------------------------------------------

Re: [meta-fsl-ppc] recent breakage

[ting.liu]

> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-
> bounces@yoctoproject.org] On Behalf Of Gary Thomas
> Sent: Friday, January 16, 2015 3:52 AM
> To: Bob Cochran; meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc] recent breakage
> 
> On 2015-01-15 12:38, Bob Cochran wrote:
> > On 01/15/2015 12:46 PM, Gary Thomas wrote:
> >> This commit breaks a build when an image contains udev-extraconf
> >>
> >> commit d7df92b6f5c1a673341d54d6b064ad2fd7b16d74
> >> Author: Ting Liu <ting.liu@freescale.com>
> >> Date:   Wed Jan 7 17:54:59 2015 +0800
> >>
> >>      udev-extraconf: replace bbappend with own recipe
> >>
> >>      * introduce variable RULE for DPAA FMan ethernet ports name rule.
> >>      * use 72-fsl-dpaa-persistent-networking.rules for t1024, same as
> >> e6500.
> >>
> >>      Signed-off-by: Ting Liu <ting.liu@freescale.com>
> >>
> >> Since this change, I get this error when building:
> >>   * check_data_file_clashes: Package udev-rules-qoriq wants to
> >> install file
> >> /local/diskstation-cutting-edge_2015-01-08/tmp/work/p1022ds-amltd-lin
> >> ux-gnuspe/amanda-server-image/1.0-r0/rootfs/etc/udev/mount.blacklist
> >>
> >>          But that file is already provided by package  *
> >> udev-extraconf
> >
> >
> > I'm wondering where the root cause is located.  The recipe isn't
> > installing the blacklist file.  It's just appending to it (if it already
> exists), so why is there a clash?  Does the recipe need modification or is the
> problem in the framework?
> 
> One recipe doesn't get to modify the results of a different recipe so the
> do_install() step is really creating a second copy of the blacklist file
> (which if you don't have udev-extraconf installed isn't used anyway)
> 
> >
> > As an aside, should the udev-rules-qoriq recipe see if the blacklists
> already exist before issuing the append (echo) statements?
> >
> > Lastly, I think I like having the separate recipe rather than the udev-
> extraconf append since I can easily just leave it out of my image, which I'm
> currently doing.
> 
> Well I would like to have udev-extraconf in my image (that's what choices are
> all about) and this version of udev-rules-qoriq is making that impossible :-(
[Liu Ting-B28495] this is a bug, we will fix it ASAP.

> 
> >
> >>
> >> I don't see why this was changed from using .bbappend for the
> >> udev-extraconf - it seems wrong headed to me.
> >>
> 
> --
> ------------------------------------------------------------
> Gary Thomas                 |  Consulting for the
> MLB Associates              |    Embedded world
> ------------------------------------------------------------
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale

Re: [meta-fsl-ppc] recent breakage

[Abdur Rehman]

ting.liu@... <ting.liu@...> writes:

> [Liu Ting-B28495] this is a bug, we will fix it ASAP.

This thread looks old but I am experiencing same build breakage with the
current meta-fsl-ppc(commit 060e2779783a8e581cdace77bf833e4b80f63279).

Any status updates on the fix?

--
Abdur Rehman

Re: [meta-fsl-ppc] recent breakage

[Ahsan, Noor]

Any update this bug?

Noor

-----Original Message-----
From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-bounces@yoctoproject.org] On Behalf Of Rehman, Abdur
Sent: Thursday, August 06, 2015 5:14 PM
To: meta-freescale@yoctoproject.org
Subject: Re: [meta-freescale] [meta-fsl-ppc] recent breakage

ting.liu@... <ting.liu@...> writes:

> [Liu Ting-B28495] this is a bug, we will fix it ASAP.

This thread looks old but I am experiencing same build breakage with the current meta-fsl-ppc(commit 060e2779783a8e581cdace77bf833e4b80f63279).

Any status updates on the fix?

--
Abdur Rehman

--
_______________________________________________
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale

Re: [meta-fsl-ppc] recent breakage

[Liu Ting]

Previously a patch was sent to fix it. But it is rejected: http://patchwork.openembedded.org/patch/86927/
Seems that we lost the emails.

A new patch was sent out.

Thanks a lot for your reminder.

-Ting

> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-
> bounces@yoctoproject.org] On Behalf Of Ahsan, Noor
> Sent: Monday, August 10, 2015 8:33 PM
> To: Rehman, Abdur; meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc] recent breakage
> 
> Any update this bug?
> 
> Noor
> 
> -----Original Message-----
> From: meta-freescale-bounces@yoctoproject.org [mailto:meta-freescale-
> bounces@yoctoproject.org] On Behalf Of Rehman, Abdur
> Sent: Thursday, August 06, 2015 5:14 PM
> To: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc] recent breakage
> 
> ting.liu@... <ting.liu@...> writes:
> 
> > [Liu Ting-B28495] this is a bug, we will fix it ASAP.
> 
> This thread looks old but I am experiencing same build breakage with the
> current meta-fsl-ppc(commit 060e2779783a8e581cdace77bf833e4b80f63279).
> 
> Any status updates on the fix?
> 
> --
> Abdur Rehman
> 
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale
> --
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale