From: Junio C Hamano <gitster@pobox.com>
To: "Ævar Arnfjörð Bjarmason" <avarab@gmail.com>
Cc: Git Mailing List <git@vger.kernel.org>
Subject: Re: Why isn't there a hook for all operations that update the working tree?
Date: Wed, 03 Aug 2011 19:06:26 -0700 [thread overview]
Message-ID: <7v1ux26icd.fsf@alter.siamese.dyndns.org> (raw)
In-Reply-To: <CACBZZX7dJhGT0H8JZRbQ_t9mNnJocaktYAXgMSihfLBuFmL3nw@mail.gmail.com> ("Ævar Arnfjörð Bjarmason"'s message of "Thu, 28 Jul 2011 13:59:30 +0200")
Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:
> ... Is there any
> reason beside omission that there isn't a post-reset hook?
> Or hooks for any other thing ...
In principle, we historically tended to avoid hooks unless absolutely
necessary. Even having to check to find out no hook needs to be run is
considered wasted cycles.
http://thread.gmane.org/gmane.comp.version-control.git/79314/focus=79321
I'd especially rather not to see a hook on something as low level as
"reset". It would be too tempting to use "reset" itself inside a hook that
is run from the post-reset hook, and I do not want to complicate the code
even further to give callers to ask disabling the hook.
prev parent reply other threads:[~2011-08-04 2:06 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-07-28 11:59 Why isn't there a hook for all operations that update the working tree? Ævar Arnfjörð Bjarmason
2011-08-04 1:56 ` Neal Kreitzinger
2011-08-04 2:06 ` Junio C Hamano [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=7v1ux26icd.fsf@alter.siamese.dyndns.org \
--to=gitster@pobox.com \
--cc=avarab@gmail.com \
--cc=git@vger.kernel.org \
/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;
as well as URLs for NNTP newsgroup(s).