git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] submodule: add more exhaustive up-path testing

From
Stefan Beller <sbeller@google.com>
Date
Aug 14, 2018, 21:32 UTC
Message-ID
<CAGZ79kZq745X2PjRpGJGWdnd+HJtPuwCnJUcePXtRoLkaTWrsg@mail.gmail.com>
In-Reply-To
<87bma4hcnl.fsf@evledraar.gmail.com>

On Tue, Aug 14, 2018 at 2:16 PM Ævar Arnfjörð Bjarmason <avarab@gmail.com> wrote:

Show 29 quoted lines
>
>
> On Tue, Aug 14 2018, Stefan Beller wrote:
>
> > On Tue, Aug 14, 2018 at 2:05 PM Ævar Arnfjörð Bjarmason
> > <avarab@gmail.com> wrote:
> >
> >> > So ideally we'd also error out as soon as the host name is touched?
> >>
> >> Do we have some utility function that'll take whatever we have in
> >> remote.<name>.url and spew out the username / host / path? We must,
> >> since the clone protocol needs it, but I haven't found it.
> >
> > Nope. Welcome to the wonderful world of submodules.
> > As submodules are in the transition state towards "in C",
> > we could do some refactorings that would allow for easy access to
> > these properties, but the transition is done via faithful conversion from
> > shell to C, which did not have these functions either, but could just
> > do some string hackery. And that is how we ended up with this
> > function in the first place.
>
> The remote/clone machinery is in C and so is the resolve-relative-url
> helper. What's the missing bridge here?
>
> Maybe we don't have that function yet, but we can presumably expose it,
> and this seems unrelated to some other bits of submodules being in
> shellscript.
>
> No?
Gah, I misread your original question. Sorry about that.
Previous: Ævar Arnfjörð Bjarmason
Message 7 of 7 in “submodule: add more exhaustive up-path testing”
  1. submodule: add more exhaustive up-path testingÆvar Arnfjörð Bjarmason, Aug 14, 2018
  2. Stefan BellerAug 14, 2018
  3. Junio C HamanoAug 14, 2018
  4. Ævar Arnfjörð BjarmasonAug 14, 2018
  5. Stefan BellerAug 14, 2018
  6. Ævar Arnfjörð BjarmasonAug 14, 2018
  7. Stefan BellerAug 14, 2018

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.