From: "Russell King (Oracle)" <linux@armlinux.org.uk>
To: Jeff King <peff@peff.net>
Cc: Todd Zullinger <tmz@pobox.com>, git@vger.kernel.org
Subject: Re: [BUG?] git-daemon 2.49.0 in F40 no longer exports user directories
Date: Sun, 27 Jul 2025 10:14:50 +0100 [thread overview]
Message-ID: <aIXuCv7hRyJq-Twm@shell.armlinux.org.uk> (raw)
In-Reply-To: <20250727064542.GA3091411@coredump.intra.peff.net>
On Sun, Jul 27, 2025 at 02:45:42AM -0400, Jeff King wrote:
> On Sat, Jul 26, 2025 at 09:53:27AM -0400, Todd Zullinger wrote:
>
> > I was curious, so I took Peff's recipe and gave it a try.
> >
> > sudo dnf -y install git-daemon
> > sudo systemctl enable --now git.socket
> > mkdir ~/public_git
> > git init --bare ~/public_git/repo.git
> > git -C ~/public_git/repo.git --work-tree=. commit --allow-empty -m foo
> > sudo git config --system --add safe.directory ~/public_git/\*
> > git ls-remote git://localhost/~test/repo.git
> >
> > And that fails as it does for Russell. I suspected SELinux,
> > which is enabled by default on Fedora. With luck, you have
> > not already ruled that out.
>
> Interesting. That would explain why I didn't see the problem on my
> Debian system.
>
> I am still puzzled why Russell would see the message he does, though.
> From my read of the code, seeing "not in directory list" but not seeing
> "...does not appear to be a git repository" implies that enter_repo()
> succeeds, but ok_paths is non-empty and forbids it.
Yes, I had an override in systemd that I'd forgotten about to work
around a previous issue (because the previous version we had on the
machine required the paths on the end of the git-daemon command line.)
I wonder if there was a bug - --export-all broke, and then required the
directories to be listed, and now it's been "fixed" and now the listed
directories become a list of those to exclude?
The override.conf had something like:
ExecStart=-/usr/libexec/git-core/git-daemon --export-all \
--user-path=public_git --inetd --log-destination=stderr --verbose \
--interpolated-path='/var/lib/git/%%H%%D' \
/var/lib/git/git.armlinux.org.uk \
/home/rmk/public_git
I can't remember exactly now as I've deleted the last two paths, and
added --base-path=/var/lib/git - and now it's working.
--
RMK's Patch system: https://www.armlinux.org.uk/developer/patches/
FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
prev parent reply other threads:[~2025-07-27 9:14 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-07-25 16:11 [BUG?] git-daemon 2.49.0 in F40 no longer exports user directories Russell King (Oracle)
2025-07-26 7:51 ` Jeff King
2025-07-26 13:53 ` Todd Zullinger
2025-07-27 6:45 ` Jeff King
2025-07-27 9:14 ` Russell King (Oracle) [this message]
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=aIXuCv7hRyJq-Twm@shell.armlinux.org.uk \
--to=linux@armlinux.org.uk \
--cc=git@vger.kernel.org \
--cc=peff@peff.net \
--cc=tmz@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.
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.