Re: svn commit: r357703 - head/share/mk

2020-02-10 Thread Kyle Evans
On Mon, Feb 10, 2020 at 12:13 PM John Baldwin  wrote:
>
> On 2/9/20 10:53 AM, Kyle Evans wrote:
> > Author: kevans
> > Date: Sun Feb  9 18:53:53 2020
> > New Revision: 357703
> > URL: https://svnweb.freebsd.org/changeset/base/357703
> >
> > Log:
> >   mips: mark GOOGLETEST broken, due to no fault of its own
> >
> >   As explained in the comment; GOOGLETEST cannot currently be compiled on 
> > any
> >   mips variant at the moment due to the cross toolchain seemingly using the
> >   wrong spec and not pulling in libgcc. We'll be fine when llvm 10 lands, at
> >   which point this should be reverted most expeditiously.
>
> Hmm, I thought you were going to do this for any external GCC?  It's also
> broken on amd64, etc. as well when using external GCC.
>

My main concern at the moment is we haven't had any functional default
builds for mips since I had unbroken it over a month ago -- I was
hoping we'd be able to come up with a fix for the external GCC spec
issue in relatively short order so as not to worry about it, but I can
look at expanding this to all external GCC.
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


Re: svn commit: r357703 - head/share/mk

2020-02-10 Thread John Baldwin
On 2/9/20 10:53 AM, Kyle Evans wrote:
> Author: kevans
> Date: Sun Feb  9 18:53:53 2020
> New Revision: 357703
> URL: https://svnweb.freebsd.org/changeset/base/357703
> 
> Log:
>   mips: mark GOOGLETEST broken, due to no fault of its own
>   
>   As explained in the comment; GOOGLETEST cannot currently be compiled on any
>   mips variant at the moment due to the cross toolchain seemingly using the
>   wrong spec and not pulling in libgcc. We'll be fine when llvm 10 lands, at
>   which point this should be reverted most expeditiously.

Hmm, I thought you were going to do this for any external GCC?  It's also
broken on amd64, etc. as well when using external GCC.

-- 
John Baldwin
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"


svn commit: r357703 - head/share/mk

2020-02-09 Thread Kyle Evans
Author: kevans
Date: Sun Feb  9 18:53:53 2020
New Revision: 357703
URL: https://svnweb.freebsd.org/changeset/base/357703

Log:
  mips: mark GOOGLETEST broken, due to no fault of its own
  
  As explained in the comment; GOOGLETEST cannot currently be compiled on any
  mips variant at the moment due to the cross toolchain seemingly using the
  wrong spec and not pulling in libgcc. We'll be fine when llvm 10 lands, at
  which point this should be reverted most expeditiously.

Modified:
  head/share/mk/src.opts.mk

Modified: head/share/mk/src.opts.mk
==
--- head/share/mk/src.opts.mk   Sun Feb  9 15:13:56 2020(r357702)
+++ head/share/mk/src.opts.mk   Sun Feb  9 18:53:53 2020(r357703)
@@ -348,7 +348,12 @@ BROKEN_OPTIONS+=LIB32
 BROKEN_OPTIONS+=LIBSOFT
 .endif
 .if ${__T:Mmips*}
-BROKEN_OPTIONS+=SSP
+# GOOGLETEST cannot currently be compiled on mips due to external 
circumstances.
+# Notably, the freebsd-gcc port isn't linking in libgcc so we end up trying ot
+# link to a hidden symbol. LLVM would successfully link this in, but some of
+# the mips variants are broken under LLVM until LLVM 10. GOOGLETEST should be
+# marked no longer broken with the switch to LLVM.
+BROKEN_OPTIONS+=GOOGLETEST SSP
 .endif
 # EFI doesn't exist on mips, powerpc, sparc or riscv.
 .if ${__T:Mmips*} || ${__T:Mpowerpc*} || ${__T:Msparc64} || ${__T:Mriscv*}
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"