git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Adam Spiers <git@adamspiers.org>
To: Junio C Hamano <gitster@pobox.com>
Cc: git list <git@vger.kernel.org>
Subject: Re: [PATCH v2] hook: add sample template for push-to-checkout
Date: Thu, 15 Oct 2020 23:54:46 +0100	[thread overview]
Message-ID: <20201015225446.b5tvyo3cquhslfry@gmail.com> (raw)
In-Reply-To: <xmqqo8l3dxu7.fsf@gitster.c.googlers.com>

On Thu, Oct 15, 2020 at 01:43:28PM -0700, Junio C Hamano wrote: 
>Adam Spiers <git@adamspiers.org> writes: 
>
>>The template is a more-or-less exact translation to shell of the C 
>>code for the default behaviour for git's push-to-checkout hook defined 
>>in the push_to_deploy() function in builtin/receive-pack.c, to serve 
>>as a convenient starting point for modification. 
>>
>>It also contains relevant text extracted from the git-config(1) and 
>>githooks(5) man pages. 
>>
>>Signed-off-by: Adam Spiers <git@adamspiers.org> 
>>---
>>  templates/hooks--push-to-checkout.sample | 74 ++++++++++++++++++++++++ 
>>  1 file changed, 74 insertions(+) 
>>  create mode 100755 templates/hooks--push-to-checkout.sample 
>>
>>diff --git a/templates/hooks--push-to-checkout.sample b/templates/hooks--push-to-checkout.sample 
>>new file mode 100755 
>>index 0000000000..2c6e06f8f1 
>>--- /dev/null 
>>+++ b/templates/hooks--push-to-checkout.sample 
>>@@ -0,0 +1,74 @@ 
>>+#!/bin/bash 
>
>If we want to make this part of the sample hooks shown to everybody, 
>we should stick to /bin/sh if we could.  Do you have to rely on any 
>bash-ism that are not found in other shells to write this script, or 
>is this just shows your inertia that you always work with bash? 

I usually work with bash and zsh, and just forgot to convert it in 
this case.

>>+# The hook receives the commit with which the tip of the current 
>>+# branch is going to be updated: 
>>+commit="$1" 
>
>Strictly speaking, a parameter on the right hand side of an 
>assignment does not have to get dquoted to protect it from getting 
>munged at $IFS, so this can be 
>
>	commit=$1

Ah, interesting thanks - not sure why it's taken me 25 years to 
discover that. 

-- >8 --
Subject: [PATCH v2] hook: add sample template for push-to-checkout

The template is a more-or-less exact translation to shell of the C
code for the default behaviour for git's push-to-checkout hook defined
in the push_to_deploy() function in builtin/receive-pack.c, to serve
as a convenient starting point for modification.

It also contains relevant text extracted from the git-config(1) and
githooks(5) man pages.

Signed-off-by: Adam Spiers <git@adamspiers.org>
---
  templates/hooks--push-to-checkout.sample | 78 ++++++++++++++++++++++++
  1 file changed, 78 insertions(+)
  create mode 100755 templates/hooks--push-to-checkout.sample

diff --git a/templates/hooks--push-to-checkout.sample b/templates/hooks--push-to-checkout.sample
new file mode 100755
index 0000000000..af5a0c0018
--- /dev/null
+++ b/templates/hooks--push-to-checkout.sample
@@ -0,0 +1,78 @@
+#!/bin/sh
+
+# An example hook script to update a checked-out tree on a git push.
+#
+# This hook is invoked by git-receive-pack(1) when it reacts to git
+# push and updates reference(s) in its repository, and when the push
+# tries to update the branch that is currently checked out and the
+# receive.denyCurrentBranch configuration variable is set to
+# updateInstead.
+#
+# By default, such a push is refused if the working tree and the index
+# of the remote repository has any difference from the currently
+# checked out commit; when both the working tree and the index match
+# the current commit, they are updated to match the newly pushed tip
+# of the branch. This hook is to be used to override the default
+# behaviour; however the code below reimplements the default behaviour
+# as a starting point for convenient modification.
+#
+# The hook receives the commit with which the tip of the current
+# branch is going to be updated:
+commit=$1
+
+# It can exit with a non-zero status to refuse the push (when it does
+# so, it must not modify the index or the working tree).
+die () {
+	echo >&2 "$*"
+	exit 1
+}
+
+# Or it can make any necessary changes to the working tree and to the
+# index to bring them to the desired state when the tip of the current
+# branch is updated to the new commit, and exit with a zero status.
+#
+# For example, the hook can simply run git read-tree -u -m HEAD "$1"
+# in order to emulate git fetch that is run in the reverse direction
+# with git push, as the two-tree form of git read-tree -u -m is
+# essentially the same as git switch or git checkout that switches
+# branches while keeping the local changes in the working tree that do
+# not interfere with the difference between the branches.
+
+# The below is a more-or-less exact translation to shell of the C code
+# for the default behaviour for git's push-to-checkout hook defined in
+# the push_to_deploy() function in builtin/receive-pack.c.
+#
+# Note that the hook will be executed from the repository directory,
+# not from the working tree, so if you want to perform operations on
+# the working tree, you will have to adapt your code accordingly, e.g.
+# by adding "cd .." or using relative paths.
+
+if ! git update-index -q --ignore-submodules --refresh
+then
+	die "Up-to-date check failed"
+fi
+
+if ! git diff-files --quiet --ignore-submodules --
+then
+	die "Working directory has unstaged changes"
+fi
+
+# This is a rough translation of:
+#
+#   head_has_history() ? "HEAD" : EMPTY_TREE_SHA1_HEX
+if git cat-file -e HEAD 2>/dev/null
+then
+	head=HEAD
+else
+	head=$(git hash-object -t tree --stdin </dev/null)
+fi
+
+if ! git diff-index --quiet --cached --ignore-submodules $head --
+then
+	die "Working directory has staged changes"
+fi
+
+if ! git read-tree -u -m "$commit"
+then
+	die "Could not update working tree to new HEAD"
+fi
-- 
2.28.0


      reply	other threads:[~2020-10-15 22:54 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-15 18:47 [PATCH v1] hook: add sample template for push-to-checkout Adam Spiers
2020-10-15 20:43 ` Junio C Hamano
2020-10-15 22:54   ` Adam Spiers [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

  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=20201015225446.b5tvyo3cquhslfry@gmail.com \
    --to=git@adamspiers.org \
    --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).