From: Emily Shaffer <emilyshaffer@google.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: git@vger.kernel.org
Subject: Re: What's cooking in git.git (Nov 2021, #03; Tue, 9)
Date: Thu, 11 Nov 2021 14:19:20 -0800 [thread overview]
Message-ID: <YY2W6ESIxSz9lakK@google.com> (raw)
In-Reply-To: <xmqqy25wygek.fsf@gitster.g>
Once more, updates to submodule-UX-overhaul related work.
On Tue, Nov 09, 2021 at 04:59:31PM -0800, Junio C Hamano wrote:
> --------------------------------------------------
> [Stalled]
>
> * ar/submodule-update (2021-10-13) 9 commits
> . submodule--helper: rename helper functions
> . submodule--helper: remove unused helpers
> . submodule: move core cmd_update() logic to C
> . submodule--helper: run update using child process struct
> . submodule--helper: allow setting superprefix for init_submodule()
> . submodule--helper: refactor get_submodule_displaypath()
> . submodule--helper: rename helpers for update-clone
> . submodule--helper: get remote names from any repository
> . submodule--helper: split up ensure_core_worktree()
>
> Rewrite of "git submodule update" in C.
>
> Kicked out of 'seen' to make room for es/superproject-aware-submodules
> which is among the topics this topic stomps on.
There is some discussion on es/superproject-aware-submodules, and
whether it is still needed. I think it is, but I'll say more downthread,
instead of next to this topic.
> --------------------------------------------------
> [Cooking]
>
> * gc/remote-with-fewer-static-global-variables (2021-10-28) 6 commits
> - remote: add struct repository parameter to external functions
> - remote: die if branch is not found in repository
> - remote: remove the_repository->remote_state from static methods
> - remote: use remote_state parameter internally
> - remote: move static variables into per-repository struct
> - t5516: add test case for pushing remote refspecs
>
> Code clean-up to eventually allow information on remotes defined
> for an arbitrary repository to be read.
>
> Will merge to 'next'?
What's still blocking this from being merged? Anything? Release
schedule?
> * js/branch-track-inherit (2021-10-18) 1 commit
> - branch: add flags and config to inherit tracking
>
> "git -c branch.autosetupmerge=inherit branch new old" makes "new"
> to have the same upstream as the "old" branch, instead of marking
> "old" itself as its upstream.
>
> Under discussion.
> cf. <87a6j6tbsv.fsf@gmgdl.gmail.com>
Got confirmation this week that Josh is actively working on the next reroll.
> * ab/config-based-hooks-2 (2021-11-01) 18 commits
> - run-command: remove old run_hook_{le,ve}() hook API
> - receive-pack: convert push-to-checkout hook to hook.h
> - read-cache: convert post-index-change to use hook.h
> - commit: convert {pre-commit,prepare-commit-msg} hook to hook.h
> - git-p4: use 'git hook' to run hooks
> - send-email: use 'git hook run' for 'sendemail-validate'
> - git hook run: add an --ignore-missing flag
> - hooks: convert worktree 'post-checkout' hook to hook library
> - hooks: convert non-worktree 'post-checkout' hook to hook library
> - merge: convert post-merge to use hook.h
> - am: convert applypatch-msg to use hook.h
> - rebase: convert pre-rebase to use hook.h
> - hook API: add a run_hooks_l() wrapper
> - am: convert {pre,post}-applypatch to use hook.h
> - gc: use hook library for pre-auto-gc hook
> - hook API: add a run_hooks() wrapper
> - hook: add 'run' subcommand
> - Merge branch 'ab/config-based-hooks-1' into ab/config-based-hooks-2
>
> More "config-based hooks".
I think I owe another review, but as always with these topics, I wrote a
lot of the code so I'm not sure how much I can really help. Other eyes
appreciated.
> * es/superproject-aware-submodules (2021-11-04) 4 commits
> - submodule: record superproject gitdir during 'update'
> - submodule: record superproject gitdir during absorbgitdirs
> - introduce submodule.superprojectGitDir record
> - t7400-submodule-basic: modernize inspect() helper
>
> A configuration variable in a submodule points at the location of
> the superproject it is bound to (RFC).
To summarize the discussion from here: Ævar suggested this topic might
not be necessary anymore, and that we should rely on in-process
discovery of the superproject's gitdir. However, after some more
thought, I think it's valuable to strive for a definitive way to tell
"yes, I am a submodule" - and I'd like for this topic to be it. I'm
planning a reroll (and an explanation in the cover letter), and to drop
language referring to that as a "cache" (because it isn't a cheap
version of an operation the submodule would be doing otherwise). I will
also add another patch to demonstrate how we can use that new
information as a point of truth, instead of a performance shim.
---
Jonathan Tan: WIP v2 Conditional config includes remotes based on remote URL
https://lore.kernel.org/git/cover.1635527389.git.jonathantanmy%40google.com
Additional discussion ongoing, Jonathan is working on another reroll,
with a substantially different implementation but same user-facing
feature. I think this discussion can benefit from reviews with an open
mind - this is another effort to solve "how can a team ship configs for
anybody developing their project, in the least painful (to the user) way
possible?" In other words, this is a spiritual successor to the
"remote-suggested hooks" topic, in that we are still trying to solve the
same problem.
- Emily
next prev parent reply other threads:[~2021-11-11 22:19 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-10 0:59 What's cooking in git.git (Nov 2021, #03; Tue, 9) Junio C Hamano
2021-11-10 12:47 ` js/scalar, was " Johannes Schindelin
2021-11-10 13:42 ` Ævar Arnfjörð Bjarmason
2021-11-11 18:25 ` Junio C Hamano
2021-11-12 9:32 ` Johannes Schindelin
2021-11-10 13:04 ` jc/fix-pull-ff-only-when-already-up-to-date, " Johannes Schindelin
2021-11-10 17:20 ` Junio C Hamano
2021-11-10 19:00 ` Alex Henrie
2021-11-10 19:09 ` Johannes Schindelin
2021-11-10 19:17 ` Junio C Hamano
2021-11-11 22:19 ` Emily Shaffer [this message]
2021-11-11 22:40 ` Emily Shaffer
2021-11-11 22:58 ` Junio C Hamano
2021-11-11 23:44 ` Glen Choo
2021-11-13 9:17 ` ab/config-based-hooks-2 etc. (was: What's cooking in git.git (Nov 2021, #03; Tue, 9)) Ævar Arnfjörð Bjarmason
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YY2W6ESIxSz9lakK@google.com \
--to=emilyshaffer@google.com \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).