From: "Yoann Congal" <yoann.congal@smile.fr>
To: "Marko, Peter" <Peter.Marko@siemens.com>
Cc: "openembedded-core@lists.openembedded.org"
<openembedded-core@lists.openembedded.org>,
"Alexander Kanavin" <alex.kanavin@gmail.com>,
"Paul Barker" <paul@pbarker.dev>
Subject: Re: [OE-core] [wrynose][PATCH 1/3] libslirp: fix upstream version check
Date: Mon, 14 Sep 2026 16:18:34 +0200 [thread overview]
Message-ID: <DLF3RQZ76CZK.C0FPH5HA196G@smile.fr> (raw)
In-Reply-To: <AS8PR10MB507373EE139A44A2BC291050FDBD2@AS8PR10MB5073.EURPRD10.PROD.OUTLOOK.COM>
On Sat Sep 12, 2026 at 6:12 PM CEST, Peter Marko wrote:
>
>
>> -----Original Message-----
>> From: Alexander Kanavin <alex.kanavin@gmail.com>
>> Sent: Friday, September 4, 2026 2:39 PM
>> To: yoann.congal@smile.fr
>> Cc: Marko, Peter (FT D EU SK BFS1) <Peter.Marko@siemens.com>;
>> openembedded-core@lists.openembedded.org
>> Subject: Re: [OE-core] [wrynose][PATCH 1/3] libslirp: fix upstream version check
>>
>> On Thu, 3 Sept 2026 at 22:45, Yoann Congal via lists.openembedded.org
>> <yoann.congal=smile.fr@lists.openembedded.org> wrote:
>> > Thanks for the series. But, with regrets, I don't think it is acceptable
>> > under the policy:
>> > * 1&3/3 are not bug fixes (UPSTREAM_CHECK_GITTAGREGEX is not used right
>> > now on stables)
>
> I don't really insist on this commit being backported.
> However, it would be very beneficial for future cherry-picks.
> Since this line is next to both SRC_URI and SRCREV, any cherry-pick results in hard conflict.
>
> I'd like to keep the series as is, but I can also send it without this commit if requested.
> Please let me know Yoann.
Hello,
I've discussed this with Paul and we are goind to take this series
through review: Every changes are low-risks and have value, so, in this
case I was overly cautious.
Expect this in the next wrynose series (this week).
Regards,
>
> Peter
>
>>
>> Being able to check upstream versions is a valid use case for stable
>> branches too, even if the functionality is not currently used by Yocto
>> project itself. Users can check what the latest (or latest stable,
>> if/when that support is backported) version is, and make their own
>> decision about updating it downstream.
>>
>> It's a *very* low risk change as well, as it is used only in the
>> version check and nowhere in actual builds.
>>
>> Alex
--
Yoann Congal
Smile ECS
prev parent reply other threads:[~2026-09-14 14:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-23 15:57 [wrynose][PATCH 1/3] libslirp: fix upstream version check Peter Marko
2026-08-23 15:57 ` [wrynose][PATCH 2/3] libslirp: upgrade 4.9.1 -> 4.9.3 Peter Marko
2026-08-23 15:57 ` [wrynose][PATCH 3/3] libslirp: add tag in SRC_URI Peter Marko
2026-09-03 20:45 ` [OE-core] [wrynose][PATCH 1/3] libslirp: fix upstream version check Yoann Congal
2026-09-04 12:38 ` Alexander Kanavin
2026-09-12 16:12 ` Marko, Peter
2026-09-14 14:18 ` Yoann Congal [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DLF3RQZ76CZK.C0FPH5HA196G@smile.fr \
--to=yoann.congal@smile.fr \
--cc=Peter.Marko@siemens.com \
--cc=alex.kanavin@gmail.com \
--cc=openembedded-core@lists.openembedded.org \
--cc=paul@pbarker.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.