* [PATCH] Disable libsanitizer if C++ is not being built
@ 2012-11-15 14:23 Siddhesh Poyarekar
2012-11-15 14:49 ` Steven Bosscher
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Siddhesh Poyarekar @ 2012-11-15 14:23 UTC (permalink / raw)
To: gcc-patches
[-- Attachment #1: Type: text/plain, Size: 293 bytes --]
Hi,
Current HEAD fails build when --enable-languages=c, i.e. C++ is not
being built. Attached patch fixes this.
Regards,
Siddhesh
ChangeLog:
2012-11-15 Siddhesh Poyarekar <siddhesh@redhat.com>
* configure.ac: Disable libsanitizer if we're not building C++.
* configure: Regenerate.
[-- Attachment #2: w.patch --]
[-- Type: text/x-patch, Size: 960 bytes --]
diff --git a/configure b/configure
index d72d4f9..16bbaa8 100755
--- a/configure
+++ b/configure
@@ -6437,11 +6437,11 @@ case ,${enable_languages},:${enable_objc_gc} in
;;
esac
-# Disable libitm if we're not building C++
+# Disable libitm and libsanitizer if we're not building C++
case ,${enable_languages}, in
*,c++,*) ;;
*)
- noconfigdirs="$noconfigdirs target-libitm"
+ noconfigdirs="$noconfigdirs target-libitm target-libsanitizer"
;;
esac
diff --git a/configure.ac b/configure.ac
index 24ea7e5..cd6721e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2008,11 +2008,11 @@ case ,${enable_languages},:${enable_objc_gc} in
;;
esac
-# Disable libitm if we're not building C++
+# Disable libitm and libsanitizer if we're not building C++
case ,${enable_languages}, in
*,c++,*) ;;
*)
- noconfigdirs="$noconfigdirs target-libitm"
+ noconfigdirs="$noconfigdirs target-libitm target-libsanitizer"
;;
esac
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-15 14:23 [PATCH] Disable libsanitizer if C++ is not being built Siddhesh Poyarekar
@ 2012-11-15 14:49 ` Steven Bosscher
2012-11-15 14:59 ` Michael Matz
2012-11-21 7:14 ` Ping: " Siddhesh Poyarekar
2012-11-27 10:05 ` Jakub Jelinek
2 siblings, 1 reply; 7+ messages in thread
From: Steven Bosscher @ 2012-11-15 14:49 UTC (permalink / raw)
To: Siddhesh Poyarekar; +Cc: gcc-patches
> Current HEAD fails build when --enable-languages=c, i.e. C++ is not
> being built. Attached patch fixes this.
Eh???
Isn't C++ always built, because gcc itself requires it?
Ciao!
Steven
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-15 14:49 ` Steven Bosscher
@ 2012-11-15 14:59 ` Michael Matz
0 siblings, 0 replies; 7+ messages in thread
From: Michael Matz @ 2012-11-15 14:59 UTC (permalink / raw)
To: Steven Bosscher; +Cc: Siddhesh Poyarekar, gcc-patches
Hi,
On Thu, 15 Nov 2012, Steven Bosscher wrote:
> > Current HEAD fails build when --enable-languages=c, i.e. C++ is not
> > being built. Attached patch fixes this.
>
> Eh???
Uhh???
> Isn't C++ always built, because gcc itself requires it?
Without bootstrapping it doesn't, no.
Ciao,
Michael.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Ping: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-15 14:23 [PATCH] Disable libsanitizer if C++ is not being built Siddhesh Poyarekar
2012-11-15 14:49 ` Steven Bosscher
@ 2012-11-21 7:14 ` Siddhesh Poyarekar
2012-11-27 10:03 ` Ping[2]: " Siddhesh Poyarekar
2012-11-27 10:05 ` Jakub Jelinek
2 siblings, 1 reply; 7+ messages in thread
From: Siddhesh Poyarekar @ 2012-11-21 7:14 UTC (permalink / raw)
To: gcc-patches
Hi,
Ping!
Siddhesh
On Thu, 15 Nov 2012 19:52:21 +0530, Siddhesh wrote:
> Hi,
>
> Current HEAD fails build when --enable-languages=c, i.e. C++ is not
> being built. Attached patch fixes this.
>
> Regards,
> Siddhesh
>
> ChangeLog:
>
> 2012-11-15 Siddhesh Poyarekar <siddhesh@redhat.com>
>
> * configure.ac: Disable libsanitizer if we're not building
> C++.
> * configure: Regenerate.
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Ping[2]: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-21 7:14 ` Ping: " Siddhesh Poyarekar
@ 2012-11-27 10:03 ` Siddhesh Poyarekar
0 siblings, 0 replies; 7+ messages in thread
From: Siddhesh Poyarekar @ 2012-11-27 10:03 UTC (permalink / raw)
To: gcc-patches
Ping!
Siddhesh
On Wed, Nov 21, 2012 at 12:43:10PM +0530, Siddhesh Poyarekar wrote:
> Hi,
>
> Ping!
>
> Siddhesh
>
> On Thu, 15 Nov 2012 19:52:21 +0530, Siddhesh wrote:
>
> > Hi,
> >
> > Current HEAD fails build when --enable-languages=c, i.e. C++ is not
> > being built. Attached patch fixes this.
> >
> > Regards,
> > Siddhesh
> >
> > ChangeLog:
> >
> > 2012-11-15 Siddhesh Poyarekar <siddhesh@redhat.com>
> >
> > * configure.ac: Disable libsanitizer if we're not building
> > C++.
> > * configure: Regenerate.
> >
>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-15 14:23 [PATCH] Disable libsanitizer if C++ is not being built Siddhesh Poyarekar
2012-11-15 14:49 ` Steven Bosscher
2012-11-21 7:14 ` Ping: " Siddhesh Poyarekar
@ 2012-11-27 10:05 ` Jakub Jelinek
2012-11-27 10:31 ` Siddhesh Poyarekar
2 siblings, 1 reply; 7+ messages in thread
From: Jakub Jelinek @ 2012-11-27 10:05 UTC (permalink / raw)
To: Siddhesh Poyarekar; +Cc: gcc-patches
On Thu, Nov 15, 2012 at 07:52:21PM +0530, Siddhesh Poyarekar wrote:
> 2012-11-15 Siddhesh Poyarekar <siddhesh@redhat.com>
>
> * configure.ac: Disable libsanitizer if we're not building C++.
> * configure: Regenerate.
Ok, thanks.
Jakub
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] Disable libsanitizer if C++ is not being built
2012-11-27 10:05 ` Jakub Jelinek
@ 2012-11-27 10:31 ` Siddhesh Poyarekar
0 siblings, 0 replies; 7+ messages in thread
From: Siddhesh Poyarekar @ 2012-11-27 10:31 UTC (permalink / raw)
To: Jakub Jelinek; +Cc: gcc-patches
On Tue, Nov 27, 2012 at 11:05:47AM +0100, Jakub Jelinek wrote:
> On Thu, Nov 15, 2012 at 07:52:21PM +0530, Siddhesh Poyarekar wrote:
> > 2012-11-15 Siddhesh Poyarekar <siddhesh@redhat.com>
> >
> > * configure.ac: Disable libsanitizer if we're not building C++.
> > * configure: Regenerate.
>
> Ok, thanks.
>
Could you please commit it for me? I don't have write access.
Thanks,
Siddhesh
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2012-11-27 10:31 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-11-15 14:23 [PATCH] Disable libsanitizer if C++ is not being built Siddhesh Poyarekar
2012-11-15 14:49 ` Steven Bosscher
2012-11-15 14:59 ` Michael Matz
2012-11-21 7:14 ` Ping: " Siddhesh Poyarekar
2012-11-27 10:03 ` Ping[2]: " Siddhesh Poyarekar
2012-11-27 10:05 ` Jakub Jelinek
2012-11-27 10:31 ` Siddhesh Poyarekar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).