Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751986AbdIMSzn (ORCPT ); Wed, 13 Sep 2017 14:55:43 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:48634 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751173AbdIMSzl (ORCPT ); Wed, 13 Sep 2017 14:55:41 -0400 Date: Wed, 13 Sep 2017 11:55:38 -0700 From: Greg Kroah-Hartman To: Mark Brown Cc: Guenter Roeck , Tom Gall , LKML , torvalds@linux-foundation.org, akpm@linux-foundation.org, Shuah Khan , patches@kernelci.org, Ben Hutchings , linux- stable Subject: Re: [PATCH 4.9 00/14] 4.9.50-stable review Message-ID: <20170913185538.GA26390@kroah.com> References: <20170912165253.709627159@linuxfoundation.org> <20170913034915.GA21161@kroah.com> <20170913152213.GI27765@roeck-us.net> <20170913163655.nfdhr5gnl4sn4zsz@sirena.org.uk> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170913163655.nfdhr5gnl4sn4zsz@sirena.org.uk> User-Agent: Mutt/1.9.0 (2017-09-02) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1501 Lines: 36 On Wed, Sep 13, 2017 at 09:36:55AM -0700, Mark Brown wrote: > On Wed, Sep 13, 2017 at 08:22:13AM -0700, Guenter Roeck wrote: > > On Wed, Sep 13, 2017 at 10:05:00AM -0500, Tom Gall wrote: > > > > Does it make sense to create tags for the RC(s) so git describe gets > > > it right? Given the right version is in the Makefile kinda feels like > > > that'd be a belt and suspenders approach. > > > Depends. A tag only makes sense if the branch isn't rebased, otherwise > > (if the tag can change) it would be misleading (as would be to report > > the version number from Makefile). > > Rebasing shouldn't be an issue for tags (they're not branches), and > changes would a disaster no matter what. Can you push --force a tag? I've never tried that, don't want to mess up a kernel.org tree by trying it out :) Because of that, I haven't been tagging the -rc trees, as I didn't think it was really needed. The linux-stable-rc tree is just a "convenience" for people to use for testing, it's not really a "cannonical" tree at the moment because of that. > > Given that, I think reporting the SHA is better, since it reports clearly > > which version was tested. > > This definitely makes sense though (especially in a generalized tool), > defensively if nothing else. I think you ideally want both. Yes, use 'make kernelversion' to get the kernel's view of the release number, don't use 'git describe' please, as it does not know about changes to the Makefile (nor should it...) thanks, greg k-h