From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Wilson Subject: Re: [PATCH] tools/xl: Add 'xen_version' to `xl info` Date: Thu, 11 Jul 2013 10:49:24 -0700 Message-ID: <20130711174922.GA7435@u109add4315675089e695.ant.amazon.com> References: <1373474790-31760-1-git-send-email-andrew.cooper3@citrix.com> <51DE7674.8010508@citrix.com> <51DE7914.7070403@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <51DE7914.7070403@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap Cc: Andrew Cooper , Ian Jackson , Ian Campbell , Xen-devel List-Id: xen-devel@lists.xenproject.org On Thu, Jul 11, 2013 at 10:21:24AM +0100, George Dunlap wrote: > On 11/07/13 10:10, Andrew Cooper wrote: [...] > > > >Even with this however, my original point still stands. Getting the > >version number is awkward. > > Is there actually a reason to print it separately at all? Why not > replace the individual lines with the unified line? Please don't do that. I'm sure that there are scripts out there that stitch together the parts into a version string. They'll break if we remove the parts. --msw