[PATCH] Add a sample hook which saves push certs as notes

2017-12-02 Thread Shikher Verma
hooks--post-receive.sample: If push cert is present, add it as a git note to the top most commit of the updated ref. Signed-off-by: Shikher Verma --- templates/hooks--post-receive.sample | 38 1 file changed, 38 insertions(+) create mode 100755 templates

Re: [RFC PATCH 0/2] Add named reference to latest push cert

2017-09-16 Thread Shikher Verma
On Thu, Sep 07, 2017 at 05:43:19PM +, Stefan Beller wrote: > On Thu, Sep 7, 2017 at 2:11 AM, Shikher Verma wrote: > > On Wed, Sep 06, 2017 at 02:31:49PM -0700, Stefan Beller wrote: > >> On Wed, Sep 6, 2017 at 2:39 AM, Shikher Verma > >> wrote: > >&g

Re: [RFC PATCH 0/2] Add named reference to latest push cert

2017-09-07 Thread Shikher Verma
On Wed, Sep 06, 2017 at 02:31:49PM -0700, Stefan Beller wrote: > On Wed, Sep 6, 2017 at 2:39 AM, Shikher Verma wrote: > > Currently, git only stores push certificates if there is a receive hook > > present. This may violate the principle of least surprise (e.g., I > >

Re: [RFC PATCH 0/2] Add named reference to latest push cert

2017-09-07 Thread Shikher Verma
On Thu, Sep 07, 2017 at 09:55:25AM +0900, Junio C Hamano wrote: > Stefan Beller writes: > > > On the ref to store the push certs: > > (a) Currently the ref points at the blob, I wonder if we'd rather want to > > point at a commit? (Then we can build up a history of > > push certs, instead

Re: [RFC PATCH 0/2] Add named reference to latest push cert

2017-09-07 Thread Shikher Verma
repo after I did a signed push, hence this RFC. Anyway this is my first time trying to contribute to a large OSS so forgive me if I make any noob mistakes. Thanks Shikher Verma [1]http://shikherverma.com/ [2]https://ssl.engineering.nyu.edu/ [3]https://public-inbox.org/git/20170707220159.12752-1

[RFC PATCH 1/2] Always write push cert to disk

2017-09-06 Thread Shikher Verma
: Shikher Verma Helped-by: Santiago Torres-Arias --- builtin/receive-pack.c | 16 +++- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/builtin/receive-pack.c b/builtin/receive-pack.c index 52c63ebfd..79195005f 100644 --- a/builtin/receive-pack.c +++ b/builtin/receive

[RFC PATCH 0/2] Add named reference to latest push cert

2017-09-06 Thread Shikher Verma
es as RFC since the documentation would have to be updated and git gc might have to be patched to not garbage collect the latest push certificate. This patch applies on master (3ec7d702a) Shikher Verma (2): Always write push cert to disk Store latest push cert ref in PUSH_CERT builtin/re

[RFC PATCH 2/2] Store latest push cert ref in PUSH_CERT

2017-09-06 Thread Shikher Verma
around the push certificate feature. Signed-off-by: Shikher Verma Helped-by: Santiago Torres-Arias --- builtin/receive-pack.c | 9 + path.c | 1 + path.h | 1 + 3 files changed, 11 insertions(+) diff --git a/builtin/receive-pack.c b/builtin/receive-pa