On Mon, Oct 18, 2021 at 4:12 PM Martin Jansa <martin.ja...@gmail.com> wrote:
>
> On Mon, Oct 18, 2021 at 1:50 AM Andreas Müller <schnitzelt...@gmail.com> 
> wrote:
>>
>> On Fri, Oct 1, 2021 at 4:17 PM Martin Jansa <martin.ja...@gmail.com> wrote:
>> >
>> > FWIW: I've fixed some bigger layers where pkgconfig was causing quite a 
>> > few build failures,
>> >
>> Had an off-oe time due to  heavy work load.
>> Think this one is the reason for the pkgconfig patch flood. You are
>> aware that there are packages not failing at build time for missing
>> pkgconfig but build output is different and bugs will pop up at
>> runtime.
>>
>> To avoid responsible maintainers have to go through EVERY recipe (and
>> combination of PACKAGECONFIGs) and check if builds are still as
>> expected - Can't believe it!
>
>
> Not sure why this seems targeted to me.
>
> Yes I've tried to fix whatever issues this change caused in my world builds 
> and I'm partially responsible for pkgconfig patch flood, because I wanted 
> layers like meta-ros/*, meta-qt5, meta-qt6, meta-webosose/* to stay 
> compatible with oe-core whenever this change from RP is merged to master.
I am not against the flood of pkconfig patches I am against this patch.
>
> I've also tried to compare buildhistory and image content (for 
> ros-image-world with ros1-melodic) to see if I can spot more cases of 
> non-fatal build output changes as in collada-dom and unfortunately there is a 
> lot of noise in buildhistory (due to many recipes included in this image not 
> really build-reproducible), so I've used mostly installed-package-sizes.txt 
> file to spot 2 more recipes where the size was different (but both don't seem 
> to be caused by pkgconfig-native in the end).
>
> I did both builds from scratch without sstate, so I would assume that it will 
> catch all possible cases, but surprisingly just yesterday I've caught another 
> missing pkgconfig-native in one of meta-ros1-melodic recipes (caught in 
> kirkstone based build - while before I was testing on top of a bit older 
> honister with this change cherry-picked just for test - to prepare recipes to 
> be future-proof for kirkstone).
>
This was huge effort - and it sounds there is still no 100% confidence
to get same you had before - right? Others have to do same.

And for what? A build acceleration of how much - this patch does not
mention numbers. Or is it: Only the perfect recipes are allowed to
survive.

To me oe-core has decided (again) to make changes without taking care:
Break dozens of layers and force maintainers to spend days to get what
they had before this change. From image perspective nothing has
changed. And the very best: Some layers were just made for fun and
since this is definitely no fun they are going to die.

After this I have to work on my motivation spending time on this
project - not an easy task.

My opinion

Andreas
> Regards,
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#157086): 
https://lists.openembedded.org/g/openembedded-core/message/157086
Mute This Topic: https://lists.openembedded.org/mt/85739636/21656
Group Owner: openembedded-core+ow...@lists.openembedded.org
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to