From: Joel Brobecker <brobecker@adacore.com>
To: "Martin Liška" <mliska@suse.cz>,
Binutils <binutils@sourceware.org>,
gdb-patches@sourceware.org
Subject: Re: Please enable fast forward for user branches
Date: Tue, 30 Mar 2021 10:17:17 +0400 [thread overview]
Message-ID: <20210330061717.GD11111@adacore.com> (raw)
In-Reply-To: <YGK41DFaIxhEGjOM@vapier>
> > I think the error above comes from Git itself, and is related
> > to the configuration of the repository. In particular, I found
> > that our binutils-gdb repository's configuration has:
> >
> > | [receive]
> > | denynonfastforwards = true
> >
> > Not sure what the history of this is.
> >
> > In the meantime, I've modified the git-hooks configuration so that,
> > if the above is lifted, users will be allowed to do non-fast-forward
> > updates on users/.* branches.
>
> iiuc, out of the box, the default git implementation has one knob for all
> branches (the one you found). we set it that way so people don't push
> non-fast-forwards to the important branches (e.g. "master" or any of the
> many release branches). that all makes perfect sense -- we don't want
> people to accidentally, or on purpose, rewrite (i.e. rewind) history once
> it's gone public.
>
> unfortunately, the default git configs don't have ref filtering to allow
> denynonfastforwards=false on refs/heads/users/*. any services that have
> that functionality implemented it themselves. which is what we'd have to
> do with custom git hooks if we wanted to. but it looks like no one has
> volunteered to implement the hook to block most refs by default but only
> allow a specific subset. probably because they don't want to be on the
> hook for when it needs debugging.
Actually, we do use the git-hooks (https://github.com/adacore/git-hooks)
which allow per-reference configuration of that restriction. That's
what I was trying to explain in my message.
Perhaps the receive.denynonfastforwards pre-dates the use of
the git-hooks.
--
Joel
next prev parent reply other threads:[~2021-03-30 6:17 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1af95023-580d-6774-c307-1a38bf3dc073@suse.cz>
2021-03-30 4:09 ` Joel Brobecker
2021-03-30 5:36 ` Mike Frysinger via Gdb-patches
2021-03-30 6:17 ` Joel Brobecker [this message]
2021-03-30 11:53 ` Mike Frysinger via Gdb-patches
2021-03-30 12:52 ` Joel Brobecker
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=20210330061717.GD11111@adacore.com \
--to=brobecker@adacore.com \
--cc=binutils@sourceware.org \
--cc=gdb-patches@sourceware.org \
--cc=mliska@suse.cz \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox