Unfortunately this isn't correct either. UPSTREAM_CHECK_URI is already checked by the wget fetcher to override the default (which is first entry in SRC_URI), so if you need it in other fetchers (e.g git), you should either add support for it there as well, or remove the code from wget fetcher at the same time. Otherwise, this change will make things very inconsistent and confusing.
Alex On Thu, 28 Mar 2024 at 16:32, Jon Mason <[email protected]> wrote: > > Currently, get_recipe_upstream_version blindly takes the first entry in > SRC_URI to see if the recipe is at the latest version. If > UPSTREAM_CHECK_URI is specified in a recipe, it is probably what should > be used to check for the latest version. Use that as the first check, > otherwise default back to the first entry in SRC_URI. > > Signed-off-by: Jon Mason <[email protected]> > --- > meta/lib/oe/recipeutils.py | 9 +++++++-- > 1 file changed, 7 insertions(+), 2 deletions(-) > > diff --git a/meta/lib/oe/recipeutils.py b/meta/lib/oe/recipeutils.py > index de1fbdd3a8c8..a42609060cd9 100644 > --- a/meta/lib/oe/recipeutils.py > +++ b/meta/lib/oe/recipeutils.py > @@ -1041,8 +1041,13 @@ def get_recipe_upstream_version(rd): > ru['datetime'] = datetime.now() > return ru > > - # XXX: we suppose that the first entry points to the upstream sources > - src_uri = src_uris.split()[0] > + # If UPSTREAM_CHECK_URI is specified, assume it is correct and use > + # it. Otherwise, use the first SRC_URI specified to determine the > + # latest version. > + if rd.getVar('UPSTREAM_CHECK_URI'): > + src_uri = str(rd.getVar('UPSTREAM_CHECK_URI')) > + else: > + src_uri = src_uris.split()[0] > uri_type, _, _, _, _, _ = decodeurl(src_uri) > > (pv, pfx, sfx) = get_recipe_pv_with_pfx_sfx(rd.getVar('PV'), uri_type) > -- > 2.30.2 > > > >
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#197584): https://lists.openembedded.org/g/openembedded-core/message/197584 Mute This Topic: https://lists.openembedded.org/mt/105198896/21656 Group Owner: [email protected] Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
