ulidtko commented on pull request #2197:
URL: https://github.com/apache/thrift/pull/2197#issuecomment-675363638


   > JAVA_HOME set in log line 12340 is java-8-oracle
   
   Okay, I understand now. Let me convince you that this is noise and Java8 is 
not actually used. Two reasons:
    * This line 12340 is part of output from `printenv | sort` from 
`build/docker/run.sh`: it shows the `JAVA_HOME` environment variable set 
**outside** the Docker container. It's not existing *inside* the container 
where `./configure` and tests run.
    * If it was — we'd see *Oracle* JDK 8, not OpenJDK, in the output of 
`./configure` (line 13101).
   
   I'm glad to collaborate for the public good and for safe and sustainable 
future of technology. Thanks for your responses, effort and friendliness in 
this space, Zezeng Wang, this is deeply appreciated.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to