Patchwork [meta-fsl-ppc,v4,1/8] remove lm_sensor as it is already in meta-oe

login
register
mail settings
Submitter Ting Liu
Date March 9, 2013, 3:30 a.m.
Message ID <1362799812-16444-2-git-send-email-b28495@freescale.com>
Download mbox | patch
Permalink /patch/45767/
State Changes Requested
Delegated to: Otavio Salvador
Headers show

Comments

Ting Liu - March 9, 2013, 3:30 a.m.
From: Ting Liu <b28495@freescale.com>

Signed-off-by: Ting Liu <b28495@freescale.com>
---
 .../lm_sensors/lmsensors-apps_3.3.2.bb             |   50 --------------------
 1 files changed, 0 insertions(+), 50 deletions(-)
 delete mode 100644 recipes-extended/lm_sensors/lmsensors-apps_3.3.2.bb
Otavio Salvador - March 9, 2013, 12:21 p.m.
On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
> From: Ting Liu <b28495@freescale.com>
>
> Signed-off-by: Ting Liu <b28495@freescale.com>

I have just checked the revision used in meta-oe and it is 'r1' so
you'll need to keep an bbappend to add PRINC otherwise we'll lost
upgrade path.
Ting Liu - March 9, 2013, 2:37 p.m.
> -----Original Message-----
> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> Behalf Of Otavio Salvador
> Sent: Saturday, March 09, 2013 8:21 PM
> To: Liu Ting-B28495
> Cc: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> lm_sensor as it is already in meta-oe
> 
> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
> > From: Ting Liu <b28495@freescale.com>
> >
> > Signed-off-by: Ting Liu <b28495@freescale.com>
> 
> I have just checked the revision used in meta-oe and it is 'r1' so you'll
> need to keep an bbappend to add PRINC otherwise we'll lost upgrade path.
> 

It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and lmsensors_3.3.2.bb in 
meta-oe. But they are the same pkg. I think we can skip to bump PR as they are using 
'different' recipe name.
If it does not make sense, I will generate new patches.

-Ting

> --
> 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
Otavio Salvador - March 9, 2013, 2:53 p.m.
On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495 <B28495@freescale.com> wrote:
>> -----Original Message-----
>> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> Behalf Of Otavio Salvador
>> Sent: Saturday, March 09, 2013 8:21 PM
>> To: Liu Ting-B28495
>> Cc: meta-freescale@yoctoproject.org
>> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> lm_sensor as it is already in meta-oe
>>
>> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
>> > From: Ting Liu <b28495@freescale.com>
>> >
>> > Signed-off-by: Ting Liu <b28495@freescale.com>
>>
>> I have just checked the revision used in meta-oe and it is 'r1' so you'll
>> need to keep an bbappend to add PRINC otherwise we'll lost upgrade path.
>>
>
> It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and lmsensors_3.3.2.bb in
> meta-oe. But they are the same pkg. I think we can skip to bump PR as they are using
> 'different' recipe name.
> If it does not make sense, I will generate new patches.

Regarding the PR you're right, good catch! However we need to take
care of binary packages. You should provide a
RPROVIDES/RREPLACES/RCONFLICTS for the binary packages which were
renamed so the upgrade path works.

--
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
Ting Liu - March 9, 2013, 3:21 p.m.
> -----Original Message-----
> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> Behalf Of Otavio Salvador
> Sent: Saturday, March 09, 2013 10:54 PM
> To: Liu Ting-B28495
> Cc: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> lm_sensor as it is already in meta-oe
> 
> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495 <B28495@freescale.com>
> wrote:
> >> -----Original Message-----
> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> >> Behalf Of Otavio Salvador
> >> Sent: Saturday, March 09, 2013 8:21 PM
> >> To: Liu Ting-B28495
> >> Cc: meta-freescale@yoctoproject.org
> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> >> lm_sensor as it is already in meta-oe
> >>
> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
> >> > From: Ting Liu <b28495@freescale.com>
> >> >
> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
> >>
> >> I have just checked the revision used in meta-oe and it is 'r1' so
> >> you'll need to keep an bbappend to add PRINC otherwise we'll lost
> upgrade path.
> >>
> >
> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I think we
> > can skip to bump PR as they are using 'different' recipe name.
> > If it does not make sense, I will generate new patches.
> 
> Regarding the PR you're right, good catch! However we need to take care
> of binary packages. You should provide a RPROVIDES/RREPLACES/RCONFLICTS
> for the binary packages which were renamed so the upgrade path works.
> 

I think we can skip it too. They are identical, just with different name. 
RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will install 
the two at the same time. They will find lmsensors-apps has gone away and 
use the one in meta-oe.

-Ting

> --
> 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
Otavio Salvador - March 9, 2013, 3:33 p.m.
On Sat, Mar 9, 2013 at 12:21 PM, Liu Ting-B28495 <B28495@freescale.com> wrote:
>> -----Original Message-----
>> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> Behalf Of Otavio Salvador
>> Sent: Saturday, March 09, 2013 10:54 PM
>> To: Liu Ting-B28495
>> Cc: meta-freescale@yoctoproject.org
>> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> lm_sensor as it is already in meta-oe
>>
>> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495 <B28495@freescale.com>
>> wrote:
>> >> -----Original Message-----
>> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> >> Behalf Of Otavio Salvador
>> >> Sent: Saturday, March 09, 2013 8:21 PM
>> >> To: Liu Ting-B28495
>> >> Cc: meta-freescale@yoctoproject.org
>> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> >> lm_sensor as it is already in meta-oe
>> >>
>> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
>> >> > From: Ting Liu <b28495@freescale.com>
>> >> >
>> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
>> >>
>> >> I have just checked the revision used in meta-oe and it is 'r1' so
>> >> you'll need to keep an bbappend to add PRINC otherwise we'll lost
>> upgrade path.
>> >>
>> >
>> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
>> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I think we
>> > can skip to bump PR as they are using 'different' recipe name.
>> > If it does not make sense, I will generate new patches.
>>
>> Regarding the PR you're right, good catch! However we need to take care
>> of binary packages. You should provide a RPROVIDES/RREPLACES/RCONFLICTS
>> for the binary packages which were renamed so the upgrade path works.
>>
>
> I think we can skip it too. They are identical, just with different name.
> RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will install
> the two at the same time. They will find lmsensors-apps has gone away and
> use the one in meta-oe.

If someone has an image and has a lmsensors-apps-<foo> it will fail to
build. That's why you should have the overrides so the images will not
need to be changed to be kept working.
Ting Liu - March 9, 2013, 3:37 p.m.
> -----Original Message-----
> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> Behalf Of Otavio Salvador
> Sent: Saturday, March 09, 2013 11:34 PM
> To: Liu Ting-B28495
> Cc: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> lm_sensor as it is already in meta-oe
> 
> On Sat, Mar 9, 2013 at 12:21 PM, Liu Ting-B28495 <B28495@freescale.com>
> wrote:
> >> -----Original Message-----
> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> >> Behalf Of Otavio Salvador
> >> Sent: Saturday, March 09, 2013 10:54 PM
> >> To: Liu Ting-B28495
> >> Cc: meta-freescale@yoctoproject.org
> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> >> lm_sensor as it is already in meta-oe
> >>
> >> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495
> >> <B28495@freescale.com>
> >> wrote:
> >> >> -----Original Message-----
> >> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com]
> >> >> On Behalf Of Otavio Salvador
> >> >> Sent: Saturday, March 09, 2013 8:21 PM
> >> >> To: Liu Ting-B28495
> >> >> Cc: meta-freescale@yoctoproject.org
> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> >> >> lm_sensor as it is already in meta-oe
> >> >>
> >> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
> >> >> > From: Ting Liu <b28495@freescale.com>
> >> >> >
> >> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
> >> >>
> >> >> I have just checked the revision used in meta-oe and it is 'r1' so
> >> >> you'll need to keep an bbappend to add PRINC otherwise we'll lost
> >> upgrade path.
> >> >>
> >> >
> >> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
> >> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I think
> >> > we can skip to bump PR as they are using 'different' recipe name.
> >> > If it does not make sense, I will generate new patches.
> >>
> >> Regarding the PR you're right, good catch! However we need to take
> >> care of binary packages. You should provide a
> >> RPROVIDES/RREPLACES/RCONFLICTS for the binary packages which were
> renamed so the upgrade path works.
> >>
> >
> > I think we can skip it too. They are identical, just with different
> name.
> > RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will
> > install the two at the same time. They will find lmsensors-apps has
> > gone away and use the one in meta-oe.
> 
> If someone has an image and has a lmsensors-apps-<foo> it will fail to
> build. That's why you should have the overrides so the images will not
> need to be changed to be kept working.

So they will know the changes and use the right pkg name in image :-)

-Ting

> 
> --
> 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
Otavio Salvador - March 9, 2013, 3:40 p.m.
On Sat, Mar 9, 2013 at 12:37 PM, Liu Ting-B28495 <B28495@freescale.com> wrote:
>> -----Original Message-----
>> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> Behalf Of Otavio Salvador
>> Sent: Saturday, March 09, 2013 11:34 PM
>> To: Liu Ting-B28495
>> Cc: meta-freescale@yoctoproject.org
>> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> lm_sensor as it is already in meta-oe
>>
>> On Sat, Mar 9, 2013 at 12:21 PM, Liu Ting-B28495 <B28495@freescale.com>
>> wrote:
>> >> -----Original Message-----
>> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> >> Behalf Of Otavio Salvador
>> >> Sent: Saturday, March 09, 2013 10:54 PM
>> >> To: Liu Ting-B28495
>> >> Cc: meta-freescale@yoctoproject.org
>> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> >> lm_sensor as it is already in meta-oe
>> >>
>> >> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495
>> >> <B28495@freescale.com>
>> >> wrote:
>> >> >> -----Original Message-----
>> >> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com]
>> >> >> On Behalf Of Otavio Salvador
>> >> >> Sent: Saturday, March 09, 2013 8:21 PM
>> >> >> To: Liu Ting-B28495
>> >> >> Cc: meta-freescale@yoctoproject.org
>> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> >> >> lm_sensor as it is already in meta-oe
>> >> >>
>> >> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
>> >> >> > From: Ting Liu <b28495@freescale.com>
>> >> >> >
>> >> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
>> >> >>
>> >> >> I have just checked the revision used in meta-oe and it is 'r1' so
>> >> >> you'll need to keep an bbappend to add PRINC otherwise we'll lost
>> >> upgrade path.
>> >> >>
>> >> >
>> >> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
>> >> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I think
>> >> > we can skip to bump PR as they are using 'different' recipe name.
>> >> > If it does not make sense, I will generate new patches.
>> >>
>> >> Regarding the PR you're right, good catch! However we need to take
>> >> care of binary packages. You should provide a
>> >> RPROVIDES/RREPLACES/RCONFLICTS for the binary packages which were
>> renamed so the upgrade path works.
>> >>
>> >
>> > I think we can skip it too. They are identical, just with different
>> name.
>> > RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will
>> > install the two at the same time. They will find lmsensors-apps has
>> > gone away and use the one in meta-oe.
>>
>> If someone has an image and has a lmsensors-apps-<foo> it will fail to
>> build. That's why you should have the overrides so the images will not
>> need to be changed to be kept working.
>
> So they will know the changes and use the right pkg name in image :-)

This breaks user image; we ought to avoid to do that. People using the
BSP/SDK are kept working nowadays so you're moving them to a new
package so please take care to not break them.

I won't ack the package without a proper upgrade path. Matt, your call.
Ting Liu - March 11, 2013, 2:10 a.m.
> -----Original Message-----
> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> Behalf Of Otavio Salvador
> Sent: Saturday, March 09, 2013 11:40 PM
> To: Liu Ting-B28495
> Cc: meta-freescale@yoctoproject.org
> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> lm_sensor as it is already in meta-oe
> 
> On Sat, Mar 9, 2013 at 12:37 PM, Liu Ting-B28495 <B28495@freescale.com>
> wrote:
> >> -----Original Message-----
> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
> >> Behalf Of Otavio Salvador
> >> Sent: Saturday, March 09, 2013 11:34 PM
> >> To: Liu Ting-B28495
> >> Cc: meta-freescale@yoctoproject.org
> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> >> lm_sensor as it is already in meta-oe
> >>
> >> On Sat, Mar 9, 2013 at 12:21 PM, Liu Ting-B28495
> >> <B28495@freescale.com>
> >> wrote:
> >> >> -----Original Message-----
> >> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com]
> >> >> On Behalf Of Otavio Salvador
> >> >> Sent: Saturday, March 09, 2013 10:54 PM
> >> >> To: Liu Ting-B28495
> >> >> Cc: meta-freescale@yoctoproject.org
> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
> >> >> lm_sensor as it is already in meta-oe
> >> >>
> >> >> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495
> >> >> <B28495@freescale.com>
> >> >> wrote:
> >> >> >> -----Original Message-----
> >> >> >> From: otavio.salvador@gmail.com
> >> >> >> [mailto:otavio.salvador@gmail.com]
> >> >> >> On Behalf Of Otavio Salvador
> >> >> >> Sent: Saturday, March 09, 2013 8:21 PM
> >> >> >> To: Liu Ting-B28495
> >> >> >> Cc: meta-freescale@yoctoproject.org
> >> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8]
> >> >> >> remove lm_sensor as it is already in meta-oe
> >> >> >>
> >> >> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
> >> >> >> > From: Ting Liu <b28495@freescale.com>
> >> >> >> >
> >> >> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
> >> >> >>
> >> >> >> I have just checked the revision used in meta-oe and it is 'r1'
> >> >> >> so you'll need to keep an bbappend to add PRINC otherwise we'll
> >> >> >> lost
> >> >> upgrade path.
> >> >> >>
> >> >> >
> >> >> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
> >> >> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I
> >> >> > think we can skip to bump PR as they are using 'different' recipe
> name.
> >> >> > If it does not make sense, I will generate new patches.
> >> >>
> >> >> Regarding the PR you're right, good catch! However we need to take
> >> >> care of binary packages. You should provide a
> >> >> RPROVIDES/RREPLACES/RCONFLICTS for the binary packages which were
> >> renamed so the upgrade path works.
> >> >>
> >> >
> >> > I think we can skip it too. They are identical, just with different
> >> name.
> >> > RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will
> >> > install the two at the same time. They will find lmsensors-apps has
> >> > gone away and use the one in meta-oe.
> >>
> >> If someone has an image and has a lmsensors-apps-<foo> it will fail
> >> to build. That's why you should have the overrides so the images will
> >> not need to be changed to be kept working.
> >
> > So they will know the changes and use the right pkg name in image :-)
> 
> This breaks user image; we ought to avoid to do that. People using the
> BSP/SDK are kept working nowadays so you're moving them to a new package
> so please take care to not break them.

Consider this case: people include poky, meta-fsl-ppc and other layers except meta-oe. If we remove lmsensors-apps_3.3.2.bb in meta-fsl-ppc, then their image that has lmsensors-apps will absolutely fail to build. I wonder how to get a proper upgrade path in this case? Dose RPROVIDES/RREPLACES/RCONFLICTS works? They have to do some changes.

-Ting

> 
> I won't ack the package without a proper upgrade path. Matt, your call.

All right.
RPROVIDES/RREPLACES/RCONFLICTS
> 
> --
> 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
Matthew McClintock - March 12, 2013, 4:21 a.m.
Ting, Otavio,

I think Otavio is referring to binary feeds to some degree here? Where
in a package has already generated an ipk/deb/rpm and it's being
upgraded in a live install. It's worth noting that some SDKs use this
method to upgrading / deploying vs. doing a build from source that
poky does.

With that in mind, given the package has changed we may need a
RREPLACES? I'm really not that familiar with this and I don't know
that anyone that uses ppc does this method yet. Do any SDKs/DISTROs
out there do this with meta-fsl-ppc? If so speak up, or let me know if
I am completely off base.

Also, please CC me on the patches and commentary so I see it sooner.

-M

On Sun, Mar 10, 2013 at 9:10 PM, Liu Ting-B28495 <B28495@freescale.com> wrote:
>> -----Original Message-----
>> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> Behalf Of Otavio Salvador
>> Sent: Saturday, March 09, 2013 11:40 PM
>> To: Liu Ting-B28495
>> Cc: meta-freescale@yoctoproject.org
>> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> lm_sensor as it is already in meta-oe
>>
>> On Sat, Mar 9, 2013 at 12:37 PM, Liu Ting-B28495 <B28495@freescale.com>
>> wrote:
>> >> -----Original Message-----
>> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com] On
>> >> Behalf Of Otavio Salvador
>> >> Sent: Saturday, March 09, 2013 11:34 PM
>> >> To: Liu Ting-B28495
>> >> Cc: meta-freescale@yoctoproject.org
>> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> >> lm_sensor as it is already in meta-oe
>> >>
>> >> On Sat, Mar 9, 2013 at 12:21 PM, Liu Ting-B28495
>> >> <B28495@freescale.com>
>> >> wrote:
>> >> >> -----Original Message-----
>> >> >> From: otavio.salvador@gmail.com [mailto:otavio.salvador@gmail.com]
>> >> >> On Behalf Of Otavio Salvador
>> >> >> Sent: Saturday, March 09, 2013 10:54 PM
>> >> >> To: Liu Ting-B28495
>> >> >> Cc: meta-freescale@yoctoproject.org
>> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8] remove
>> >> >> lm_sensor as it is already in meta-oe
>> >> >>
>> >> >> On Sat, Mar 9, 2013 at 11:37 AM, Liu Ting-B28495
>> >> >> <B28495@freescale.com>
>> >> >> wrote:
>> >> >> >> -----Original Message-----
>> >> >> >> From: otavio.salvador@gmail.com
>> >> >> >> [mailto:otavio.salvador@gmail.com]
>> >> >> >> On Behalf Of Otavio Salvador
>> >> >> >> Sent: Saturday, March 09, 2013 8:21 PM
>> >> >> >> To: Liu Ting-B28495
>> >> >> >> Cc: meta-freescale@yoctoproject.org
>> >> >> >> Subject: Re: [meta-freescale] [meta-fsl-ppc][PATCH v4 1/8]
>> >> >> >> remove lm_sensor as it is already in meta-oe
>> >> >> >>
>> >> >> >> On Sat, Mar 9, 2013 at 12:30 AM,  <b28495@freescale.com> wrote:
>> >> >> >> > From: Ting Liu <b28495@freescale.com>
>> >> >> >> >
>> >> >> >> > Signed-off-by: Ting Liu <b28495@freescale.com>
>> >> >> >>
>> >> >> >> I have just checked the revision used in meta-oe and it is 'r1'
>> >> >> >> so you'll need to keep an bbappend to add PRINC otherwise we'll
>> >> >> >> lost
>> >> >> upgrade path.
>> >> >> >>
>> >> >> >
>> >> >> > It is named as lmsensors-apps_3.3.2.bb in meta-fsl-ppc, and
>> >> >> > lmsensors_3.3.2.bb in meta-oe. But they are the same pkg. I
>> >> >> > think we can skip to bump PR as they are using 'different' recipe
>> name.
>> >> >> > If it does not make sense, I will generate new patches.
>> >> >>
>> >> >> Regarding the PR you're right, good catch! However we need to take
>> >> >> care of binary packages. You should provide a
>> >> >> RPROVIDES/RREPLACES/RCONFLICTS for the binary packages which were
>> >> renamed so the upgrade path works.
>> >> >>
>> >> >
>> >> > I think we can skip it too. They are identical, just with different
>> >> name.
>> >> > RPROVIDES/RREPLACES/RCONFLICTS does not make sense. No one will
>> >> > install the two at the same time. They will find lmsensors-apps has
>> >> > gone away and use the one in meta-oe.
>> >>
>> >> If someone has an image and has a lmsensors-apps-<foo> it will fail
>> >> to build. That's why you should have the overrides so the images will
>> >> not need to be changed to be kept working.
>> >
>> > So they will know the changes and use the right pkg name in image :-)
>>
>> This breaks user image; we ought to avoid to do that. People using the
>> BSP/SDK are kept working nowadays so you're moving them to a new package
>> so please take care to not break them.
>
> Consider this case: people include poky, meta-fsl-ppc and other layers except meta-oe. If we remove lmsensors-apps_3.3.2.bb in meta-fsl-ppc, then their image that has lmsensors-apps will absolutely fail to build. I wonder how to get a proper upgrade path in this case? Dose RPROVIDES/RREPLACES/RCONFLICTS works? They have to do some changes.
>
> -Ting
>
>>
>> I won't ack the package without a proper upgrade path. Matt, your call.
>
> All right.
> RPROVIDES/RREPLACES/RCONFLICTS
>>
>> --
>> 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
>
>
> _______________________________________________
> meta-freescale mailing list
> meta-freescale@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-freescale
Otavio Salvador - March 12, 2013, 12:43 p.m.
On Tue, Mar 12, 2013 at 1:21 AM, Matthew McClintock
<msm-oss@mcclintock.net> wrote:
> Ting, Otavio,
>
> I think Otavio is referring to binary feeds to some degree here? Where
> in a package has already generated an ipk/deb/rpm and it's being
> upgraded in a live install. It's worth noting that some SDKs use this
> method to upgrading / deploying vs. doing a build from source that
> poky does.

Yes; correct.

> With that in mind, given the package has changed we may need a
> RREPLACES? I'm really not that familiar with this and I don't know
> that anyone that uses ppc does this method yet. Do any SDKs/DISTROs
> out there do this with meta-fsl-ppc? If so speak up, or let me know if
> I am completely off base.

RPROVIDES, RCONFLICTS and RREPLACES.

> Also, please CC me on the patches and commentary so I see it sooner.

Will do :-) Thanks :-D

Patch

diff --git a/recipes-extended/lm_sensors/lmsensors-apps_3.3.2.bb b/recipes-extended/lm_sensors/lmsensors-apps_3.3.2.bb
deleted file mode 100644
index 62da814..0000000
--- a/recipes-extended/lm_sensors/lmsensors-apps_3.3.2.bb
+++ /dev/null
@@ -1,50 +0,0 @@ 
-SUMMARY = "A hardware health monitoring package for Linux"
-DESCRIPTION = "Lm-sensors is a hardware health monitoring package for Linux. \
-               It allows you to access information from temperature, voltage, \
-               and fan speed sensors."
-HOMEPAGE = "http://www.lm-sensors.org/"
-DEPENDS = "sysfsutils virtual/libiconv \
-           bison-native flex-native"
-LICENSE = "GPLv2 & LGPLv2.1"
-LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe \
-                    file://COPYING.LGPL;md5=4fbd65380cdd255951079008b364516c"
-
-PR = "r2"
-PACKAGE_ARCH = "${MACHINE_ARCH}"
-
-SRC_URI = "http://dl.lm-sensors.org/lm-sensors/releases/lm_sensors-${PV}.tar.bz2"
-
-SRC_URI[md5sum] = "f357ba00b080ab102a170f7bf8bb2578"
-SRC_URI[sha256sum] = "f13dd885406841a7352ccfb8b9ccb23c4c057abe3de4258da5444c149a9e3ae1"
-
-S = "${WORKDIR}/lm_sensors-${PV}"
-
-EXTRA_OEMAKE = 'LINUX=${STAGING_KERNEL_DIR} EXLDFLAGS="${LDFLAGS}" \
-		MACHINE=${TARGET_ARCH} PREFIX=${prefix} CC="${CC}" \
-		AR="${AR}" MANDIR=${mandir}'
-
-do_compile() {
-	oe_runmake user PROG_EXTRA=sensors
-}
-
-do_install() {
-	oe_runmake user_install DESTDIR=${D}
-}
-
-PACKAGES =+ "libsensors libsensors-dev libsensors-staticdev libsensors-dbg libsensors-doc"
-PACKAGES =+ "lmsensors-sensors lmsensors-sensors-dbg lmsensors-sensors-doc"
-PACKAGES =+ "lmsensors-scripts"
-
-FILES_lmsensors-scripts = "${bindir}/*.pl ${bindir}/ddcmon ${sbindir}/fancontrol* ${sbindir}/pwmconfig ${sbindir}/sensors-detect"
-RDEPENDS_lmsensors-scripts += "lmsensors-sensors perl bash"
-RDEPENDS_lmsensors-apps += "perl-module-strict perl-module-vars perl-module-warnings-register perl-module-warnings"
-RDEPENDS_lmsensors-scripts += "perl-module-fcntl perl-module-exporter perl-module-xsloader perl-module-exporter-heavy perl-module-file-basename perl-module-constant"
-
-FILES_lmsensors-sensors = "${bindir}/sensors ${sysconfdir}"
-FILES_lmsensors-sensors-dbg += "${bindir}/.debug/sensors"
-FILES_lmsensors-sensors-doc = "${mandir}/man1 ${mandir}/man5"
-FILES_libsensors = "${libdir}/libsensors.so.*"
-FILES_libsensors-dbg += "${libdir}/.debug"
-FILES_libsensors-dev = "${libdir}/libsensors.so ${includedir}"
-FILES_libsensors-staticdev = "${libdir}/libsensors.a"
-FILES_libsensors-doc = "${mandir}/man3"