From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on dcvr.yhbt.net X-Spam-Level: X-Spam-ASN: AS31976 209.132.180.0/23 X-Spam-Status: No, score=-3.3 required=3.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_HI, T_RP_MATCHES_RCVD shortcircuit=no autolearn=ham autolearn_force=no version=3.4.0 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by dcvr.yhbt.net (Postfix) with ESMTP id DC21820954 for ; Fri, 24 Nov 2017 11:08:22 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753460AbdKXLIT (ORCPT ); Fri, 24 Nov 2017 06:08:19 -0500 Received: from smtp-out-4.talktalk.net ([62.24.135.68]:52249 "EHLO smtp-out-4.talktalk.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753239AbdKXLIQ (ORCPT ); Fri, 24 Nov 2017 06:08:16 -0500 Received: from lindisfarne.localdomain ([92.22.34.132]) by smtp.talktalk.net with SMTP id IBq0eNps6Ap17IBqBeb1C1; Fri, 24 Nov 2017 11:08:15 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=talktalk.net; s=cmr1711; t=1511521695; bh=TI4uM+1BGv6WGUUxMiImyjF4Wcsx492V3Hxvx1V4Mws=; h=From:To:Cc:Subject:Date:In-Reply-To:References:Reply-To; b=bdrdkCet1lBkIBqd577tT457R5TeDleLADlmw5S8LvH/qjpBRbW751XExXukNRqzj K7Y0oRwAlsahCnlY1nQxdMACZPOaDxHAy9FPN1QjYSJuLAOI8b3+6hh4lDMqCKD+Iz Ia4HeMs4kU2xlRKlp1ZZG/ekX2eYZOyR+l91Z5zo= X-Originating-IP: [92.22.34.132] X-Spam: 0 X-OAuthority: v=2.2 cv=EsGilWUA c=1 sm=1 tr=0 a=2gYdyS03q/cwff7SV6P5Ng==:117 a=2gYdyS03q/cwff7SV6P5Ng==:17 a=evINK-nbAAAA:8 a=qo_IBofmjA8VuP4PJYkA:9 a=FTkRqzf3F1kRNC0N:21 a=x326BpXhDvg8PScu:21 a=RfR_gqz1fSpA9VikTjo0:22 From: Phillip Wood To: Git Mailing List Cc: Johannes Schindelin , Junio C Hamano , Ramsay Jones , Adam Dinwoodie , Stefan Beller , Phillip Wood Subject: [PATCH v4 6/9] sequencer: simplify adding Signed-off-by: trailer Date: Fri, 24 Nov 2017 11:07:55 +0000 Message-Id: <20171124110758.9406-7-phillip.wood@talktalk.net> X-Mailer: git-send-email 2.15.0 In-Reply-To: <20171124110758.9406-1-phillip.wood@talktalk.net> References: <20170925101041.18344-1-phillip.wood@talktalk.net> <20171124110758.9406-1-phillip.wood@talktalk.net> Reply-To: Phillip Wood X-CMAE-Envelope: MS4wfH8TKEAhOUqlKo1McuTIUQFTFRAvda/StH8BtjSrDZei7kxClCB64wbJffACTQdX6gVW+b33lPCuEcUHMcOhuqPnsTuCyPPiVFEdtQWsLXoO0nkvLNiB lQRi3n0PWrzL2lR9s7Ea5entR4T9NM81P2lPeurvA5RjVS8CYPOnaLKxVhzSScOJtZhAthq24z6/kPjRtfyhjqMw7DPjZsd3iiGx40tvNMBcLeY1VduDPcZs whVUimu3OR6d62LDJQLwCPsEbMZNL7XE0guC0TUK1kfOXTixabXhcjQ3ZkKj06d+R0JZRM/NIvdKqdjqScF5Bri60/gqVHuIAwRR9bd05C6VyfVGRJo7nQ6M Z8XH4cwQkZDAHNHJ2e/s0xYqB8MAesp2/Ehe6/XHOBvFeCmLl+8= Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org From: Phillip Wood Add the Signed-off-by: trailer in one place rather than adding it to the message when doing a recursive merge and specifying '--signoff' when running 'git commit'. This means that if there are conflicts when merging with a strategy other than 'recursive' the Signed-off-by: trailer will be added if the user commits the resolution themselves without passing '--signoff' to 'git commit'. It also simplifies the in-process commit that is about to be added to the sequencer. Signed-off-by: Phillip Wood --- sequencer.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/sequencer.c b/sequencer.c index a2cf6f5e06ffec5108f0faf43d1a4cb605264c3f..7400df5522037583108534755af6f542117667c2 100644 --- a/sequencer.c +++ b/sequencer.c @@ -477,9 +477,6 @@ static int do_recursive_merge(struct commit *base, struct commit *next, _(action_name(opts))); rollback_lock_file(&index_lock); - if (opts->signoff) - append_signoff(msgbuf, 0, 0); - if (!clean) append_conflicts_hint(msgbuf); @@ -657,8 +654,6 @@ static int run_git_commit(const char *defmsg, struct replay_opts *opts, argv_array_push(&cmd.args, "--amend"); if (opts->gpg_sign) argv_array_pushf(&cmd.args, "-S%s", opts->gpg_sign); - if (opts->signoff) - argv_array_push(&cmd.args, "-s"); if (defmsg) argv_array_pushl(&cmd.args, "-F", defmsg, NULL); if ((flags & CLEANUP_MSG)) @@ -1347,6 +1342,9 @@ static int do_pick_commit(enum todo_command command, struct commit *commit, } } + if (opts->signoff) + append_signoff(&msgbuf, 0, 0); + if (is_rebase_i(opts) && write_author_script(msg.message) < 0) res = -1; else if (!opts->strategy || !strcmp(opts->strategy, "recursive") || command == TODO_REVERT) { -- 2.15.0