git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH v2 8/9] rebase: add the --gpg-sign option

From
brian m. carlson <sandals@crustytoothpaste.net>
Date
Jan 27, 2014, 23:59 UTC
Message-ID
<20140127235917.GA91854@vauxhall.crustytoothpaste.net>
In-Reply-To
<xmqqha8pm1yr.fsf@gitster.dls.corp.google.com>
On Mon, Jan 27, 2014 at 03:36:44PM -0800, Junio C Hamano wrote:
Show 30 quoted lines
> "brian m. carlson" <sandals@crustytoothpaste.net> writes:
> 
> > diff --git a/git-rebase--interactive.sh b/git-rebase--interactive.sh
> > index 43c19e0..73d32dd 100644
> > --- a/git-rebase--interactive.sh
> > +++ b/git-rebase--interactive.sh
> > @@ -181,7 +181,7 @@ exit_with_patch () {
> >  	git rev-parse --verify HEAD > "$amend"
> >  	warn "You can amend the commit now, with"
> >  	warn
> > -	warn "	git commit --amend"
> > +	warn "	git commit --amend $gpg_sign_opt"
> 
> I think this is meant to be cut-&-pasted, so you may have to quote
> the RHS of --gpg-sign=<key> (or the key part of -S<key>).
> 
> The same comment as the one for 'git pull' patch applies around
> 'eval's in the remainder of the patch.
> 
> > @@ -248,7 +248,8 @@ pick_one () {
> >  
> >  	test -d "$rewritten" &&
> >  		pick_one_preserving_merges "$@" && return
> > -	output eval git cherry-pick "$strategy_args" $empty_args $ff "$@"
> > +	output eval git cherry-pick ${gpg_sign_opt:+"$gpg_sign_opt"} \
> > +			"$strategy_args" $empty_args $ff "$@"
> >  }
> > [rest snipped]
> 
> Thanks.
Thanks for the review.  I'll get working on a reroll.
-- 
brian m. carlson / brian with sandals: Houston, Texas, US
+1 832 623 2791 | http://www.crustytoothpaste.net/~bmc | My opinion only
OpenPGP: RSA v4 4096b: 88AC E9B2 9196 305B A994 7552 F1BA 225C 0223 B187
Previous: Junio C HamanoNext: brian m. carlson
Message 16 of 22 in “add --gpg-sign to rebase and pull”
  1. 0/9 add --gpg-sign to rebase and pullbrian m. carlson, Jan 24, 2014
  2. 1/9 cherry-pick, revert: add the --gpg-sign optionbrian m. carlson, Jan 24, 2014
  3. Junio C HamanoJan 24, 2014
  4. brian m. carlsonJan 24, 2014
  5. Junio C HamanoJan 24, 2014
  6. 2/9 git-sh-setup.sh: add variable to use the stuck-long modebrian m. carlson, Jan 24, 2014
  7. Junio C HamanoJan 27, 2014
  8. 3/9 am: parse options in stuck-long modebrian m. carlson, Jan 24, 2014
  9. 4/9 am: add the --gpg-sign optionbrian m. carlson, Jan 24, 2014
  10. Junio C HamanoJan 27, 2014
  11. 5/9 rebase: remove useless arguments checkbrian m. carlson, Jan 24, 2014
  12. 6/9 rebase: don't try to match -M optionbrian m. carlson, Jan 24, 2014
  13. 7/9 rebase: parse options in stuck-long modebrian m. carlson, Jan 24, 2014
  14. 8/9 rebase: add the --gpg-sign optionbrian m. carlson, Jan 24, 2014
  15. Junio C HamanoJan 27, 2014
  16. brian m. carlsonJan 27, 2014
  17. 9/9 pull: add the --gpg-sign option.brian m. carlson, Jan 24, 2014
  18. Junio C HamanoJan 27, 2014
  19. brian m. carlsonFeb 1, 2014
  20. Junio C HamanoFeb 3, 2014
  21. Nicolas VigierJan 24, 2014
  22. Junio C HamanoJan 24, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.