[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] tools/misc/xencov_split: Add python 3 compatibility


  • To: Javi Merino <javi.merino@xxxxxxxxx>
  • From: Jan Beulich <jbeulich@xxxxxxxx>
  • Date: Mon, 4 Sep 2023 08:51:31 +0200
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=TF2lOeCbTErEfk6SsN41ExQh882zt8sOJvz5ZkwPoWY=; b=RciAYagqaaECs5nsfi2v4DRTsBeBONNnwoGN3PZ2Wn7VNRVB2Jx0cdK7OhR8filIP/UG5mynNbADoNl2UOk/uxujUbN4tRs7qFbWe9+HZuR2GKx9Vk8OMFOD6wQdTmr5zp/eOjg/SccseL0DVBpSOtJG7roYzrSlJXtDjw1DqP0+DyqFFVvjq/pIxyBHlFvzi7SwqlrksvoIoQOPZDVTJXp2dE2ZXDDxziIZ5SbiN8wCM0peCPPbiJtHRauxXwcGllJKJeFe+GLi4z0a83vC5WTo4iUHHaxT72zGJjfALMAj2zmjTSxnDrbJM2iTWgJWPKOaMv856bxHD5jEzujOdw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=b4rrbQC7NuJ2YPxVI2oJKowd0y+Kz8pMLe5yYLd568iww2tFcCjggbOGq6OME//pQ7l7DSzq/8mYZe4NmWyKzjV6Pg+yuFgXOT0TvhGSycp8lQKAPwoN8O8fjXe4eK0nIGhA01bjhG99EdGgRAqZ5FXPlRMwOuQ1w6wI5JOunBQZEkq6DwnPjQc0siJ8IRflOjWjREqWWjl/dPkD9/IEQvarAhv1gJ/jp12bSSOuzNhTdg6NMoq4h5JsnAOhvzDVdx1q834Hp+UmLj45HhSHYtMYfihTrDjTAcRgPTi/0vOdDxOX7XOGQypONBvno3ITrfm6RWupk4LdU0qxJd2tNw==
  • Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com;
  • Cc: Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx
  • Delivery-date: Mon, 04 Sep 2023 06:51:39 +0000
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On 02.09.2023 18:21, Javi Merino wrote:
> Closes #154
> 
> Signed-off-by: Javi Merino <javi.merino@xxxxxxxxx>

The title isn't really in line with ...

> --- a/tools/misc/xencov_split
> +++ b/tools/misc/xencov_split
> @@ -1,5 +1,7 @@
> -#!/usr/bin/env python
> +#!/usr/bin/env python3

... this part of the change, and making Py3 a requirement here (assuming
that's indeed necessary) surely wants adding a few words as description.
Grep-ing the tools/ sub-tree I notice that so far we've avoided explicit
uses of "python3", and I assume we would better continue doing so as on
a distro with only Py3 a "python3" alias may legitimately not exist.

Jan



 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.