fixinclude: shell-quote testing file

Message ID	965D691B-295D-4BF9-93DC-33B63B53634F@adacore.com
State	New
Headers	show Return-Path: <gcc-patches-return-291946-incoming=patchwork.ozlabs.org@gcc.gnu.org> From: Tristan Gingold <gingold@adacore.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: quoted-printable Subject: [Patch] fixinclude: shell-quote testing file Date: Mon, 16 May 2011 15:15:33 +0200 Message-Id: <965D691B-295D-4BF9-93DC-33B63B53634F@adacore.com> To: "gcc-patches@gcc.gnu.org Patches" <gcc-patches@gcc.gnu.org>, bkorb@gnu.org Mime-Version: 1.0 (Apple Message framework v1084) Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk Sender: gcc-patches-owner@gcc.gnu.org

Message ID

965D691B-295D-4BF9-93DC-33B63B53634F@adacore.com

State

New

Headers

From: Tristan Gingold <gingold@adacore.com>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: quoted-printable
Subject: [Patch] fixinclude: shell-quote testing file
Date: Mon, 16 May 2011 15:15:33 +0200
Message-Id: <965D691B-295D-4BF9-93DC-33B63B53634F@adacore.com>
To: "gcc-patches@gcc.gnu.org Patches" <gcc-patches@gcc.gnu.org>,
	bkorb@gnu.org
Mime-Version: 1.0 (Apple Message framework v1084)
Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
Sender: gcc-patches-owner@gcc.gnu.org

Commit Message

Tristan Gingold May 16, 2011, 1:15 p.m. UTC

Hi,

one of the main VMS include files has a dollar in its name (decc$types.h), which is not shell friendly.

As a consequence, when the name is reused by check.tpl to create a baseline file, it must be quoted.
This is what is implemented in this patch.

Tested by 'make check' in fixincludes/

Ok for trunk ?

Tristan.

fixincludes/
2011-05-16  Tristan Gingold  <gingold@adacore.com>

	* check.tpl: Shell-quote testing file.

Comments

Andreas Schwab May 16, 2011, 1:26 p.m. UTC | #1

Tristan Gingold <gingold@adacore.com> writes:

> 2011-05-16  Tristan Gingold  <gingold@adacore.com>
>
> 	* check.tpl: Shell-quote testing file.
>
> diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl
> index b239d8b..0f3937a 100644
> --- a/fixincludes/check.tpl
> +++ b/fixincludes/check.tpl
> @@ -87,7 +87,7 @@ FOR fix  =][=
>    ELSE   =]
>  cat >> [=
>      IF (exist? "files") =][=
> -      files[0] =][=
> +      (string-substitute (get "files[0]") "$" "\\$") =][=

If you want to reliably shell-quote the string you should use
raw-shell-str.

Andreas.

Bruce Korb May 16, 2011, 1:33 p.m. UTC | #2

On Mon, May 16, 2011 at 6:15 AM, Tristan Gingold <gingold@adacore.com> wrote:
> Hi,
>
> one of the main VMS include files has a dollar in its name (decc$types.h), which is not shell friendly.
>
> As a consequence, when the name is reused by check.tpl to create a baseline file, must be quoted.

> Ok for trunk ?

> diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl
> index b239d8b..0f3937a 100644
> --- a/fixincludes/check.tpl
> +++ b/fixincludes/check.tpl
> @@ -87,7 +87,7 @@ FOR fix  =][=
>   ELSE   =]
> -cat >> [=
> -   IF (exist? "files") =][=
> -      files[0] =][=
> -   ELSE =]testing.h[=
> -   ENDIF =] <<_HACK_EOF_

Easier would be to just single (apostrophe) quote the string:

  cat >> '[= (if (exist? "files") (get "files[0]") "testing.h")
            =]' <<_HACK_EOF_

Or, as Andreas said after I typed the above:)

  cat >> [= (raw-shell-str (if (exist? ...))) =] <<_HACK_EOF_

Tristan Gingold May 16, 2011, 1:33 p.m. UTC | #3

On May 16, 2011, at 3:26 PM, Andreas Schwab wrote:

> Tristan Gingold <gingold@adacore.com> writes:
> 
>> 2011-05-16  Tristan Gingold  <gingold@adacore.com>
>> 
>> 	* check.tpl: Shell-quote testing file.
>> 
>> diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl
>> index b239d8b..0f3937a 100644
>> --- a/fixincludes/check.tpl
>> +++ b/fixincludes/check.tpl
>> @@ -87,7 +87,7 @@ FOR fix  =][=
>>   ELSE   =]
>> cat >> [=
>>     IF (exist? "files") =][=
>> -      files[0] =][=
>> +      (string-substitute (get "files[0]") "$" "\\$") =][=
> 
> If you want to reliably shell-quote the string you should use
> raw-shell-str.

Works for me too.  Thanks.

Tristan.

diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl
index b239d8b..0f3937a 100644
--- a/fixincludes/check.tpl
+++ b/fixincludes/check.tpl
@@ -87,7 +87,7 @@  FOR fix  =][=
   ELSE   =]
 cat >> [=
     IF (exist? "files") =][=
-      files[0] =][=
+      (string-substitute (get "files[0]") "$" "\\$") =][=
     ELSE =]testing.h[=
     ENDIF =] <<_HACK_EOF_