Re: A patch for autotools.bbclas which correct .la file

Re: A patch for autotools.bbclas which correct .la file

[Otavio Salvador]

Julian Chu <julian_chu@openmoko.com> writes:

>         The attachment is a patch which works for me. Hopes it help.

Please fix indentation; it doesn't follow the code style.

-- 
        O T A V I O    S A L V A D O R
---------------------------------------------
 E-mail: otavio@debian.org      UIN: 5906116
 GNU/Linux User: 239058     GPG ID: 49A5F855
 Home Page: http://otavio.ossystems.com.br
---------------------------------------------
"Microsoft sells you Windows ... Linux gives
 you the whole house."

Re: A patch for autotools.bbclas which correct .la file

[Kalev Lember]

Julian Chu wrote:
>         It supposed to be replaced by autotools.bbclass. However, it
>         doesn't work when a recipe inherit autotools and overwrite do_install.
>   

Hello,

I think it might not be a good idea to do it in do_install_append(),
because lots of recipes which inherit autotools bbclass also define
do_install_append(). So the current approach would fix some, but break a
set of other packages.


-- 
Kalev

Re: A patch for autotools.bbclas which correct .la file

[Holger Freyther]

On Thursday 30 October 2008 11:57:24 Kalev Lember wrote:

> I think it might not be a good idea to do it in do_install_append(),
> because lots of recipes which inherit autotools bbclass also define
> do_install_append(). So the current approach would fix some, but break a
> set of other packages.

why? got an example?

z.

Re: A patch for autotools.bbclas which correct .la file

[Kalev Lember]

Holger Freyther wrote:
> On Thursday 30 October 2008 11:57:24 Kalev Lember wrote:
>   
>> I think it might not be a good idea to do it in do_install_append(),
>> because lots of recipes which inherit autotools bbclass also define
>> do_install_append(). So the current approach would fix some, but break a
>> set of other packages.
>>     
>
> why? got an example?
>   

Seems I was wrong about how _append is exactly handled. Please disregard
 my earlier mail.

Sorry for the noise.

-- 
Kalev

Re: A patch for autotools.bbclas which correct .la file

[Rodrigo Vivi]

+1

We have a similar patch on mamona tree, but this one seems more complete.

On Thu, Oct 30, 2008 at 10:40 AM, Kalev Lember <kalev@smartlink.ee> wrote:
> Holger Freyther wrote:
>> On Thursday 30 October 2008 11:57:24 Kalev Lember wrote:
>>
>>> I think it might not be a good idea to do it in do_install_append(),
>>> because lots of recipes which inherit autotools bbclass also define
>>> do_install_append(). So the current approach would fix some, but break a
>>> set of other packages.
>>>
>>
>> why? got an example?
>>
>
> Seems I was wrong about how _append is exactly handled. Please disregard
>  my earlier mail.
>
> Sorry for the noise.
>
> --
> Kalev
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>



-- 
Rodrigo Vivi
INdT - Instituto Nokia de Tecnologia
Blog: http://blog.vivi.eng.br
GPG: 0x905BE242 @ wwwkeys.pgp.net