{"action":"created","issue":{"url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/12","repository_url":"https://api.github.com/repos/AdoptOpenJDK/stf","labels_url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/12/labels{/name}","comments_url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/12/comments","events_url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/12/events","html_url":"https://github.com/AdoptOpenJDK/stf/issues/12","id":268869827,"number":12,"title":"Ant environment variable \"ant.java.version\" returns 1.8 for Openj9 SDK","user":{"login":"Mesbah-Alam","id":5099003,"avatar_url":"https://avatars1.githubusercontent.com/u/5099003?v=4","gravatar_id":"","url":"https://api.github.com/users/Mesbah-Alam","html_url":"https://github.com/Mesbah-Alam","followers_url":"https://api.github.com/users/Mesbah-Alam/followers","following_url":"https://api.github.com/users/Mesbah-Alam/following{/other_user}","gists_url":"https://api.github.com/users/Mesbah-Alam/gists{/gist_id}","starred_url":"https://api.github.com/users/Mesbah-Alam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mesbah-Alam/subscriptions","organizations_url":"https://api.github.com/users/Mesbah-Alam/orgs","repos_url":"https://api.github.com/users/Mesbah-Alam/repos","events_url":"https://api.github.com/users/Mesbah-Alam/events{/privacy}","received_events_url":"https://api.github.com/users/Mesbah-Alam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"Mesbah-Alam","id":5099003,"avatar_url":"https://avatars1.githubusercontent.com/u/5099003?v=4","gravatar_id":"","url":"https://api.github.com/users/Mesbah-Alam","html_url":"https://github.com/Mesbah-Alam","followers_url":"https://api.github.com/users/Mesbah-Alam/followers","following_url":"https://api.github.com/users/Mesbah-Alam/following{/other_user}","gists_url":"https://api.github.com/users/Mesbah-Alam/gists{/gist_id}","starred_url":"https://api.github.com/users/Mesbah-Alam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mesbah-Alam/subscriptions","organizations_url":"https://api.github.com/users/Mesbah-Alam/orgs","repos_url":"https://api.github.com/users/Mesbah-Alam/repos","events_url":"https://api.github.com/users/Mesbah-Alam/events{/privacy}","received_events_url":"https://api.github.com/users/Mesbah-Alam/received_events","type":"User","site_admin":false},"assignees":[{"login":"Mesbah-Alam","id":5099003,"avatar_url":"https://avatars1.githubusercontent.com/u/5099003?v=4","gravatar_id":"","url":"https://api.github.com/users/Mesbah-Alam","html_url":"https://github.com/Mesbah-Alam","followers_url":"https://api.github.com/users/Mesbah-Alam/followers","following_url":"https://api.github.com/users/Mesbah-Alam/following{/other_user}","gists_url":"https://api.github.com/users/Mesbah-Alam/gists{/gist_id}","starred_url":"https://api.github.com/users/Mesbah-Alam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mesbah-Alam/subscriptions","organizations_url":"https://api.github.com/users/Mesbah-Alam/orgs","repos_url":"https://api.github.com/users/Mesbah-Alam/repos","events_url":"https://api.github.com/users/Mesbah-Alam/events{/privacy}","received_events_url":"https://api.github.com/users/Mesbah-Alam/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2017-10-26T18:49:09Z","updated_at":"2017-10-26T18:56:55Z","closed_at":null,"author_association":"NONE","body":"**Overview**\r\nWe have a dependency on Ant environment variable \"ant.java.version\" in our logic in openjdk.build/include/top.xml (target check-for-tools-jar), where we are trying to figure out the SDK we are testing. If it's Java 8, we copy tools.jar in prereqs folder, if Java 9 we do not. \r\n\r\nThis however doesn't work very well, since Ant is returning 1.8 as the value for \"ant.java.version\" even when we are running on Open9. \r\n\r\n**To reproduce**\r\n1) Log into a Linux machine\r\n2) Set JAVA_HOME to OpenJ9 SDK (e,g, http://vmtank.ottawa.ibm.com/vm_archive/R29/xa6490/ibuild_good/xa6490_openj9.zip) \r\n3) Run: ant -diagnostics | grep ant.java.version -- it will print 1.8. \r\n"},"comment":{"url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/comments/339764930","html_url":"https://github.com/AdoptOpenJDK/stf/issues/12#issuecomment-339764930","issue_url":"https://api.github.com/repos/AdoptOpenJDK/stf/issues/12","id":339764930,"user":{"login":"Mesbah-Alam","id":5099003,"avatar_url":"https://avatars1.githubusercontent.com/u/5099003?v=4","gravatar_id":"","url":"https://api.github.com/users/Mesbah-Alam","html_url":"https://github.com/Mesbah-Alam","followers_url":"https://api.github.com/users/Mesbah-Alam/followers","following_url":"https://api.github.com/users/Mesbah-Alam/following{/other_user}","gists_url":"https://api.github.com/users/Mesbah-Alam/gists{/gist_id}","starred_url":"https://api.github.com/users/Mesbah-Alam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Mesbah-Alam/subscriptions","organizations_url":"https://api.github.com/users/Mesbah-Alam/orgs","repos_url":"https://api.github.com/users/Mesbah-Alam/repos","events_url":"https://api.github.com/users/Mesbah-Alam/events{/privacy}","received_events_url":"https://api.github.com/users/Mesbah-Alam/received_events","type":"User","site_admin":false},"created_at":"2017-10-26T18:56:55Z","updated_at":"2017-10-26T18:56:55Z","author_association":"NONE","body":"\r\n**Note**\r\nInterestingly, if you do \"ant -diagnostics\" to print the full diagnostics of Ant while you have OpenJ9 SDK on the path, you get, among others, these value: \r\nant.java.version : 1.8\r\njava.vm.version : 2.9\r\njava.vm.vendor : Eclipse OpenJ9\r\njava.runtime.version : 9-internal+0-adhoc.openj9.openjdk\r\njava.fullversion : 9-internal+0-adhoc.openj9.openjdk\r\nJRE 9 IBM J9 2.9 Linux amd64-64 Compressed References 20171025_368423 (JIT enabled, AOT enabled)\r\nOpenJ9   - b13c187\r\nOMR      - 604be15\r\njava.awt.graphicsenv : sun.awt.X11GraphicsEnvironment\r\nos.arch : amd64\r\ncom.ibm.vm.bitmode : 64\r\njava.vm.specification.version : 9\r\njava.version : 9-internal\r\n\r\nSo, as a workaround, in our top.xml, we could depend on \"java.version\" instead of \"ant.java.version\"\r\n\r\n@lumpfish - any suggestion? "}}
{"actor":{"display_login":"Mesbah-Alam"}}