New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Configure script doesn't recognize GCC 6 or 7 #16089
Comments
From kbrown@cornell.eduWhen testing whether to use -D_FORTIFY_SOURCE, the Configure script only Ken Flags: Site configuration information for perl 5.26.0: Configured by kbrown at Wed Jul 19 09:30:20 EDT 2017. Summary of my perl5 (revision 5 version 26 subversion 0) configuration: Platform: @INC for perl 5.26.0: Environment for perl 5.26.0: PATH=/home/kbrown/bin:/usr/local/bin:/usr/bin:/c/ProgramData/Oracle/Java/javapath:/c/Program |
From kbrown@cornell.edu0001-Configure-check-for-GCC-6-and-7.patchFrom 49534dc34fa62de65a1adc6ef89ba7e5cc323052 Mon Sep 17 00:00:00 2001
From: Ken Brown <kbrown@cornell.edu>
Date: Thu, 27 Jul 2017 11:57:44 -0400
Subject: [PATCH] Configure: check for GCC 6 and 7
---
Configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Configure b/Configure
index 3259249500..bdcfaf17ef 100755
--- a/Configure
+++ b/Configure
@@ -23612,7 +23612,7 @@ fi
: add -D_FORTIFY_SOURCE if feasible and not already there
case "$gccversion" in
-[45].*) case "$optimize$ccflags" in
+[4567].*) case "$optimize$ccflags" in
*-O*) case "$ccflags$cppsymbols" in
*_FORTIFY_SOURCE=*) # Don't add it again.
echo "You seem to have -D_FORTIFY_SOURCE already, not adding it." >&4
--
2.13.2
|
From @tonycozOn Thu, 27 Jul 2017 09:24:44 -0700, kbrown@cornell.edu wrote:
Thanks, applied as b948647. Tony |
The RT System itself - Status changed from 'new' to 'open' |
@tonycoz - Status changed from 'open' to 'pending release' |
From @khwilliamsonThank you for filing this report. You have helped make Perl better. With the release yesterday of Perl 5.28.0, this and 185 other issues have been Perl 5.28.0 may be downloaded via: If you find that the problem persists, feel free to reopen this ticket. |
@khwilliamson - Status changed from 'pending release' to 'resolved' |
Migrated from rt.perl.org#131809 (status was 'resolved')
Searchable as RT131809$
The text was updated successfully, but these errors were encountered: