From mboxrd@z Thu Jan  1 00:00:00 1970
From: mathieu.desnoyers@efficios.com (Mathieu Desnoyers)
Date: Tue, 19 Sep 2017 17:07:52 +0000 (UTC)
Subject: [lttng-dev] [PATCH lttng-ust 1/2] Fix: lttng-gen-tp: only
 replace file extension
In-Reply-To: <20170919160834.23713-1-jonathan.rajotte-julien@efficios.com>
References: <20170919160834.23713-1-jonathan.rajotte-julien@efficios.com>
Message-ID: <1971304394.13635.1505840872725.JavaMail.zimbra@efficios.com>

Merged into master, 2.10, 2.9, thanks!

Mathieu

----- On Sep 19, 2017, at 12:08 PM, Jonathan Rajotte jonathan.rajotte-julien at efficios.com wrote:

> Previous replace was done on the complete path. A path containing
> .c or .o would result in a corrupted file path.
> 
> Reported-by: Gunnar Strand <Gunnar.Strand at ericsson.com>
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
> ---
> tools/lttng-gen-tp | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
> 
> diff --git a/tools/lttng-gen-tp b/tools/lttng-gen-tp
> index b62cd990..6aa1bba1 100755
> --- a/tools/lttng-gen-tp
> +++ b/tools/lttng-gen-tp
> @@ -79,7 +79,9 @@ class CFile:
>     def write(self):
>         outputFile = open(self.outputFilename,"w")
> 
> -        headerFilename = self.outputFilename.replace(".c",".h")
> +        headerFilename = self.outputFilename
> +        if headerFilename.endswith(".c"):
> +            headerFilename = headerFilename[:-2] + ".h"
> 
>         outputFile.write(CFile.FILE_TPL.format(
>                                            headerFilename = headerFilename))
> @@ -126,7 +128,10 @@ class ObjFile:
>         return cc
> 
>     def write(self):
> -        cFilename = self.outputFilename.replace(".o",".c")
> +        cFilename = self.outputFilename
> +        if cFilename.endswith(".o"):
> +            cFilename = cFilename[:-2] + ".c"
> +
>         cc = self._detectCC()
>         if cc == "":
>             raise RuntimeError("No C Compiler detected")
> --
> 2.11.0

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