[Bug target/105719] RFE: fixincludes should handle Frameworks

2023-05-15 Thread bkorb at gnu dot org via Gcc-bugs
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

[Bug target/93082] macOS Authorization.h needs fixinclude

2022-05-15 Thread bkorb at gnu dot org via Gcc-bugs
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. :)

[Bug target/93082] macOS Authorization.h needs fixinclude

2022-05-15 Thread bkorb at gnu dot org via Gcc-bugs
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:

[Bug other/91085] fixincludes breaks

2021-06-27 Thread bkorb at gnu dot org via Gcc-bugs
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

[Bug other/91085] fixincludes breaks

2021-06-27 Thread bkorb at gnu dot org via Gcc-bugs
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.

[Bug other/91085] fixincludes breaks

2020-08-20 Thread bkorb at gnu dot org
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

[Bug tree-optimization/88973] [8/9 Regression] New -Wrestrict warning since r268048

2019-02-19 Thread bkorb at gnu dot org
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

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

2015-03-08 Thread bkorb at gnu dot org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=51776 Bruce Korb bkorb at gnu dot org changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED

[Bug target/57792] fixincludes doesn't honor the use of --with-sysroot during bootstrap

2013-07-04 Thread bkorb at gnu dot org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57792 Bruce Korb bkorb at gnu dot org changed: What|Removed |Added CC||bonzini at gnu dot org

[Bug preprocessor/51776] fixincludes hacks around a C++ deficiency

2012-10-08 Thread bkorb 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.

[Bug bootstrap/51705] [4.7 Regression] FreeBSD uses unsupported C++11 features when __cplusplus == 201103L

2012-01-11 Thread bkorb at gnu dot org
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.cgi?id=51705 Andreas

[Bug bootstrap/51705] [4.7 Regression] FreeBSD uses unsupported C++11 features when __cplusplus == 201103L

2012-01-09 Thread bkorb at gnu dot org
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++ fully conforms

[Bug bootstrap/51705] [4.7 Regression] FreeBSD uses unsupported C++11 features when __cplusplus == 201103L

2012-01-06 Thread bkorb at gnu dot org
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'll add

[Bug preprocessor/51776] New: fixincludes hacks around a C++ deficiency

2012-01-06 Thread bkorb at gnu dot org
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

[Bug bootstrap/51705] [4.7 Regression] FreeBSD uses unsupported C++11 features when __cplusplus == 201103L

2012-01-06 Thread bkorb at gnu dot org
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

[Bug bootstrap/51705] [4.7 Regression] FreeBSD uses unsupported C++11 features when __cplusplus == 201103L

2012-01-06 Thread bkorb at gnu dot org
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 follows. Here

[Bug c/25509] can't disable __attribute__((warn_unused_result))

2010-08-30 Thread bkorb at gnu dot org
--- 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 After having waded through

[Bug target/16300] Bug in vendor /usr/include/net/if.h needs fixincluding

2009-02-28 Thread bkorb at gnu dot org
--- 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 |Added

[Bug bootstrap/29311] toplevel configure script does not test for GNU Make

2008-12-28 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/29311] toplevel configure script does not test for GNU Make

2008-12-28 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/33200] install fails when trying to install fix-header since fix-header wasn't built

2008-01-19 Thread bkorb at gnu dot org
--- 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-header also starts

[Bug other/32771] Fixincludes should fix realloc in stdlib.h in glibc

2007-07-15 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/30678] [4.3 regression] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-08 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/30678] [4.3 regression] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-07 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/30678] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-03 Thread bkorb at gnu dot org
--- 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_fix = format; c_fix_arg = \n

[Bug bootstrap/30678] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-03 Thread bkorb at gnu dot org
--- 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 purpose

[Bug bootstrap/30678] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-02 Thread bkorb at gnu dot org
--- 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

[Bug bootstrap/30678] sysmacros.h get currupt from Fixincludes with updated glibc.

2007-02-02 Thread bkorb at gnu dot org
--- 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

[Bug driver/29311] New: configure script does not test for GNU Make

2006-10-01 Thread bkorb at gnu dot org
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: *-*-solaris* http://gcc.gnu.org/bugzilla/show_bug.cgi

[Bug testsuite/23400] [4.1/4.2 Regression] make check fixinclude failure

2005-11-24 Thread bkorb at gnu dot org
--- 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 patch