RFR: JDK-8034788 Rewrite toolchain.m4 to support multiple toolchains per platform
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Thu Feb 13 13:39:28 UTC 2014
On 2014-02-13 13:20, Erik Joelsson wrote:
> I like these improvements but I can certainly see the need for more.
>
> Some minor things:
>
> flags.m4:
> 95: needs -> need
> 152: No need to "assume"
>
> toolchain.m4
> 171: detext
> ~230-300: Many rather long lines, consider breaking them up?
> 605: is -> are
> ~650: Why define MACOSX_VERSION_MIN in toolchain.m4 when it's used in
> flags.m4?
As for the last question. I moved this to flags.m4, but I still don't
think it belongs there, so the FIXME to move it elsewhere moved along
with it. :-)
I have addressed the rest of your concerns as well, and those raised by
Henry Jen. I solved the "list" issue by changing the
--with-toolchain-type description to:
--with-toolchain-type the toolchain type (or family) to use, use
'--help'
to show possible values [platform dependent]
And when your type "configure --help", at the end you will get:
The following toolchains are available as arguments to
--with-toolchain-type.
Which are valid to use depends on the build platform.
gcc GNU Compiler Collection
clang clang/LLVM
solstudio Oracle Solaris Studio
xlc IBM XL C/C++
microsoft Microsoft Visual Studio
Unfortunately, we can't get the current platform at that point, so I'll
have to list all. But I think that's okay. My main point was that you
must be able to type "configure --help" and see what kind of options you
can give. If you can't figure out that "microsoft" is invalid on linux,
you'll be told soon enough by configure.
Updated webrev:
http://cr.openjdk.java.net/~ihse/JDK-8034788-rewrite-toolchain-m4/webrev.02
/Magnus
More information about the build-dev
mailing list