Hi!
I have made some progress with your help. I have fixed the sed part:
(1) there were missing 's' in the scripts, first I did not noticed it,
then I did not know if I was supposed to povide it;
(2) I have replaced the [ \t]+ by [ \t][ \t]*
to get:
--- ../_gcc_clean/fixincludes/inclhack.def
On Tue, 7 Apr 2009, Dominique Dhumieres wrote:
> +sed = "/#define[ \t]+INTPTR_MIN[\t]+INT64_MIN/#define INTPTR_MIN
> ((intptr_t) INT64_MIN)/";
> +sed = "/#define[ \t]+INTPTR_MIN[\t]+INT32_MIN/#define INTPTR_MIN
> ((intptr_t) INT32_MIN)/";
> +sed = "/#define[ \t]+INTPTR_MAX[\t]+INT64_
>> sed does not have +.
>
> Thanks for the hint. Apparently GNU sed version 4.1.5 has it, provided you
> use -r,
It also has \+ without -r, but neither -r nor \+ are portable.
Paolo
Paolo,
> sed does not have +.
Thanks for the hint. Apparently GNU sed version 4.1.5 has it, provided you use
-r,
but I was wondering what to do since I did not see it in fixincl.x.
So I will use [ \t][ \t]*.
Dominique
Dominique Dhumieres wrote:
> Dave,
>
> I have looked more closely to the sed part and it is probably incorrect,
> i.e. doing nothing. I'll coninue to experiment, but if you or someone else
> have idea, I'll be glad to use it.
sed does not have +. You need to use [ \t][ \t]* assuming that autoge
Dave,
I have looked more closely to the sed part and it is probably incorrect,
i.e. doing nothing. I'll coninue to experiment, but if you or someone else
have idea, I'll be glad to use it.
Thanks,
Dominique
Dominique Dhumieres wrote:
> I have reported the error in pr445#15 and explained in comment #6:
PR448, to be precise. I see Joseph is back from his AFK and has added
comment#16.
> /opt/gcc/gcc-4.5-work/gcc/testsuite/gcc.dg/c99-stdint-1.c: In function
> 'test_ptr': /opt/gcc/gcc-4.5-work/gcc/te
Dave,
Thanks for the quick answer.
> I don't know what it's trying to tell you with the fixincludes FAIL. Did
> you verify manually if the fixes perhaps didn't match against the stdint.h you
> have on your release of the O/S?
AFAICT the patch looks fine, but I cannot rule out typos. I'll try
Dominique Dhumieres wrote:
> FX Coudert has sent me the following patch for fixincludes/inclhack.def:
[ snipped all but one representative line. ]
> +sed = "/#define[ \t]+INTPTR_MIN[\t]+INT64_MIN/#define INTPTR_MIN
> ((intptr_t) INT64_MIN)/";
> I have succeeded to regenerate fixincludes/fi
Hi,
FX Coudert has sent me the following patch for fixincludes/inclhack.def:
--- ../_gcc_clean/fixincludes/inclhack.def 2009-03-31 22:37:57.0
+0200
+++ fixincludes/inclhack.def2009-04-06 19:50:43.0 +0200
@@ -1023,6 +1023,35 @@
/*
+ * Fix stdint.h header on Darwin.
10 matches
Mail list logo
