On Thu, 13 Jan 2022 14:06:52 GMT, Erik Joelsson <er...@openjdk.org> wrote:

> Looks good.
> 
> Did you consider automatically inferring the VCS by looking for 
> `${this_script_dir}/../../{.git,.hg}`? To resolve `this_script_dir`, you can 
> use the same snippet we have in e.g. top level configure:
> 
> ```
> # Get an absolute path to this script, since that determines the top-level 
> directory.
> this_script_dir=`dirname $0`
> this_script_dir=`cd $this_script_dir > /dev/null && pwd`
> ```

Thank you for the review, and for the suggestion! I've implemented the VCS 
auto-detection logic now, could you please take another look?

-------------

PR: https://git.openjdk.java.net/jdk/pull/7027

Reply via email to