All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Theodore Tso" <tytso@mit.edu>
To: Matthias Goergens <matthias.goergens@gmail.com>
Cc: Jan Kara <jack@suse.cz>, linux-ext4@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: name the ext4 dev branch
Date: Wed, 23 Sep 2026 23:14:31 -0400	[thread overview]
Message-ID: <arSMwd1bGO1vrif-@mit.edu> (raw)
In-Reply-To: <20260923101749.3505886-1-matthias.goergens@gmail.com>

On Wed, Sep 23, 2026 at 06:17:49PM -0500, Matthias Goergens wrote:
> The ext4 T: entry names the repository without a branch, so tools that
> default to the repository's HEAD select master, which still points at a
> merge from October 2020 (96485e446260).  Development happens on dev,
> which is also the branch linux-next merges.  Name it.

Huh.  I wasn't even aware that you *could* specify the branch name in
the T: entry.  A quick "grep ^T: MAINTAINERS" there are only a few
repos using T: that include branch.

I also tried looking at the documentation, and I can't find any
mention that the git branch could be specified.  Maybe we should
update the documentation for T: in a separate patch to the MAINTAINERS
file?  Or did I miss something?  "git grep MAINTAINERS | grep branch"
didn't turn up anything.

> The Sashiko review bot, for one, falls back to HEAD this way and has
> been reviewing ext4 patches against that tree.  Its review of an EA
> inode refcount fix reported a Critical double decrement and an
> undefined function after applying the patch to that stale baseline;
> neither holds on dev.

... and I didn't know that Sashiko uses the branch specifier.  I
wonder if ext4 is the only tree which might have this issue...

   git log --merges linux-next

... nope, apparently not.  I see a lot of branches which are merged
into linux-net which are not mentioned in the MAINTAINERS file.

Thanks for pointing this out for the ext4 MAINTAINERS entry!

						- Ted

  parent reply	other threads:[~2026-09-24  3:15 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-23 10:17 [PATCH] MAINTAINERS: name the ext4 dev branch Matthias Goergens
2026-09-23 10:44 ` Jan Kara
2026-09-24  3:14 ` Theodore Tso [this message]
2026-09-24  3:32   ` Matthias Goergens
2026-09-24  4:00     ` Eric Biggers
2026-09-24  8:30       ` Matthias Goergens
2026-09-24  8:53       ` Jan Kara
2026-09-25 13:06       ` Niklas Cassel
2026-09-25 18:43         ` Eric Biggers
2026-09-26  6:11         ` Matthias Goergens
2026-09-26 15:57           ` Niklas Cassel
2026-10-01 11:25     ` Matthias Goergens
2026-10-08 15:32 ` Theodore Ts'o

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=arSMwd1bGO1vrif-@mit.edu \
    --to=tytso@mit.edu \
    --cc=jack@suse.cz \
    --cc=linux-ext4@vger.kernel.org \
    --cc=matthias.goergens@gmail.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.