From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934068AbbI1O0T (ORCPT ); Mon, 28 Sep 2015 10:26:19 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:55765 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933794AbbI1O0Q (ORCPT ); Mon, 28 Sep 2015 10:26:16 -0400 Date: Mon, 28 Sep 2015 07:09:44 -0700 From: Greg KH To: Alexander Kapshuk Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org Subject: Re: Subject: [PATCH] ver_linux: uniform output across various linux distros Message-ID: <20150928140944.GB20559@kroah.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 28, 2015 at 07:50:37AM +0300, Alexander Kapshuk wrote: > Hello, Hi! First off, your Subject: is a bit odd, you might want to fix up your email client :) > > Having run 'scripts/ver_linux' on my Gentoo system, as well as having > looked through some recent bug reports on the kernel bugzilla website > showing the output of the script in question, I have observed that > the output is not accurate across various distros. While the current > implementation of the script expects the version info to be found in > particular fields, some of the utilities invoked by the script, output > their version information in varying formats, which results in the > script displaying information other than the version number. > > The proposed implementation relies mostly on sed to detect the version > numbers more accurately. Running the patched version of the script on > the distros below resulted in accurate and uniform output. > > Gentoo Linux > Debian 6.0.10 > Oracle Linux Server release 7.1 > Arch Linux > > The items left unchanged are those I did not have access to. I would > be willing to work on those too, if supplied the output of the affected > commands whose format differs based on the distro. Your patch is a bit complex, you are changing the formatting, and at the same time, also changing the code. Please do this in multiple patches, one to fix any issues you see, and the second to then clean up the formatting, making it easier to actually see what the difference is here. Also, take a look at Documentation/SubmittingPatches for the proper format of a kernel patch to ensure we are able to accept it (hint, your patch is missing the signed-off-by line in the body of the changelog area). thanks, greg k-h