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

[Otavio Salvador]

On Sat, Mar 9, 2013 at 12:37 PM, Liu Ting-B28495 <B28495 at freescale.com> wrote:
>> -----Original Message-----
>> From: otavio.salvador at gmail.com [mailto:otavio.salvador at gmail.com] On
>> Behalf Of Otavio Salvador
>> Sent: Saturday, March 09, 2013 11:34 PM
>> To: Liu Ting-B28495
>> Cc: meta-freescale at 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 at freescale.com>
>> wrote:
>> >> -----Original Message-----
>> >> From: otavio.salvador at gmail.com [mailto:otavio.salvador at gmail.com] On
>> >> Behalf Of Otavio Salvador
>> >> Sent: Saturday, March 09, 2013 10:54 PM
>> >> To: Liu Ting-B28495
>> >> Cc: meta-freescale at 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 at freescale.com>
>> >> wrote:
>> >> >> -----Original Message-----
>> >> >> From: otavio.salvador at gmail.com [mailto:otavio.salvador at gmail.com]
>> >> >> On Behalf Of Otavio Salvador
>> >> >> Sent: Saturday, March 09, 2013 8:21 PM
>> >> >> To: Liu Ting-B28495
>> >> >> Cc: meta-freescale at 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 at freescale.com> wrote:
>> >> >> > From: Ting Liu <b28495 at freescale.com>
>> >> >> >
>> >> >> > Signed-off-by: Ting Liu <b28495 at 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.

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