https://gcc.gnu.org/bugzilla/show_bug.cgi?id=105719
--- Comment #5 from Bruce Korb ---
It's been a long time since I mucked with fixincludes. My first guess, without
going back and reading code, would be to provide fixincludes with a list of
trees to traverse and not have it burned into fixinclud
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93082
--- Comment #9 from Bruce Korb ---
Um, I didn't finish my misthink for item#1 above. Please ignore. :)
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=93082
--- Comment #8 from Bruce Korb ---
It should not be terribly difficult to add another include tree for fixinc to
search. Also, unless 32 is cast as forever being the only value
kAuthorizationExternalFormLength could possibly ever have:
selec
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91085
--- Comment #15 from Bruce Korb ---
Obviously, "print_quote()" was needed early on (1999) and then saved for
prosperity :). Your patch is inadequate because it will have to not expand
'linux' in a line such as:
#if __has_include()
In other
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91085
--- Comment #14 from Bruce Korb ---
Reworking fixfixes.c seems pretty heavy duty.
I'm downloading the GCC sources now. I'll take a peek tomorrow.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91085
--- Comment #12 from Bruce Korb ---
I'll put it on my to-do list, but I might be participating in a fire evacuation
tonight or tomorrow and I haven't built GCC in several years now. I'm going to
guess that you have to not do the substitution when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88973
Bruce Korb changed:
What|Removed |Added
CC||bkorb at gnu dot org
--- Comment #5 from
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776
Bruce Korb changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Resolution|---
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57792
Bruce Korb changed:
What|Removed |Added
CC||bonzini at gnu dot org,
|
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776
--- Comment #3 from bkorb at gnu dot org 2012-10-08 17:46:04 UTC ---
(In reply to comment #2)
> adding ... PR as a dependency so Bruce gets prodded when it changes state
Bruce has now been prodded. :) I will fix this in 3 weeks or so.
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
--- Comment #47 from bkorb at gnu dot org 2012-01-11 14:31:15 UTC ---
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
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
--- Comment #37 from bkorb at gnu dot org 2012-01-09 22:11:09 UTC ---
It is getting too confusing.
+/*
+ * 'g++ -std=c++11' defines __cplusplus to 201103L, which suggests
+ * that it conforms to ISO/IEC 14882:2011. Until G++ full
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
--- Comment #32 from bkorb at gnu dot org 2012-01-06 16:14:08 UTC ---
(In reply to comment #29)
> +c_fix_arg = "__attribute__\(\(__noreturn__\)\)";
In a double quoted string, the backslash quotes always quote
the character that f
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
--- Comment #30 from bkorb at gnu dot org 2012-01-06 16:10:33 UTC ---
(In reply to comment #28)
> I'll remember to add a comment to this PR once ...
I fully intend to remember, too.
Probably the appropriate thing is a new bug #51776
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776
Bug #: 51776
Summary: fixincludes hacks around a C++ deficiency
Classification: Unclassified
Product: gcc
Version: 4.7.0
Status: UNCONFIRMED
Severity: normal
Priority
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51705
--- Comment #25 from bkorb at gnu dot org 2012-01-06 15:09:46 UTC ---
RE: Comment #16 through Comment #23 -- Do not use sed. There is no compelling
reason to do so. Please read the fixincludes README, wherein "sed" is
disparaged. I
--- Comment #25 from bkorb at gnu dot org 2010-08-30 14:42 ---
> > gcc currently has no middle option.
>
> Also this attribute is not on by default in glibc so you are asking to
> turn on the style based warnings.
(In reply to comment #24)
> FIXED in GCC 4.5
--- Comment #17 from bkorb at gnu dot org 2009-02-28 16:39 ---
Bruce Korb left Veritas 3 years ago. It's called "Symantec" now anyway.
--
bkorb at gnu dot org changed:
What|Removed
--- Comment #4 from bkorb at gnu dot org 2008-12-29 01:24 ---
If GNU Make is only required for the GCC build, then the configure script
for the gcc piece is the only place where such code would be needed.
Anyway, my basic point is that it would be very helpful to not force folks
to find
--- Comment #2 from bkorb at gnu dot org 2008-12-28 23:01 ---
It would seem to me that requiring one to read through installation
documentation is unreasonable when a trivial change to the configure script can
tell the hapless builder what is wrong. You are correct in that it is
--- Comment #6 from bkorb at gnu dot org 2008-01-19 17:23 ---
fixincludes has nothing to do with fixproto, other than both working
with fixing up headers and having names that start with "fix". So,
I know little about it and even less about canadian crosses. "fix-head
--- Comment #1 from bkorb at gnu dot org 2007-07-15 22:53 ---
I'd take ownership, but I cannot.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=32771
--- Comment #15 from bkorb at gnu dot org 2007-02-08 20:53 ---
The commit notice went out to gcc-patches, too. I'd have updated this,
but apparently I need more than gcc-CVS commit permissions to do so.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30678
--- Comment #13 from bkorb at gnu dot org 2007-02-07 21:02 ---
The problem was supposed to have been fixed:
http://gcc.gnu.org/ml/gcc-cvs/2007-02/msg00194.html
http://gcc.gnu.org/ml/gcc-cvs/2007-02/msg00172.html
If you are still having problems, please show the nature of the difficulty
--- Comment #7 from bkorb at gnu dot org 2007-02-03 20:37 ---
Subject: Re: sysmacros.h get currupt from Fixincludes
with updated glibc.
ron3763 at msn dot com wrote:
> --- Comment #6 from ron3763 at msn dot com 2007-02-03 19:33 ---
> Well I guess is the intended purp
--- Comment #5 from bkorb at gnu dot org 2007-02-03 18:32 ---
I'm looking at the inclhack fix now:
> fix = {
>hackname = glibc_c99_inline_4;
>files = sys/sysmacros.h, '*/sys/sysmacros.h';
>bypass= "__STDC_VERSION__";
>c_f
--- Comment #4 from bkorb at gnu dot org 2007-02-02 21:32 ---
I intend to do this tomorrow. I cannot fiddle the bugzilla tho.
It says I'm only authorized to add comments.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30678
--- Comment #2 from bkorb at gnu dot org 2007-02-02 14:57 ---
Thanks for all the detail. I'll fix it to only do that for
"extern" as a full word. (i.e. when not attached to a "__"
prefix or suffix)
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=30678
.2.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: driver
AssignedTo: unassigned at gcc dot gnu dot org
ReportedBy: bkorb at gnu dot org
GCC build triplet: *-*-solaris*
GCC host triplet: *-*-solaris*
GCC target triplet: *-*-so
--- Comment #8 from bkorb at gnu dot org 2005-11-25 07:25 ---
A couple of issues:
1. Replacement fixes should not be checked, so it is interesting that
the failure shows up.
2. fixincl should not create files without terminating new lines anyway.
I've just applied a pat
30 matches
Mail list logo
