* pkgconfig.bbclass changes
@ 2008-09-23 11:54 Koen Kooi
2008-09-23 12:22 ` Henning Heinold
0 siblings, 1 reply; 5+ messages in thread
From: Koen Kooi @ 2008-09-23 11:54 UTC (permalink / raw)
To: openembedded-devel
Hi,
Henning fixed up pkgconfig.bbclass (which should die, since
autotools.bbclass does a much better job), but I think this:
- for pc in `find ${S} -name '*.pc' -type f | grep -v --
'-uninstalled.pc$'`; do
+ for pc in `find ${D} -name '*.pc' -type f | grep -v --
'-uninstalled.pc$'`; do
breaks -native recipes.
regards,
Koen
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: pkgconfig.bbclass changes
2008-09-23 11:54 pkgconfig.bbclass changes Koen Kooi
@ 2008-09-23 12:22 ` Henning Heinold
2008-09-23 13:38 ` Koen Kooi
2008-09-23 17:52 ` Philip Balister
0 siblings, 2 replies; 5+ messages in thread
From: Henning Heinold @ 2008-09-23 12:22 UTC (permalink / raw)
To: openembedded-devel
On Tue, Sep 23, 2008 at 01:54:23PM +0200, Koen Kooi wrote:
> Hi,
>
> Henning fixed up pkgconfig.bbclass (which should die, since
> autotools.bbclass does a much better job), but I think this:
>
> - for pc in `find ${S} -name '*.pc' -type f | grep -v --
> '-uninstalled.pc$'`; do
> + for pc in `find ${D} -name '*.pc' -type f | grep -v --
> '-uninstalled.pc$'`; do
>
> breaks -native recipes.
>
> regards,
>
> Koen
Hi,
sorry you are right. This indeed breaks native packages I didnt
thought of it, while fixing the pc file of xine.
I think we can not let die pkgconfig.bbclass, because not all features are
in autotools.class and there are cases where staging or install
do not depend on the autotools install routines.
Bye Henning
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: pkgconfig.bbclass changes
2008-09-23 12:22 ` Henning Heinold
@ 2008-09-23 13:38 ` Koen Kooi
2008-09-23 17:52 ` Philip Balister
1 sibling, 0 replies; 5+ messages in thread
From: Koen Kooi @ 2008-09-23 13:38 UTC (permalink / raw)
To: openembedded-devel
Henning Heinold wrote:
> On Tue, Sep 23, 2008 at 01:54:23PM +0200, Koen Kooi wrote:
>> Hi,
>>
>> Henning fixed up pkgconfig.bbclass (which should die, since
>> autotools.bbclass does a much better job), but I think this:
>>
>> - for pc in `find ${S} -name '*.pc' -type f | grep -v --
>> '-uninstalled.pc$'`; do
>> + for pc in `find ${D} -name '*.pc' -type f | grep -v --
>> '-uninstalled.pc$'`; do
>>
>> breaks -native recipes.
>>
>> regards,
>>
>> Koen
>
> Hi,
>
> sorry you are right. This indeed breaks native packages I didnt
> thought of it, while fixing the pc file of xine.
I've been there myself: fixing bugs and then hearing people about
-native being broken.
> I think we can not let die pkgconfig.bbclass, because not all features are
> in autotools.class and there are cases where staging or install
> do not depend on the autotools install routines.
You are completely right about that, but we should move over autotools
based recipes to the autotools code and only use pkgconfig.bbclass for
non-autotooled recipes.
regards,
Koen
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: pkgconfig.bbclass changes
2008-09-23 12:22 ` Henning Heinold
2008-09-23 13:38 ` Koen Kooi
@ 2008-09-23 17:52 ` Philip Balister
2008-09-23 18:21 ` Koen Kooi
1 sibling, 1 reply; 5+ messages in thread
From: Philip Balister @ 2008-09-23 17:52 UTC (permalink / raw)
To: openembedded-devel
[-- Attachment #1: Type: text/plain, Size: 1145 bytes --]
Henning Heinold wrote:
> On Tue, Sep 23, 2008 at 01:54:23PM +0200, Koen Kooi wrote:
>> Hi,
>>
>> Henning fixed up pkgconfig.bbclass (which should die, since
>> autotools.bbclass does a much better job), but I think this:
>>
>> - for pc in `find ${S} -name '*.pc' -type f | grep -v --
>> '-uninstalled.pc$'`; do
>> + for pc in `find ${D} -name '*.pc' -type f | grep -v --
>> '-uninstalled.pc$'`; do
>>
>> breaks -native recipes.
>>
>> regards,
>>
>> Koen
>
> Hi,
>
> sorry you are right. This indeed breaks native packages I didnt
> thought of it, while fixing the pc file of xine.
Does anyone know what the proper fix is? .dev does not build at the moment.
Philip
> I think we can not let die pkgconfig.bbclass, because not all features are
> in autotools.class and there are cases where staging or install
> do not depend on the autotools install routines.
>
> Bye Henning
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>
[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/x-pkcs7-signature, Size: 3303 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: pkgconfig.bbclass changes
2008-09-23 17:52 ` Philip Balister
@ 2008-09-23 18:21 ` Koen Kooi
0 siblings, 0 replies; 5+ messages in thread
From: Koen Kooi @ 2008-09-23 18:21 UTC (permalink / raw)
To: openembedded-devel
Philip Balister wrote:
> Henning Heinold wrote:
>> On Tue, Sep 23, 2008 at 01:54:23PM +0200, Koen Kooi wrote:
>>> Hi,
>>>
>>> Henning fixed up pkgconfig.bbclass (which should die, since
>>> autotools.bbclass does a much better job), but I think this:
>>>
>>> - for pc in `find ${S} -name '*.pc' -type f | grep -v --
>>> '-uninstalled.pc$'`; do
>>> + for pc in `find ${D} -name '*.pc' -type f | grep -v --
>>> '-uninstalled.pc$'`; do
>>>
>>> breaks -native recipes.
>>>
>>> regards,
>>>
>>> Koen
>>
>> Hi,
>>
>> sorry you are right. This indeed breaks native packages I didnt
>> thought of it, while fixing the pc file of xine.
>
> Does anyone know what the proper fix is? .dev does not build at the moment.
revert the above hunk, bump PR on affected packages.
regards,
Koen
>
> Philip
>
>
>
>> I think we can not let die pkgconfig.bbclass, because not all features
>> are
>> in autotools.class and there are cases where staging or install
>> do not depend on the autotools install routines.
>>
>> Bye Henning
>>
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
>>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.linuxtogo.org/cgi-bin/mailman/listinfo/openembedded-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2008-09-23 18:25 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-09-23 11:54 pkgconfig.bbclass changes Koen Kooi
2008-09-23 12:22 ` Henning Heinold
2008-09-23 13:38 ` Koen Kooi
2008-09-23 17:52 ` Philip Balister
2008-09-23 18:21 ` Koen Kooi
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.