Thanks Mike. I'd been struggling to work out why this was failing (as I
didn't want to pull in your back-out changeset. I was left puzzling why
this is even checking for something gcc related on Solaris when I'm not
even building with gcc ???
In any case this looks ok now.
David
Mike Duigou
On Dec 2 2010, at 14:41 , Kelly O'Hair wrote:
> I think the rules are broken.
>
> The ifdef and ifndef and ifeq and ifneq statements are Make-isms and should
> not use TAB characters.
> But the shell logic of the rule needs to have the line start with a TAB, so
> the "@if [..." line needs to s
I think the rules are broken.
The ifdef and ifndef and ifeq and ifneq statements are Make-isms and
should not use TAB characters.
But the shell logic of the rule needs to have the line start with a
TAB, so the "@if [..." line needs to start
with a TAB.
It is hard to see in a webrev, but I t
The previous patch (CR 6998016) which attempted to fix the incorrect ifdef
nesting in the Sanity.gmk sane-gcc-compiler rule ran into a problem on 32 bit
versions of solaris due to a previously unreported error in the use of
REQUIRED_GCC_VERSION (which should have been REQUIRED_GCC_VER). The fixe
On Dec 1, 2010, at 7:20 PM, David Holmes wrote:
Kelly O'Hair said the following on 12/02/10 07:26:
On Dec 1, 2010, at 12:56 PM, Ulf Zibis wrote:
But yes, those scenarios could be improved, but IMHO smarter with
something like "java -version:java", (or interpreting the existing
"java -versi
Am 02.12.2010 04:20, schrieb David Holmes:
Exactly. The whole Eclipse problem that motivated this stemmed from the fact that they wanted a
way to identify the JVM, given a "java" command, without having to launch the VM and parse some
version output. This is what this simple file aims to do.
E
Am 01.12.2010 22:26, schrieb Kelly O'Hair:
That's what using java -version does, you have launched the VM twice. It's the
same thing.
You might be avoiding some class file loads with java -version, and it is
probably quicker, but the
native VM library is loaded and run when you do a 'java -ver
