RE: [bitbake-devel][PATCH V2 2/5] fetch2/wget.py: add filter_regex parameter to latest_versionstring

"Chen, Qi" <[email protected]>
Newsgroups org.openembedded.lists.bitbake-devel
Message-ID <CO6PR11MB5602943EB4E823A8DD375DFEED3D2@CO6PR11MB5602.namprd11.prod.outlook.com>
I'll add note and simplify the function calls.

Regards,
Qi

-----Original Message-----
From: Alexander Kanavin <[email protected]> 
Sent: Thursday, May 7, 2026 6:43 PM
To: Chen, Qi <[email protected]>
Cc: [email protected]; MacLeod, Randy <[email protected]>
Subject: Re: [bitbake-devel][PATCH V2 2/5] fetch2/wget.py: add filter_regex parameter to latest_versionstring

On Thu, 7 May 2026 at 08:19, <[email protected]> wrote:
> +                if filter_regex:
> +                    if not re.match(filter_regex, re.sub('_', '.', newver[1])):

This needs to include a note explaining that filter_regex is derived from PV, and so uses . as separator, while upstream may be using _ as separator (as seen below in the code where the same conversion happens before the discovered version is returned).

> -                            package, package_regex, current_version, ud, d)
> +                            package, package_regex, current_version, 
> + ud, d, filter_regex=filter_regex)

Here and in other places you can perhaps simply pass in filter_regex without naming the argument?

Alex
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.