[OE-core] [PATCH 16/17] cmake: Update to 2.8.10.1

[Otavio Salvador]

On Mon, Nov 19, 2012 at 4:12 PM, Saul Wold <sgw at linux.intel.com> wrote:

> On 11/19/2012 02:20 AM, Otavio Salvador wrote:
>
>> On Mon, Nov 19, 2012 at 3:54 AM, Saul Wold <sgw at linux.intel.com
>> <mailto:sgw at linux.intel.com>> wrote:
>>
>>     Rebase the patches due to macro name changes.
>>
>>     Signed-off-by: Saul Wold <sgw at linux.intel.com
>>     <mailto:sgw at linux.intel.com>>
>>
>>     ...
>>
>>     -PR = "${INC_PR}.0"
>>     -
>>     -SRC_URI[md5sum] = "**ba74b22c788a0c8547976b880cd02b**17"
>>     -SRC_URI[sha256sum] =
>>     "**2b59897864d6220ff20aa8eac64cac**8994e004898a1c0f899c8cb4d7b757**
>> 0b46"
>>
>>
>> I find it hard to spot the checksums to be in the .inc file.
>>
>> I'd prefer to have them in the .bb recipes (even if in this case it'd be
>> a duplication).
>>
>>  I am not sure why this is an issue, we already have about a dozen places
> where the SRC_URI and checksum info is located in the .inc file.


We have two recipes and nothing force them to be always in sync so the
checksum seems to be better placed in the .bb file


>
>  In case you would like to avoid the duplication (which I agree) you
>> might before, or later, merge the native handle onto the cmake_<ver>.bb
>> recipe.
>>
>> What do you think?
>>
>>  Looks like there are bigger issues with trying to merge native into the
> standard recipe, mostly due to the fact that the target recipe inherits
> cmake, while the native version does not.
>

Yes; I think I did a brief try in past and gave up.

So please, think about my comment above...

-- 
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20121119/f6c916af/attachment-0002.html>