From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: stefan.herbrechtsmeier-oss@weidmueller.com,
bitbake-devel@lists.openembedded.org
Cc: Stefan Herbrechtsmeier <stefan.herbrechtsmeier@weidmueller.com>
Subject: Re: [bitbake-devel] [RFC PATCH 0/6] fetch2: add support for implicit urls
Date: Sun, 07 Sep 2025 16:52:13 +0100 [thread overview]
Message-ID: <c1124597a57241518f4ddee17c5786ecb11caf2f.camel@linuxfoundation.org> (raw)
In-Reply-To: <20250902065507.35737-1-stefan.herbrechtsmeier-oss@weidmueller.com>
On Tue, 2025-09-02 at 08:55 +0200, Stefan Herbrechtsmeier via lists.openembedded.org wrote:
> The patch series add support for implicit URLs inside the fetcher. The
> implicit URLs could be defined inside a source like a version control
> system (git submodule) or a lock file (package-lock.json, cargo.lock or
> go.sum). The integration of implicit URLs beside explicit URLs
> simplifies the fetcher classes and avoid bugs because of iterations
> between the Fetch and FetchMethod classes.
>
> The series remove most methods inside the gitsm fetcher and only leaves
> the parsing of the git submodules and the unpack functionality. It
> allows the gitsm fetcher to use the premirror only feature. The current
> implementation leads to problems because the download of the git
> submodules is triggered via the download method which is called deeply
> inside the fetcher code.
We had the discussion a while back and the conclusion seemed to be that
implict urls were disliked by a significant number of people as the
were too unclear about what was going on behind the scenes and also
made things like software manifests harder. There was a strong
preference for metadata helpers and explicit lists of components which
we have for crates/rust and now for go too.
It feels like this series is moving us back to the other direction. Is
that correct and if so, what has changed in the approach since the last
discussion?
Cheers,
Richard
next prev parent reply other threads:[~2025-09-07 15:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-02 6:55 [RFC PATCH 0/6] fetch2: add support for implicit urls Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 1/6] fetch2: rename u to url in Fetch class Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 2/6] fetch2: call functions within loops of " Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 3/6] fetch2: add helper to get urldata in " Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 4/6] fetch2: add support for implicit urls Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 5/6] fetch2: gitsm: use implicit urls feature Stefan Herbrechtsmeier
2025-09-02 6:55 ` [RFC PATCH 6/6] tests: fetch: add test case for gitsm implicit local paths Stefan Herbrechtsmeier
2025-09-04 6:00 ` [bitbake-devel] [RFC PATCH 0/6] fetch2: add support for implicit urls Mathieu Dubois-Briand
2025-09-04 6:09 ` Stefan Herbrechtsmeier
2025-09-05 7:01 ` Stefan Herbrechtsmeier
2025-09-07 15:52 ` Richard Purdie [this message]
2025-09-08 9:20 ` Stefan Herbrechtsmeier
2025-09-08 10:26 ` Richard Purdie
2025-09-09 12:48 ` Stefan Herbrechtsmeier
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=c1124597a57241518f4ddee17c5786ecb11caf2f.camel@linuxfoundation.org \
--to=richard.purdie@linuxfoundation.org \
--cc=bitbake-devel@lists.openembedded.org \
--cc=stefan.herbrechtsmeier-oss@weidmueller.com \
--cc=stefan.herbrechtsmeier@weidmueller.com \
/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.