From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40737) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aHD1k-0004Lf-04 for qemu-devel@nongnu.org; Thu, 07 Jan 2016 11:03:04 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aHD1e-0000h8-Uf for qemu-devel@nongnu.org; Thu, 07 Jan 2016 11:03:03 -0500 Received: from mx1.redhat.com ([209.132.183.28]:54678) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aHD1e-0000ga-QO for qemu-devel@nongnu.org; Thu, 07 Jan 2016 11:02:58 -0500 References: <1449749584-23214-1-git-send-email-frankja@linux.vnet.ibm.com> <1449749584-23214-30-git-send-email-frankja@linux.vnet.ibm.com> <568E872F.3010300@redhat.com> <568E8A38.8090201@linux.vnet.ibm.com> From: Paolo Bonzini Message-ID: <568E8C2D.4050608@redhat.com> Date: Thu, 7 Jan 2016 17:02:53 +0100 MIME-Version: 1.0 In-Reply-To: <568E8A38.8090201@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 29/34] scripts/kvm/kvm_stat: Move to argparse and add description List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Janosch Frank , qemu-devel@nongnu.org Cc: cornelia.huck@de.ibm.com On 07/01/2016 16:54, Janosch Frank wrote: > On 01/07/2016 04:41 PM, Paolo Bonzini wrote: >> >> >> On 10/12/2015 13:12, Janosch Frank wrote: >>> The OptionParser is deprecated since the introduction of the >>> ArgumentParser in 2.7. >>> >>> Additionally added a description text for the script, so new users >>> don't have to guess its purpose and inner workings. >> >> QEMU still supports 2.6, so this cannot be done. >> >> Paolo >> > > I guess you don't want to have the dependency on the python-argparse > package which is provided down to 2.3? I would at least make it a separate patch, because unlike other patches in this series it would require a mention in the release notes. One would also have to check whether the supported Linux distributions include it (for example, RHEL6 only includes it since RHEL6.4; I don't know about SLES-11). Paolo