Message ID | 1470054511-31217-1-git-send-email-christian.ehrhardt@canonical.com (mailing list archive) |
---|---|
State | Superseded, archived |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id E4A65532C; Mon, 1 Aug 2016 14:28:34 +0200 (CEST) Received: from youngberry.canonical.com (youngberry.canonical.com [91.189.89.112]) by dpdk.org (Postfix) with ESMTP id C777F4A63 for <dev@dpdk.org>; Mon, 1 Aug 2016 14:28:33 +0200 (CEST) Received: from 1.general.paelzer.uk.vpn ([10.172.196.172] helo=localhost.localdomain) by youngberry.canonical.com with esmtpsa (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.76) (envelope-from <christian.ehrhardt@canonical.com>) id 1bUCKf-0004LD-LX; Mon, 01 Aug 2016 12:28:33 +0000 From: Christian Ehrhardt <christian.ehrhardt@canonical.com> To: christian.ehrhardt@canonical.com, dev@dpdk.org Date: Mon, 1 Aug 2016 14:28:31 +0200 Message-Id: <1470054511-31217-1-git-send-email-christian.ehrhardt@canonical.com> X-Mailer: git-send-email 2.7.4 Subject: [dpdk-dev] [PATCH] examples: fix unusual-interpreter X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Commit Message
Christian Ehrhardt
Aug. 1, 2016, 12:28 p.m. UTC
Due to regular lintian checks in Debian packaging it surfaced that these
two scripts had a space in their #! statement which renders it to be
human, but not shell readable.
Fixes: 8673a3e8 ("examples/ip_pipeline: add config diagram generator")
Fixes: fa667b46 ("examples/ip_pipeline: add core mappings script")
Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com>
---
examples/ip_pipeline/config/diagram-generator.py | 2 +-
examples/ip_pipeline/config/pipeline-to-core-mapping.py | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
Comments
2016-08-01 14:28, Christian Ehrhardt: > Due to regular lintian checks in Debian packaging it surfaced that these > two scripts had a space in their #! statement which renders it to be > human, but not shell readable. [...] > -#! /usr/bin/python2 > +#!/usr/bin/python2 I think we can have a space in the shebang (it works with shells I know). But maybe lintian do not like it (and it is a sufficient reason to accept this trivial patch). However, a better fix would be to run something else than python2, like /usr/bin/env python. Some other python scripts in tools dir may be fixed.
Hi Thomas, I agree on both changes you suggested, but not being the scripts author I wanted to change as few as possible. Also thanks for taking it into consideration even if just for lintian :-) If acceptable to you I'd ask to accept this as-is and consider the patch a head-up for all script owners to change their headers. Christian Ehrhardt Software Engineer, Ubuntu Server Canonical Ltd On Mon, Aug 1, 2016 at 2:50 PM, Thomas Monjalon <thomas.monjalon@6wind.com> wrote: > 2016-08-01 14:28, Christian Ehrhardt: > > Due to regular lintian checks in Debian packaging it surfaced that these > > two scripts had a space in their #! statement which renders it to be > > human, but not shell readable. > [...] > > -#! /usr/bin/python2 > > +#!/usr/bin/python2 > > I think we can have a space in the shebang (it works with shells I know). > But maybe lintian do not like it (and it is a sufficient reason to accept > this trivial patch). > > However, a better fix would be to run something else than python2, > like /usr/bin/env python. > > Some other python scripts in tools dir may be fixed. >
2016-08-01 15:12, Christian Ehrhardt: > On Mon, Aug 1, 2016 at 2:50 PM, Thomas Monjalon <thomas.monjalon@6wind.com> > wrote: > > 2016-08-01 14:28, Christian Ehrhardt: > > > Due to regular lintian checks in Debian packaging it surfaced that these > > > two scripts had a space in their #! statement which renders it to be > > > human, but not shell readable. > > [...] > > > -#! /usr/bin/python2 > > > +#!/usr/bin/python2 > > > > I think we can have a space in the shebang (it works with shells I know). > > But maybe lintian do not like it (and it is a sufficient reason to accept > > this trivial patch). > > > > However, a better fix would be to run something else than python2, > > like /usr/bin/env python. > > > > Some other python scripts in tools dir may be fixed. > > I agree on both changes you suggested, but not being the scripts author I > wanted to change as few as possible. > Also thanks for taking it into consideration even if just for lintian :-) > > If acceptable to you I'd ask to accept this as-is and consider the patch a > head-up for all script owners to change their headers. We can remove the space in every scripts, at least. Then we can wait a little for the opinion of the script authors to do more changes. PS: Please avoid top posting.
> -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Christian Ehrhardt > Sent: Monday, August 1, 2016 1:29 PM > To: christian.ehrhardt@canonical.com; dev@dpdk.org > Subject: [dpdk-dev] [PATCH] examples: fix unusual-interpreter > > Due to regular lintian checks in Debian packaging it surfaced that these > two scripts had a space in their #! statement which renders it to be > human, but not shell readable. > > Fixes: 8673a3e8 ("examples/ip_pipeline: add config diagram generator") > Fixes: fa667b46 ("examples/ip_pipeline: add core mappings script") > > Signed-off-by: Christian Ehrhardt <christian.ehrhardt@canonical.com> > --- Acked-by: Cristian Dumitrescu <cristian.dumitrescu@intel.com> Christian and Thomas, Looking at this email thread, if there is an even better more robust solution, please suggest. It would not hurt to document it in the coding guidelines for scripts. Thanks, Cristian
> -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Thomas Monjalon > Sent: Monday, August 1, 2016 2:26 PM > To: Christian Ehrhardt <christian.ehrhardt@canonical.com> > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [PATCH] examples: fix unusual-interpreter > > 2016-08-01 15:12, Christian Ehrhardt: > > On Mon, Aug 1, 2016 at 2:50 PM, Thomas Monjalon > > <thomas.monjalon@6wind.com> > > wrote: > > > 2016-08-01 14:28, Christian Ehrhardt: > > > > Due to regular lintian checks in Debian packaging it surfaced that > > > > these two scripts had a space in their #! statement which renders > > > > it to be human, but not shell readable. > > > [...] > > > > -#! /usr/bin/python2 > > > > +#!/usr/bin/python2 > > > > > > I think we can have a space in the shebang (it works with shells I > know). > > > But maybe lintian do not like it (and it is a sufficient reason to > > > accept this trivial patch). > > > > > > However, a better fix would be to run something else than python2, > > > like /usr/bin/env python. > > > > > > Some other python scripts in tools dir may be fixed. > > > > I agree on both changes you suggested, but not being the scripts > > author I wanted to change as few as possible. > > Also thanks for taking it into consideration even if just for lintian > > :-) > > > > If acceptable to you I'd ask to accept this as-is and consider the > > patch a head-up for all script owners to change their headers. > > We can remove the space in every scripts, at least. > Then we can wait a little for the opinion of the script authors to do more > changes. > Hi, The script is Python2/3 compatible so remove the space and change to /usr/bin/python or similar. John
diff --git a/examples/ip_pipeline/config/diagram-generator.py b/examples/ip_pipeline/config/diagram-generator.py index f20cbcb..7b1f8d6 100755 --- a/examples/ip_pipeline/config/diagram-generator.py +++ b/examples/ip_pipeline/config/diagram-generator.py @@ -1,4 +1,4 @@ -#! /usr/bin/python2 +#!/usr/bin/python2 # BSD LICENSE # diff --git a/examples/ip_pipeline/config/pipeline-to-core-mapping.py b/examples/ip_pipeline/config/pipeline-to-core-mapping.py index 37b131c..5ffc632 100755 --- a/examples/ip_pipeline/config/pipeline-to-core-mapping.py +++ b/examples/ip_pipeline/config/pipeline-to-core-mapping.py @@ -1,4 +1,4 @@ -#! /usr/bin/python2 +#!/usr/bin/python2 # BSD LICENSE #