Return-Path: X-Original-To: apmail-drill-issues-archive@minotaur.apache.org Delivered-To: apmail-drill-issues-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 62587172C0 for ; Mon, 13 Apr 2015 23:58:12 +0000 (UTC) Received: (qmail 16524 invoked by uid 500); 13 Apr 2015 23:58:12 -0000 Delivered-To: apmail-drill-issues-archive@drill.apache.org Received: (qmail 16495 invoked by uid 500); 13 Apr 2015 23:58:12 -0000 Mailing-List: contact issues-help@drill.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@drill.apache.org Delivered-To: mailing list issues@drill.apache.org Received: (qmail 16449 invoked by uid 99); 13 Apr 2015 23:58:12 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 13 Apr 2015 23:58:12 +0000 Date: Mon, 13 Apr 2015 23:58:12 +0000 (UTC) From: "Steven Phillips (JIRA)" To: issues@drill.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (DRILL-2774) Updated drill-patch-review.py to use git-format-patch MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/DRILL-2774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14493313#comment-14493313 ] Steven Phillips commented on DRILL-2774: ---------------------------------------- Created reviewboard > Updated drill-patch-review.py to use git-format-patch > ----------------------------------------------------- > > Key: DRILL-2774 > URL: https://issues.apache.org/jira/browse/DRILL-2774 > Project: Apache Drill > Issue Type: Bug > Components: Tools, Build & Test > Reporter: Steven Phillips > Assignee: Steven Phillips > Priority: Minor > Fix For: 0.9.0 > > Attachments: DRILL-2774.patch > > > The tool currently uses git diff to generate the patches, which does not preserve commit information, and which is required for submitting patches in the Drill community. > This doesn't work properly when there are multiple commits, so as part of this change, we enforce the requirement that the branch used to create the patch is exactly one commit ahead and zero behind the remote branch. -- This message was sent by Atlassian JIRA (v6.3.4#6332)