[lttng-dev] [PATCH lttng-ust] Fix: Corrected python version assumptions in lttng-gen-tp

[Mathieu Desnoyers]

----- Original Message -----
> From: jderehag at hotmail.com
> To: lttng-dev at lists.lttng.org
> Cc: "Jesper Derehag" <jderehag at hotmail.com>
> Sent: Monday, March 24, 2014 10:29:44 AM
> Subject: [lttng-dev] [PATCH lttng-ust] Fix: Corrected python version	assumptions in lttng-gen-tp
> 
> From: Jesper Derehag <jderehag at hotmail.com>
> 
> This modification calls the interpreter through env instead of
> directly, which should be more portable for those distros which do not
> install
> python into /usr/bin/.
> 
> Also, since lttng-gen-tp is not python2 compatible, I changed it to strictly
> call python3 which is the recommended way.
> See PEP 394.
> http://legacy.python.org/dev/peps/pep-0394/

Hi Jesper,

I somehow thought that lttng-gen-tp was meant to work with
Python 2.7 and Python 3. But maybe I'm missing something.
Where have you seen the dependency of this script on python
3 in our docs ?

Thanks,

Mathieu


> ---
>  tools/lttng-gen-tp |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/lttng-gen-tp b/tools/lttng-gen-tp
> index c49e8a5..c7bf295 100755
> --- a/tools/lttng-gen-tp
> +++ b/tools/lttng-gen-tp
> @@ -1,4 +1,4 @@
> -#!/usr/bin/python
> +#!/usr/bin/env python3
>  #
>  # Copyright (c)  2012 Yannick Brosseau <yannick.brosseau at gmail.com>
>  #
> --
> 1.7.9.5
> 
> 
> _______________________________________________
> lttng-dev mailing list
> lttng-dev at lists.lttng.org
> http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
> 

-- 
Mathieu Desnoyers
EfficiOS Inc.
http://www.efficios.com