David, if we use host /usr/bin/objcopy for cross compiling, changes looks good for me.
-Dmitry On 2012-11-01 07:41, 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/ > > 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 -- Dmitry Samersoff Java development team, SPB04 * There will come soft rains ...