On Mon, Nov 19, 2012 at 4:12 PM, Saul Wold <s...@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 <s...@linux.intel.com
>> <mailto:s...@linux.intel.com>> wrote:
>>
>>     Rebase the patches due to macro name changes.
>>
>>     Signed-off-by: Saul Wold <s...@linux.intel.com
>>     <mailto:s...@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: ota...@ossystems.com.br  http://www.ossystems.com.br
Mobile: +55 53 9981-7854              http://projetos.ossystems.com.br
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-core

Reply via email to