On 06/26/14 02:18, Rainer Orth wrote:
Ok for mainline?
Could you please reformat the c_fix_arg's and test-text to be here strings a
la:
c_fix_arg = - _EOS_
#undef signbit
#define signbit(x) (sizeof(x) == sizeof(float) \
\ ?
Hi Rainer,
On Tue, Jul 1, 2014 at 4:22 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
It's not yet in autogen 5.9: I've diffed the fixincl.x generated with my
original patch and the amended one and those backslashes after the
leading tab are still there.
5.9 is 7 years old now.
On 04/16/14 04:16, Rainer Orth wrote:
I've already verified that trunk fails to build no sparc-sun-solaris2.9
and i386-pc-solaris2.9. Bootstraps on {i386,sparc}-*-solaris2.{10,11}
(and x86_64-unknown-linux-gnu for good measure) are in progress. I'll
verify that there are no unexpected
On 01/27/14 18:20, Hans-Peter Nilsson wrote:
You'd need some additional conditions. There might be the
additional issue that any lex is expected to work too, not
just flex.
It isn't committed 'cuz nobody said, Okay.
I do wish either someone would say, Okay. or come up with something
that
In retrospect, it occurs to me that a am-i-ready-to-build.sh script
in the contrib directory might be useful, too.
On 10/21/14 02:30, Uros Bizjak wrote:
2014-10-21 Uros Bizjak ubiz...@gmail.com
* inclhack.def (glibc_c99_inline_4): Add pthread.h to files.
* fixincl.x: Regenerate.
Bootstrapped and regression tested on CentOS 5.11 x86_64-linux-gnu {,-m32}.
OK for mainline?
On 10/25/14 10:40, Bruce Korb wrote:
On 10/21/14 02:30, Uros Bizjak wrote:
2014-10-21 Uros Bizjak ubiz...@gmail.com
* inclhack.def (glibc_c99_inline_4): Add pthread.h to files.
* fixincl.x: Regenerate.
Bootstrapped and regression tested on CentOS 5.11 x86_64-linux-gnu {,-m32}.
OK
On 12/30/12 01:42, Paolo Bonzini wrote:
Not my territory anymore, but it looks much better! CCing Bruce.
Hi Alexandre,
Long time. It's no wonder you've forgotten this little world! :)
Anyway, please make the expressions more readable and strip
out the generated text from the review message.
On 01/06/13 12:12, Alexandre Oliva wrote:
On Dec 30, 2012, Bruce Korb bk...@gnu.org wrote:
On 12/30/12 01:42, Paolo Bonzini wrote:
Not my territory anymore, but it looks much better! CCing Bruce.
Long time. It's no wonder you've forgotten this little world! :)
Indeed!
Anyway
On Mon, Jan 7, 2013 at 11:49 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
I think a patch for each directory will need posting separately for review
of such things as whether any imported / generated files are mistakenly
changed.
So fixincludes/ separate from gcc/, and every
Hi,
On Mon, Jan 14, 2013 at 12:42 AM, Richard Biener rguent...@suse.de wrote:
fixincludes mixes common declarations (in fixlib.h) and non-common
decls (where they are defined) for verbose_level and fixinc_mode.
XCode on Mountain Lion doesn't like this and its linker (rightfully)
complains.
On 02/03/13 02:19, Richard Sandiford wrote:
Not exactly the most exciting patches, and certainly not worth more than
one ping, but:
fixincludes copyright
http://gcc.gnu.org/ml/gcc-patches/2013-01/msg00442.html
You missed the file header. Why bother with dual update issues?
---
On 02/03/13 09:42, Richard Sandiford wrote:
You missed the file header.
That was deliberately in patch 2 though.
OK.
Why bother with dual update issues?
Well, the point is that patch 2 is scripted.
OK:
echo $copyright | sed 's/(C) 2002-/(C) /'
and now you print the right date and
On Mon, Feb 4, 2013 at 11:55 AM, Ben Elliston b...@air.net.au wrote:
On Mon, Feb 04, 2013 at 06:59:38PM +, Richard Sandiford wrote:
These days the guideline encourage updating all files, even ones
that haven't changed, so I was hoping we could do that gcc-wide.
If that is what the
I stumbled over this code using a source code analyzer.
It incorrectly assumed that switch labels cannot be inserted
into random places inside of enclosed compound statements.
It correctly assumes that you shouldn't be doing that. :)
I propose making this change solely for aesthetics:
static
Hi Tristan,
On Thu, Sep 22, 2011 at 6:59 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
currently alpha-vms is listed as a particular target in
fixincludes/mkfixinc.sh but ia64-vms isn't.
As I will submit a patch to add some rules for both alpha and ia64 VMS, first
fixincludes must be
On 09/22/11 08:00, Tristan Gingold wrote:
Hi,
this patch adds some fixes in fixincludes specific to VMS.
Until now we were using a manually modified subset of the VMS headers, but
using fix includes is the right way.
Ok for mainline ?
Looks good to me. Please, thank you.
Hi Tristan,
On Tue, Apr 17, 2012 at 5:57 AM, Tristan Gingold ging...@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
Hi,
When I approved a patch in 2008, there was a typo. I didn't
notice and it was fixed by removing a formatting element.
Your patch corrects the error.
Please apply your changes to active branches. Thank you!
Regards, Bruce
On 08/22/12 17:05, rbmj wrote:
Hello Everyone,
I have ten patches which are approved or obvious but waiting on commit
The include fixing stuff looks fine to me.
However I think it might be simpler to tweak mkfixinc.sh to
sed '/if test -s .{MACRO_LIST}/s/$/ false/' \
On 08/24/12 11:50, rbmj wrote:
On 8/22/2012 8:52 PM, Bruce Korb wrote:
However I think it might be simpler to tweak mkfixinc.sh to
sed '/if test -s .{MACRO_LIST}/s/$/ false/' \
${srcdir}/fixinc.in ${target}
for vxworks rather than all that configury rigmarole.
That would eliminate
Hi Robert,
If you are going to defer, then:
On Fri, Aug 24, 2012 at 1:20 PM, rbmj r...@verizon.net wrote:
diff --git a/fixincludes/fixinc.in b/fixincludes/fixinc.in
index e73aed9..de7be35 100755
--- a/fixincludes/fixinc.in
+++ b/fixincludes/fixinc.in
@@ -128,6 +128,18 @@ fi
# # # # # # #
Hi Robert,
On Thu, Aug 30, 2012 at 6:30 AM, rbmj r...@verizon.net wrote:
Done, and patch is attached.
OK. make install doesn't seem to like it as much as I do. It complains
because it tries to install macro_list and can't find it. Proposed
solutions:
2. Change line to read test -f
On 09/09/12 08:54, rbmj wrote:
Just because I *love* bothering everyone with emails...
I don't mind, as long as you don't expect me to do anything
until I'm certain you've stabilized the patch ;)
I'm glad you rolled it up into one patch, because I was
eventually going to ask you to do that.
Hi,
On Mon, Sep 10, 2012 at 10:48 AM, rbmj r...@verizon.net wrote:
On the other hand, I've read this on the website:
Don't mix together changes made for different reasons. Send them
individually. Ideally, each change you send should be impossible to
subdivide into
parts that we might want
On 07/25/11 08:17, Rainer Orth wrote:
As discussed in the PR, the Solaris 10+spawn.h header needs a fix to
make it work with g++. The following patch implements it.
It passed a i386-pc-solaris2.11 bootstrap without regressions and make
check in fixincludes works without failures.
Ok for
On 06/06/12 09:33, rbmj wrote:
Hi everyone,
OK, I'm slow. I just woke up from a late night. test -r vxWorks.h
is the mechanism to ensure that tests only fire on a vxworks platform.
Seems a bit obscure, but if it is easier than other mechanisms, then
fine. I do think a mach test would be
RE: http://gcc.gnu.org/ml/gcc-help/2012-05/msg00245.html
perhaps it would be useful to echo a warning into the
stderr stream when platforms get bypassed?
On Sun, Jun 10, 2012 at 10:57 AM, Nathan Sidwell nat...@acm.org wrote:
On 06/06/12 17:33, rbmj wrote:
Hi everyone,
This patch series is the result of this [1] thread about fixincludes on
VxWorks.
It resolves bugs 53457 and 53378, and a few other issues that previously
required manual
Hi David,
On Tue, Jun 19, 2012 at 7:16 AM, David Edelsohn dje@gmail.com wrote:
Okay?
Okay.
Cheers - Bruce
On 06/24/12 15:54, rbmj wrote:
+ c_fix_arg = %0\n
+ #ifdef IN_GCC\n
+ #define mkdir(dir, mode) ((mode), (mkdir)(dir))\n
+ #endif\n;
+ c_fix_arg = extern[\t ]+STATUS[\t ]+mkdir[\t ]*
+ \\([\t ]*const[\t
On Mon, Nov 7, 2011 at 12:59 PM, David Edelsohn dje@gmail.com wrote:
Okay?
Well, okay, but I'd prefer one fix something like the following (and I confess
I've not had any test time...) because it looks simpler to me:
/*
* pthread.h on AIX defines PTHREAD_ONCE_INIT without enough braces.
Hi David,
On Tue, Nov 8, 2011 at 10:20 AM, David Edelsohn dje@gmail.com wrote:
Okay?
Well, okay, but
Also, there are other fixincludes fixes for missing braces that
directly add the braces, not a wrapper. They are not multi-line
macros, but there is a precedent on the style of
Hi Tristan,
On Mon, May 16, 2011 at 6:39 AM, Tristan Gingold ging...@adacore.com wrote:
Sure. [If] The patch below is ok for 'make check', is it also ok for you ?
Yes, thank you.
Hi Tristan,
On Mon, May 16, 2011 at 8:27 AM, Tristan Gingold ging...@adacore.com wrote:
Yes, it is ok on GNU/Linux and Solaris.
Do I really need to test on HP/UX and AIX ? Won't be easy for me.
Solaris and BSD are usually the most unusual, so I'd say go ahead for mainline
and see if you get
On Tue, Aug 9, 2011 at 2:35 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Ok for mainline (no point in backporting to release branches since this
will never be used)?
Hi Rainer,
never be used? OK. Looks good to me. Cheers - Bruce
On 03/05/12 09:01, Rainer Orth wrote:
This is where I need explicit approval and/or guidance:
* There are some fixincludes hacks that from their names seem to be
osf-specific, but are not restricted to alpha*-dec-osf*. Bruce,
what's the best way to handle those? Disable them e.g. with a
CF:
fixincludes:
* inclhack.def (alpha___extern_prefix): Remove.
(alpha___extern_prefix_standards): Remove.
(alpha___extern_prefix_sys_stat): Remove.
(alpha_bad_lval): Remove.
(alpha_pthread): Remove.
(alpha_pthread_gcc): Remove.
On Mon, Mar 5, 2012 at 3:13 PM, Joseph S. Myers jos...@codesourcery.com wrote:
On Mon, 5 Mar 2012, Rainer Orth wrote:
* There are some fixincludes hacks that from their names seem to be
osf-specific, but are not restricted to alpha*-dec-osf*. Bruce,
what's the best way to handle those?
On Mon, Mar 12, 2012 at 3:05 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
this patch adds a few fix include entries for VMS so that:
[things work]
Looks good to me.
On Mon, Mar 12, 2012 at 10:03 AM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
I'm happy with the stuff going away! :)
I suppose after a decade of being gone, it's time to stop referencing it.
OK for *ALL* active branches.
On Wed, Mar 14, 2012 at 2:36 AM, Tristan Gingold ging...@adacore.com wrote:
Hi,
fixinc.sh is not anymore in gcc/ directory. So I propose to adjust
fixincludes/README.
Ok for trunk ?
Hi Rainer,
On Wed, Mar 21, 2012 at 9:25 AM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
As reported in PR other/52626, make check in fixincludes is currently
failing since I neglected to adapt the baseline for the Solaris 8
removal ;-( I always meant to run make check, but forgot.
On
Rather simple... In a file called fixinclude.sum, put
PASS: unique string
or
FAIL: unique string
one per line,
Bonus points if you can total passes and failures:
I think you just put it into the check template so it creates the result.
Since multiple fixes tweak the same file,
On Fri, Mar 23, 2012 at 11:47 AM, David Edelsohn dje@gmail.com wrote:
Some early releases of AIX 6 include a malloc.h header with an
incorrect use of builtin keyword:
#ifdef __cplusplus
extern C {
extern builtin char *__alloca (size_t);
# define alloca __alloca
}
#endif /* def
To eliminate any possible ambiguity, the patch is approved
On 01/10/12 23:49, andreast at gcc dot gnu.org wrote:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
Andreas Toblerandreast at gcc dot gnu.org changed:
What|Removed |Added
Sure. Looks fine. Please apply to all active branches.
On Sat, Aug 31, 2013 at 4:40 PM, David Edelsohn dje@gmail.com wrote:
On 11/06/13 15:29, Ian Lance Taylor wrote:
When fenv.h is not fixed, libquadmath does not build.
This patch works around the problem. Bootstrapped and tested on
x86_64-unknown-linux-gnu.
OK for mainline?
Hi Ian,
Yes, please.
This time, I'm on my dev box and looked at the code.
You
So is this the right patch?
$ svn diff inclhack.def
Index: inclhack.def
===
--- inclhack.def(revision 204533)
+++ inclhack.def(working copy)
@@ -1738,7 +1738,7 @@
versions. */
fix = {
hackname =
branch.
Ian
On Thu, Nov 7, 2013 at 10:16 AM, Ian Lance Taylor i...@google.com wrote:
On Thu, Nov 7, 2013 at 8:48 AM, Bruce Korb bk...@gnu.org wrote:
This time, I'm on my dev box and looked at the code.
You remembered correctly that the first file name in the list
of file names needs
Sure. Looks good to me. Thanks
On Fri, Nov 8, 2013 at 2:57 AM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On 4 April 2013 22:20, Bruce Korb bk...@gnu.org wrote:
Except as noted below, fine by me.
On 04/04/13 12:56, Bernhard Reutner-Fischer wrote:
Bootstrapped and regtested
Hi Doug,
On Fri, Jun 3, 2011 at 10:00 AM, Douglas B Rupp r...@gnat.com wrote:
I've been working on a resurrection patch for Interix, one remaining problem
is fixincludes/fixincl.c must be compiled on native Interix with
-D_ALL_SOURCE in order to find the getpagesize prototype.
This works
On 06/04/11 13:43, Douglas B Rupp wrote:
Here's my proposed patch, along the lines you suggested.
Hi Doug,
Excellent. Just a couple nits:
It is more normal and easier to read when you quote the entire shell
script fragment, as below (removing unnecessary eval, too):
diff -rupN
On 06/05/11 21:16, Peter O'Gorman wrote:
Ok for trunk?
Peter
Index: ChangeLog
===
--- ChangeLog (revision 174678)
2011-??-?? Peter O'Gorman po...@thewrittenword.com
* inclhack.def (aix_net_if_arp): New fix.
Hi Richard,
On Mon, Jul 4, 2011 at 4:04 AM, Richard Guenther rguent...@suse.de wrote:
It happens that OpenBSD suffers from a bogus fixinclude that changes
its perfectly valid NULL define from (void *)0 to 0. The fix itself
appears to be very old and is completely bogus - it replaces
(void
Say, what? Anyone know what this is about?
cp ../.././gcc/../fixincludes/README-fixinc include-fixed/README
chmod a+r include-fixed/README
echo timestamp stmp-int-hdrs
TARGET_CPU_DEFAULT= \
HEADERS=auto-host.h ansidecl.h DEFINES=USED_FOR_TARGET \
/bin/sh ../.././gcc/mkconfig.sh tconfig.h
The first two patches I've applied. The remaining two are needed to fully
enable building the VxWorks flavor of GCC, but those bits affect parts
outside of fixincludes and there is some breakage somewhere.
All evidence seems to me to show fixincludes still doing its thing correctly,
but somewhere
Hi Robert,
On Thu, Nov 1, 2012 at 6:35 AM, rbmj r...@verizon.net wrote:
and now my patches will build on top of
trunk. Bruce, can you give steps on how to reproduce the error you
reported?
rm -rf gcc-bld gcc-ins
cp -l gcc-svn gcc-bld
pfx=$PWD/gcc-ins
cd gcc-bld
./configure
Index: gcc/gcov-io.c
===
--- gcc/gcov-io.c (revision 193123)
+++ gcc/gcov-io.c (working copy)
@@ -92,7 +92,8 @@
{
/* Read-only mode - acquire a read-lock. */
s_flock.l_type = F_RDLCK;
- fd = open (name,
From my part, I'm willing to push the patch, but I need confirmation
from Paolo and Nathan
because some of it affects code outside of my authority.
On Thu, Sep 20, 2012 at 7:02 PM, rbmj r...@verizon.net wrote:
Ping? Just did a full pull and rebuild today and everything still works :)
Robert
Huh? Sorry. Don't know what else to say:
Per: http://gcc.gnu.org/wiki/GitMirror
$ git svn init -Ttrunk --prefix=origin/ svn+ssh://gcc.gnu.org/svn/gcc
Network connection closed unexpectedly: Unable to connect to a repository at \
URL 'svn+ssh://gcc.gnu.org/svn/gcc': To better debug SSH
On 09/22/12 12:49, Bruce Korb wrote:
Huh? Sorry. Don't know what else to say:
Similar results using straight up svn:
$ bash contrib/gcc_update
Updating SVN tree
svn: E155036: Please see the 'svn upgrade' command
svn: E155036: Working copy '/u/gnu/proj/gcc-svn' is too old (format 10, created
Hi Gaby,
On 09/22/12 13:07, Gabriel Dos Reis wrote:
SVN is behaving unexpectedly. Help, please?
Have you upgraded your system or SVN recently?
Not deliberately. :) Switched openSuSE releases (12.2 now).
If yes, you need to issue the command
$ svn upgrade
It's been crunching on
Does make check complain if a replacement/wrap fix doesn't have a test?
If so, I'll be fixing it when I have time -- towards the end of October.
(The patch I sent a few days ago ought to do it, but I've not fully tested yet.)
Anyway, looks pretty reasonable to me...
On Sat, Sep 29, 2012 at 9:45
Hi David,
Looks good to me, but for a small nit:
On Tue, Apr 30, 2013 at 7:40 AM, David Edelsohn dje@gmail.com wrote:
+c_fix_arg = #ifndef NULL\n
+ #ifdef __cplusplus\n
+ #ifdef __GNUG__\n
+ #define NULL\t__null\n
+ #else\t /*
Hi,
On Tue, Apr 30, 2013 at 9:42 AM, David Edelsohn dje@gmail.com wrote:
I coped c_fix_arg from openbsd_null_definition fix. I don't know if
two different formats for similar fixes would be confusing, but I do
not have a problem using the EOF version.
I haven't reviewed all patches and I
On 05/16/13 06:41, Rainer Orth wrote:
Work is going on to incorporate all applicable fixincludes fixes into
the Solaris headers proper. One fix is currently problematic since it
uses an G++-internal macro (__GXX_EXPERIMENTAL_CXX0X__) where libstdc++
cmath already switched to testing
Index: ChangeLog
===
--- ChangeLog (revision 200737)
+++ ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2013-07-06 Bruce Korb bk...@gnu.org
+
+ * inclhack.def (cdef_cplusplus): removed, per Bug 51776
+
2013-05-23 Alexander Ivchenko
,
but it is protocol to ask. Also, I prefer that the hacks get inserted
alphabetically. So, actually, there are a few small complaints.
$ fgrep -i fixincludes ../MAINTAINERS
fixincludes Bruce Korb bk...@gnu.org
The patch looks pretty reasonable, but I think someone else
should verify
Hi,
On Mon, Jul 8, 2013 at 5:05 AM, Alexander Ivchenko aivch...@gmail.com wrote:
Hi Bruce,
That was my original letter:
Hi,
Could you please take a look at the attached fixinclude patch
that addresses the problem:
We have test fail for gcc.dg/cpp/trad/include.c on Android. The
reason for
On Mon, Jul 8, 2013 at 11:42 AM, Alexander Ivchenko aivch...@gmail.com wrote:
It sounds like it is better to fix the system headers instead. Via a
fixincludes for older headers and have the android folks fix them for
newer releases.
Would that count for verifing? :)
:) Indeed. I confess I
On Mon, Jul 29, 2013 at 6:22 AM, Andrew Haley a...@redhat.com wrote:
There should be a better diagnostic.
If you remember, the start of this thread was:
Why is it that configure worked but stubs-32.h was not found?
That is the correct thing to do. The reply, basically, was:
It's too
204533)
+++ fixincludes/ChangeLog (working copy)
@@ -1,3 +1,9 @@
+2013-09-20 Bruce Korb bk...@gnu.org
+
+ * inclhack.def: many of the headers found under bits/ are
+ often stashed under architecture directories. Apply fixes
+ to those, too. Also, re-ordered a couple of misordered fixes.
+
2013-09
Adjusted for Richard Biener's patch
Index: fixincludes/ChangeLog
===
--- fixincludes/ChangeLog (revision 205790)
+++ fixincludes/ChangeLog (working copy)
@@ -1,3 +1,14 @@
+2013-12-07 Bruce Korb bk...@gnu.org
+
+ * inclhack.def
On 12/08/13 13:06, Gerald Pfeifer wrote:
Lovely. Thank you very much!
$ svn diff
Index: configure.ac
===
--- configure.ac(revision 205790)
+++ configure.ac(working copy)
@@ -1319,10 +1319,17 @@
# Used for
On Sun, Jan 26, 2014 at 9:38 PM, Hans-Peter Nilsson h...@bitrange.com wrote:
On Sun, 8 Dec 2013, Bruce Korb wrote:
On 12/08/13 13:06, Gerald Pfeifer wrote:
Lovely. Thank you very much!
(Looks like nobody replied to this and it isn't committed.)
No, flex isn't required to build GCC
Except as noted below, fine by me.
On 04/04/13 12:56, Bernhard Reutner-Fischer wrote:
Bootstrapped and regtested on x86_64-unknown-linux-gnu and
x86_64-mine-linux-uclibc without regressions, ok for trunk?
fixincludes/ChangeLog:
2013-04-04 Bernhard Reutner-Fischer al...@gcc.gnu.org
Hi,
On Tue, Aug 5, 2014 at 4:35 AM, Alexander Ivchenko aivch...@gmail.com wrote:
Testing for *android* is less than ideal, because of the possibility of
configuring a *-linux* toolchain to have multilibs using various different
C libraries (with -mandroid being used to select the Android
Hi,
On Tue, Aug 5, 2014 at 10:36 AM, enh e...@google.com wrote:
you can see the current version of bionic's stdio.h here:
https://android.googlesource.com/platform/bionic/+/master/libc/include/stdio.h
i'm happy to add any string to the header file that makes things
easier. if you want
, Aug 5, 2014 at 5:09 PM, enh e...@google.com wrote:
does https://android-review.googlesource.com/103445 look okay?
On Tue, Aug 5, 2014 at 12:01 PM, Bruce Korb bk...@gnu.org wrote:
Hi,
On Tue, Aug 5, 2014 at 10:36 AM, enh e...@google.com wrote:
you can see the current version of bionic's
Hi,
On Wed, Aug 6, 2014 at 4:51 AM, Alexander Ivchenko aivch...@gmail.com wrote:
We still have to remove fix for compiler.h:
Correct. Thank you.
Bruce, I think I formally have to ask for your approval again :)
I don't think so. You've selected one of the changes we wrote about,
so With
This is the first time I've touched the fixincludes directory;
is this the correct way to make a change here?
Well, I'd like to see it -- especially since it's your first.
Please send to this gmail account or wait until I get my GNU email this weekend.
Thanks!
Successfully bootstrapped
On 12/05/14 12:32, Jeff Law wrote:1
fixincludes/ChangeLog:
PR other/63613
* inclhack.def (dejagnu_h_make_inline_functions_static): New fix.
* fixincl.x: Regenerate.
* tests/base/dejagnu.h: New.
OK.
No, actually not.
+fix = {
+hackname =
On 12/14/14 14:07, Uros Bizjak wrote:
Hello Bruce!
I have posted a small fixincludes patch [1] that fixes a warning
during the fixincludes build,
Please apply. However, I'm sure it is not the last remaining warning
since new warnings get invented. :)
Thank you - Bruce
2014-12-09 Uros
===
--- fixincludes/ChangeLog (revision 218991)
+++ fixincludes/ChangeLog (working copy)
@@ -1,9 +1,20 @@
+2013-12-07 Bruce Korb bk...@gnu.org
+
+ * fixincludes/fixincl.tpl: add handling for sum selection
+ criteria and clean up layout
On 01/29/15 05:38, Rainer Orth wrote:
So I saw. If all else fails, we can still commit the (ugly/hard to
read) initial version, otherwise libgo won't build on Solaris before
some (quite recent) Solaris 11.2 patch, breaking bootstrap.
Having it work at all seems like a nice feature.
I think
Hi Rainer,
Sorry for the long delay. Anyway:
On 01/28/15 06:12, Rainer Orth wrote:
* In test_text, I had to backslash-escape the trailing \, otherwise they
were eaten up. Whether or not I do this makes no difference for the
generated fixincl.x, but only with the escaping does make
On 01/28/15 10:13, Bruce Korb wrote:
Hi Rainer,
Sorry for the long delay. Anyway:
On 01/28/15 06:12, Rainer Orth wrote:
* In test_text, I had to backslash-escape the trailing \, otherwise they
were eaten up. Whether or not I do this makes no difference for the
generated fixincl.x
On 01/28/15 10:15, Bruce Korb wrote:
On 01/28/15 10:13, Bruce Korb wrote:
Hi Rainer,
Sorry for the long delay. Anyway:
On 01/28/15 06:12, Rainer Orth wrote:
* In test_text, I had to backslash-escape the trailing \, otherwise they
were eaten up. Whether or not I do this makes
On Mon, Feb 9, 2015 at 3:30 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
That worked fine indeed and is considerably more readable than my
previous version.
Excellent! Thank you.
It produced the identical fixincl.x, passed fixincludes make check and
Solaris 10 and 11 bootstraps.
Looks good to me.
On Sun, Feb 15, 2015 at 12:49 PM, David Edelsohn dje@gmail.com wrote:
The stdlib.h header in AIX 4.3 does not correctly declare strtof with
a const char* argument. Users are building the latest releases of GCC
on AIX 4.3 The appended patch from Richard G Daniel uses
On 05/06/15 01:58, Michael Haubenwallner wrote:
Trivial patch for fixincludes.
A) sufficiently trivial that explicit permission ought not be required
B) it is now officially blessed that we can coalesce year lists.
Let's do so, okay?
Am 2015-05-05 um 18:03 schrieb Michael Haubenwallner:
OK. You might consider updating autogen. It seems 5.18 doesn't
handle the version test quite right. Any 5.18.n should do fine. I
guess I didn't test the version test with older versions. :)
On Thu, May 21, 2015 at 6:58 AM, David Edelsohn dje@gmail.com wrote:
The AIX port of GCC is one of
Definitely much better. I won't apply it until the weekend, so
someone else will likely beat me to it. Thank you.
On Mon, Jul 27, 2015 at 7:36 PM, Eric Gallager eg...@gwmail.gwu.edu wrote:
On 7/27/15, Andreas Schwab sch...@linux-m68k.org wrote:
Eric Gallager eg...@gwmail.gwu.edu writes:
Looks good to me...
On Thu, Aug 13, 2015 at 10:33 AM, David Edelsohn dje@gmail.com wrote:
AIX stdio.h header includes code specific for C++ that looks like:
extern C {
#ifdef __cplusplus
#ifdef ferror
#undef ferror
inline int ferror(FILE * _p)
{
return ((_p)-_flag _IOERR);
BTW, OK by me :) Now that I'm retired, it is starting to look like
less time for this stuff... ;)
On Thu, Jun 9, 2016 at 10:25 AM, David Edelsohn wrote:
> Index: inclhack.def
> ===
> --- inclhack.def
On 01/31/16 17:03, John David Anglin wrote:
The attached hack fixes missing const from the first argument of the
declararation for vsscanf on hpux.
Approved for all active dev branches.
Looks good to me.
On Sat, Aug 13, 2016 at 9:55 AM, John David Anglin wrote:
> Currently, trunk fails to boot fortran on hpux because of the following error:
>
> /xxx/gnu/gcc/objdir/./prev-gcc/xg++ -B/xxx/gnu/gcc/objdir/./prev-gcc/
>
On Wed, Jul 13, 2016 at 11:39 AM, Marek Polacek wrote:
> Most likely what you saw was in cxx_pretty_printer::declaration_specifiers.
I only saw it once and, of course, it was once too often. ;)
If you fix it, it would sooth my sensibilities as the fixincludes maintainer,
> On Mon, Jul 11, 2016 at 01:36:02PM -0700, Bruce Korb wrote:
> [[putrid code deleted]]
>> Does this patch mean that the above got fixed? I mean, if you're
>> going to fret over linguistic tags to make falling through explicit,
>> it would seem the above code is pretty
1 - 100 of 132 matches
Mail list logo