[lttng-dev] [PATCH lttng-ust] Fix: .split() the CC environment variable in lttng-gen-tp

[Mathieu Desnoyers]

merged, thanks!

Mathieu

----- Original Message -----
> From: "Jérémie Galarneau" <jeremie.galarneau at efficios.com>
> To: lttng-dev at lists.lttng.org
> Cc: "mathieu desnoyers" <mathieu.desnoyers at efficios.com>, "jan glauber" <jan.glauber at gmail.com>, "Jérémie Galarneau"
> <jeremie.galarneau at efficios.com>
> Sent: Tuesday, April 8, 2014 11:28:23 AM
> Subject: [PATCH lttng-ust] Fix: .split() the CC environment variable in lttng-gen-tp
> 
> The CC variable might contain command line arguments which lttng-gen-tp
> currently does not handle. This fix ensures that the arguments are
> properly passed on to the compiler binary as if a normal Makefile
> was used.
> 
> Signed-off-by: Jérémie Galarneau <jeremie.galarneau at efficios.com>
> ---
>  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 a3a01f2..b62cd99 100755
> --- a/tools/lttng-gen-tp
> +++ b/tools/lttng-gen-tp
> @@ -94,7 +94,7 @@ class ObjFile:
>          if 'CC' in os.environ:
>              cc = os.environ['CC']
>              try:
> -                subprocess.call(cc,
> +                subprocess.call(cc.split(),
>                                  stdout=subprocess.PIPE,
>                                  stderr=subprocess.PIPE)
>              except OSError as msg:
> --
> 1.9.1
> 
> 

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