On 2/11/2012 2:20 AM, Daniel D. Daugherty wrote:
On 10/31/12 11:41 PM, David Holmes wrote:
No takers so far - don't be shy, it's not a difficult one I promise :)
Updated webrev: http://cr.openjdk.java.net/~dholmes/8002034/webrev.01/
make/linux/makefiles/defs.make
The new BUILD_FLAVOR check on line 176 is also applicable to Solaris
so you should make the same change there. We don't want things to
diverge unless absolutely necessary.
Whoa! Thanks Dan! That change also applies to the Windows version. And I
was also missing the empty OBJCOPY change in the Solaris version!
Updated webrev:
http://cr.openjdk.java.net/~dholmes/8002034/webrev.02/
David
-----
make/linux/makefiles/vm.make
No comments.
Dan
Because of the multiple sub-make invocations that occur during a build
the FDS logic gets processed numerous times, many of which are not
needed and which might show confusing output (eg that FDS is enabled
when you have disabled it). So I added another conditional to at least
check that BUILD_FLAVOR has been set - that excludes a couple of the
unnecessary executions.
Thanks,
David
On 31/10/2012 5:29 PM, David Holmes wrote:
http://cr.openjdk.java.net/~dholmes/8002034/webrev/
This mainly addresses
JDK-8002034 Allow Full Debug Symbols when cross-compiling
The initial FDS work simply disables FDS when cross-compilation is
involved. But we're now ready to deal with the cross-compilation case
(and even if we weren't these changes would be fine anyway). So if FDS
is initially enabled we will rely on ALT_OBJCOPY being set to the
correct value for cross-compilation; and we don't set the default
OBJCOPY in that case. So if you do nothing when cross-compiling OBJCOPY
will be empty and that will cause the build to disable FDS, just as if
the OBJCOPY command was not found.
As this is the same code that causes:
JDK-8001756 Hotspot makefiles report missing OBJCOPY command in the
wrong circumstances
I also fixed this by moving the OBJCOPY checks inside the
ENABLE_FULL_DEBUG_SYMBOLS=1 conditional. Basically unless we are asking
for FDS we shouldn't complain if the OBJCOPY command is not found.
This will be pushed to the hotspot-emb repo as it is needed there ASAP.
I'll be doing a similar change, under 8002040, for the JDK side of
things.
Thanks,
David
