From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932115AbdBUTfH (ORCPT ); Tue, 21 Feb 2017 14:35:07 -0500 Received: from mail-it0-f42.google.com ([209.85.214.42]:33264 "EHLO mail-it0-f42.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754235AbdBUTeq (ORCPT ); Tue, 21 Feb 2017 14:34:46 -0500 Subject: Re: [GIT PULL] Block pull request for- 4.11-rc1 To: Linus Torvalds References: Cc: "linux-block@vger.kernel.org" , "linux-kernel@vger.kernel.org" From: Jens Axboe Message-ID: Date: Tue, 21 Feb 2017 12:34:43 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/21/2017 12:11 PM, Linus Torvalds wrote: > On Sun, Feb 19, 2017 at 4:10 PM, Jens Axboe wrote: >> >> Please pull! Either this pre-merged branch: >> >> git://git.kernel.dk/linux-block.git for-4.11/linus-merge-signed >> >> or >> >> git://git.kernel.dk/linux-block.git for-4.11/block-signed >> git://git.kernel.dk/linux-block.git for-4.11/next-signed > > So normally I'd merge them separately, but since you didn't actually > give me explanations for what the two branches were (ie "block-signed > does X, next-signed does Y") I didn't feel like I could write a sane > merge message for the two branches - so I took the pre-merged one. > > Which does bring me to my next issue: *your* merge messages suck too. > They don't actually talk about what you are merging and why. > > A merge is a commit, and needs to have a message, unless it's really > really obvious (and they seldom are - merges are generally a lot less > obvious than most non-merge commits). So just saying > > Merge branch 'for-4.11/block' into for-4.11/linus-merge > > Signed-off-by: Jens Axboe > > is simply not an acceptable merge message. What are you merging, and why? > > Please. We've been very good at having good commit messages in the > kernel. Merges need good commit messages too! You are right, and honestly I don't think I've ever done merge commit messages for my own merges, I only do it when I merge other peoples branches. I'll improve on it. -- Jens Axboe