Thank you for the feedback.  See below:

On 2016-10-15 03:23, Tim Bell wrote:
Please review this rework of the langtools/test/Makefile fix done in
8166648. [1]  ARCH_DATA_MODEL is not set in all test scenarios, so a
better approach is to check the VM about to be tested.

Bug report:


   Successful JPRT runs on all platforms.  Inspected the
<platform>_<arch>.product-c2-langtools_jtreg.log files to verify the
expected flag was used only on 32-bit VMs.

Hi Tim,

Seems a bit complicated to create a temporary javascript and execute it.
Why not try something like:
DATA_MODEL=`${JT_JAVA}/bin/java -XshowSettings:properties -version 2>&1
| grep | awk '{print $3}'`

Excellent idea.  This is much simpler.

Here is an updated webrev. Tested as before, by submitting a JPRT job and inspecting the .log files.


Thanks in advance-


[1] JDK-8166648

Reply via email to