This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Fixinclude]: Fix typo and default to twoprocess on VMS
- From: Bruce Korb <bkorb at gnu dot org>
- To: Tristan Gingold <gingold at adacore dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 17 Apr 2012 07:21:21 -0700
- Subject: Re: [Fixinclude]: Fix typo and default to twoprocess on VMS
- References: <B54A5ED3-8DE8-4C8C-A389-96BAF2E09FCB@adacore.com>
Hi Tristan,
On Tue, Apr 17, 2012 at 5:57 AM, Tristan Gingold <gingold@adacore.com> wrote:
> Hi,
>
> one-process methodology cannot be used on VMS[...]
> But, when twoprocess is selected, gcc emits a warning[...]
> Ok for trunk ?
> diff --git a/fixincludes/configure.ac b/fixincludes/configure.ac
> index e7de791..f1fb2ff 100644
> --- a/fixincludes/configure.ac
> +++ b/fixincludes/configure.ac
> @@ -53,7 +53,8 @@ fi],
> ? ? ? ?i?86-*-msdosdjgpp* | \
> ? ? ? ?i?86-*-mingw32* | \
> ? ? ? ?x86_64-*-mingw32* | \
> - ? ? ? *-*-beos* )
> + ? ? ? *-*-beos* | \
> + ? ? ? ?*-*-*vms*)
> ? ? ? ? ? ? ? ?TARGET=twoprocess
> ? ? ? ? ? ? ? ?;;
This, definitely.
> diff --git a/fixincludes/fixincl.c b/fixincludes/fixincl.c
> index 9f399ab..1133534 100644
> --- a/fixincludes/fixincl.c
> +++ b/fixincludes/fixincl.c
> @@ -829,7 +829,7 @@ fix_with_system (tFixDesc* p_fixd,
> ? ? ? /*
> ? ? ? ?* ?Now add the fix number and file names that may be needed
> ? ? ? ?*/
> - ? ? ?sprintf (pz_scan, " %ld '%s' '%s'", ?(long) (p_fixd - fixDescList),
> + ? ? ?sprintf (pz_scan, " %ld '%s' '%s' '%s'", (long) (p_fixd - fixDescList),
> ? ? ? ? ? ? ? pz_fix_file, pz_file_source, pz_temp_file);
> ? ? }
> ? else /* NOT an "internal" fix: */
This, almost certainly. I'll take a peek at the source and convince myself of
this decade old mistake tomorrow & send my grateful thanks and approval then.
(No access to source today.)
Thank you! Cheers - Bruce