"Option three: both patches are correct :)"

I agree in general, but in this particular case both patches are logically
interfering with each other (one patch adds a PACKAGECONFIG option for
USE_ACL, and the other patch renames USE_ACL to WITH_ACL).
I will wait a couple days in the hope that my opkg patch gets
reviewed/merged upstream and then send a backport to oe-core.

Étienne




On Thu, Nov 27, 2025 at 7:37 PM Alexander Kanavin <[email protected]>
wrote:

> On Thu, 27 Nov 2025 at 19:26, Gyorgy Sarvari via
> lists.openembedded.org <[email protected]>
> wrote:
> >
> > On 11/27/25 19:22, Etienne Cordonnier wrote:
> > > Hi Gyorgy,
> > > I have sent a patch to opkg upstream to fix this issue
> > > (
> https://urldefense.proofpoint.com/v2/url?u=https-3A__lists.yoctoproject.org_g_opkg_message_175&d=DwIBaQ&c=ncDTmphkJTvjIDPh0hpF_4vCHvabgGkICC2epckfdiw&r=AhkbNonVuMIGRfPx_Qj9TsRih1DULJTKUkSGa66m67E&m=skv6vITjRx1biFf-pISzu3mA39-5qXt1NoaexzWpNijLhnsMTj14xC_aJdOuQ9uD&s=tbZyO5O988bK4-b-uyVVvSeBbZ4hTvKf9TQGPps-bhQ&e=
> ).
> > > I think it would be better to either:
> > > - add my opkg patch to oe-core and not merge yours
> > > - or to keep your patch and not add mine, but at least add those lines
> > > as os-core patch:
> > Please feel free to send your patch - I just sent this as one possible
> > solution, but I am not particularly attached to it :)
>
> Option three: both patches are correct :) Regardless of upstream
> behaviour, adding missing packageconfigs to recipes is always
> desirable as it lowers the chance of build non-determinism and
> floating configurations.
>
> Alex
>
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#226901): 
https://lists.openembedded.org/g/openembedded-core/message/226901
Mute This Topic: https://lists.openembedded.org/mt/116504167/21656
Group Owner: [email protected]
Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub 
[[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to