changeset 22492:f22fd96605ee

renamed get_jvmci_jdk to get_jvmci_jdk_dir and where relevant, variables named jdk to jdkDir
author Doug Simon <doug.simon@oracle.com>
date Sun, 30 Aug 2015 12:29:24 +0200
parents 917b3ad10c32
children 63b12d69147d
files mx.jvmci/mx_jvmci.py
diffstat 1 files changed, 37 insertions(+), 37 deletions(-) [+]
line wrap: on
line diff
--- a/mx.jvmci/mx_jvmci.py	Sun Aug 30 12:28:37 2015 +0200
+++ b/mx.jvmci/mx_jvmci.py	Sun Aug 30 12:29:24 2015 +0200
@@ -334,8 +334,8 @@
 
     # archive different build types of hotspot
     for vmBuild in _vmbuildChoices:
-        jdkpath = join(_jdksDir(), vmBuild)
-        if not exists(jdkpath):
+        jdkDir = join(_jdksDir(), vmBuild)
+        if not exists(jdkDir):
             mx.logv("skipping " + vmBuild)
             continue
 
@@ -343,7 +343,7 @@
         mx.logv("creating basejdk " + tarName)
         vmSet = set()
         with tarfile.open(tarName, 'w:gz') as tar:
-            for root, _, files in os.walk(jdkpath):
+            for root, _, files in os.walk(jdkDir):
                 if basename(root) in _vmChoices.keys():
                     # TODO: add some assert to check path assumption
                     vmSet.add(root)
@@ -411,32 +411,32 @@
         return join('jre', 'bin')
     return join('jre', 'lib', mx.get_arch())
 
-def vmLibDirInJdk(jdk):
+def vmLibDirInJdk(jdkDir):
     """
     Gets the directory within a JDK where the server and client
     sub-directories are located.
     """
-    return join(jdk, relativeVmLibDirInJdk())
+    return join(jdkDir, relativeVmLibDirInJdk())
 
-def getVmJliLibDirs(jdk):
+def getVmJliLibDirs(jdkDir):
     """
     Get the directories within a JDK where the jli library designates to.
     """
     mxos = mx.get_os()
     if mxos == 'darwin':
-        return [join(jdk, 'jre', 'lib', 'jli')]
+        return [join(jdkDir, 'jre', 'lib', 'jli')]
     if mxos == 'windows' or mxos == 'cygwin':
-        return [join(jdk, 'jre', 'bin'), join(jdk, 'bin')]
-    return [join(jdk, 'jre', 'lib', mx.get_arch(), 'jli'), join(jdk, 'lib', mx.get_arch(), 'jli')]
+        return [join(jdkDir, 'jre', 'bin'), join(jdkDir, 'bin')]
+    return [join(jdkDir, 'jre', 'lib', mx.get_arch(), 'jli'), join(jdkDir, 'lib', mx.get_arch(), 'jli')]
 
-def getVmCfgInJdk(jdk, jvmCfgFile='jvm.cfg'):
+def getVmCfgInJdk(jdkDir, jvmCfgFile='jvm.cfg'):
     """
     Get the jvm.cfg file.
     """
     mxos = mx.get_os()
     if mxos == "windows" or mxos == "cygwin":
-        return join(jdk, 'jre', 'lib', mx.get_arch(), jvmCfgFile)
-    return join(vmLibDirInJdk(jdk), jvmCfgFile)
+        return join(jdkDir, 'jre', 'lib', mx.get_arch(), jvmCfgFile)
+    return join(vmLibDirInJdk(jdkDir), jvmCfgFile)
 
 def _jdksDir():
     return os.path.abspath(join(_installed_jdks if _installed_jdks else _suite.dir, 'jdk' + str(mx.get_jdk().version)))
@@ -452,7 +452,7 @@
             return
     mx.abort('You need to run "mx --vm ' + vm + ' --vmbuild ' + bld + ' build" to build the selected VM')
 
-def get_jvmci_jdk(build=None, vmToCheck=None, create=False, deployDists=True):
+def get_jvmci_jdk_dir(build=None, vmToCheck=None, create=False, deployDists=True):
     """
     Gets the path of the JVMCI JDK corresponding to 'build' (or '_vmbuild'), creating it
     first if it does not exist and 'create' is True. If the JDK was created or
@@ -461,16 +461,16 @@
     """
     if not build:
         build = _vmbuild
-    jdk = join(_jdksDir(), build)
+    jdkDir = join(_jdksDir(), build)
     if create:
         srcJdk = mx.get_jdk().home
-        if not exists(jdk):
-            mx.log('Creating ' + jdk + ' from ' + srcJdk)
-            shutil.copytree(srcJdk, jdk)
+        if not exists(jdkDir):
+            mx.log('Creating ' + jdkDir + ' from ' + srcJdk)
+            shutil.copytree(srcJdk, jdkDir)
 
             # Make a copy of the default VM so that this JDK can be
             # reliably used as the bootstrap for a HotSpot build.
-            jvmCfg = getVmCfgInJdk(jdk)
+            jvmCfg = getVmCfgInJdk(jdkDir)
             if not exists(jvmCfg):
                 mx.abort(jvmCfg + ' does not exist')
 
@@ -492,8 +492,8 @@
                         jvmCfgLines += [line]
 
             assert defaultVM is not None, 'Could not find default VM in ' + jvmCfg
-            chmodRecursive(jdk, JDK_UNIX_PERMISSIONS_DIR)
-            shutil.move(join(vmLibDirInJdk(jdk), defaultVM), join(vmLibDirInJdk(jdk), 'original'))
+            chmodRecursive(jdkDir, JDK_UNIX_PERMISSIONS_DIR)
+            shutil.move(join(vmLibDirInJdk(jdkDir), defaultVM), join(vmLibDirInJdk(jdkDir), 'original'))
 
             if mx.get_os() != 'windows':
                 os.chmod(jvmCfg, JDK_UNIX_PERMISSIONS_FILE)
@@ -502,7 +502,7 @@
                     fp.write(line)
 
             # patch 'release' file (append jvmci revision)
-            releaseFile = join(jdk, 'release')
+            releaseFile = join(jdkDir, 'release')
             if exists(releaseFile):
                 releaseFileLines = []
                 with open(releaseFile) as f:
@@ -527,13 +527,13 @@
 
             # Install a copy of the disassembler library
             try:
-                hsdis([], copyToDir=vmLibDirInJdk(jdk))
+                hsdis([], copyToDir=vmLibDirInJdk(jdkDir))
             except SystemExit:
                 pass
     else:
-        if not exists(jdk):
+        if not exists(jdkDir):
             if _installed_jdks:
-                mx.log("The selected JDK directory does not (yet) exist: " + jdk)
+                mx.log("The selected JDK directory does not (yet) exist: " + jdkDir)
             _handle_missing_VM(build, vmToCheck)
 
     if deployDists:
@@ -543,7 +543,7 @@
                 _installDistInJdks(jdkDist)
 
     if vmToCheck is not None:
-        jvmCfg = getVmCfgInJdk(jdk)
+        jvmCfg = getVmCfgInJdk(jdkDir)
         found = False
         with open(jvmCfg) as f:
             for line in f:
@@ -553,11 +553,11 @@
         if not found:
             _handle_missing_VM(build, vmToCheck)
 
-    return jdk
+    return jdkDir
 
-def _updateInstalledJVMCIOptionsFile(jdk):
+def _updateInstalledJVMCIOptionsFile(jdkDir):
     jvmciOptions = join(_suite.dir, 'jvmci.options')
-    jreLibDir = join(jdk, 'jre', 'lib')
+    jreLibDir = join(jdkDir, 'jre', 'lib')
     if exists(jvmciOptions):
         shutil.copy(jvmciOptions, join(jreLibDir, 'jvmci.options'))
     else:
@@ -755,7 +755,7 @@
 
 def jdkhome(vm=None):
     """return the JDK directory selected for the 'vm' command"""
-    return get_jvmci_jdk(deployDists=False)
+    return get_jvmci_jdk_dir(deployDists=False)
 
 def print_jdkhome(args, vm=None):
     """print the JDK directory selected for the 'vm' command"""
@@ -1013,13 +1013,13 @@
     parser.add_argument('-D', action='append', help='set a HotSpot build variable (run \'mx buildvars\' to list variables)', metavar='name=value')
 
     # initialize jdk
-    get_jvmci_jdk(create=True)
+    get_jvmci_jdk_dir(create=True)
 
     mx.build(['--source', '1.7'] + args, parser=parser)
 
 
-def updateJvmCfg(jdk, vm):
-    jvmCfg = getVmCfgInJdk(jdk)
+def updateJvmCfg(jdkDir, vm):
+    jvmCfg = getVmCfgInJdk(jdkDir)
     if not exists(jvmCfg):
         mx.abort(jvmCfg + ' does not exist')
 
@@ -1072,15 +1072,15 @@
         mx.abort("conflicting working directories: do not set --vmcwd for this command")
 
     build = vmbuild if vmbuild else _vmbuild
-    jdk = get_jvmci_jdk(build, vmToCheck=vm, deployDists=False)
-    _updateInstalledJVMCIOptionsFile(jdk)
+    jdkDir = get_jvmci_jdk_dir(build, vmToCheck=vm, deployDists=False)
+    _updateInstalledJVMCIOptionsFile(jdkDir)
     mx.expand_project_in_args(args)
     if _make_eclipse_launch:
         mx.make_eclipse_launch(_suite, args, _suite.name + '-' + build, name=None, deps=mx.dependencies())
     jacocoArgs = mx_gate.get_jacoco_agent_args()
     if jacocoArgs:
         args = jacocoArgs + args
-    exe = join(jdk, 'bin', mx.exe_suffix('java'))
+    exe = join(jdkDir, 'bin', mx.exe_suffix('java'))
     pfx = _vm_prefix.split() if _vm_prefix is not None else []
 
     # Support for -G: options
@@ -1383,7 +1383,7 @@
                                mainClass=None, excludedLibs=[], distDependencies=[], javaCompliance=None, platformDependent=False, theLicense=None)
         d.make_archive()
         env = os.environ.copy()
-        env['JAVA_HOME'] = get_jvmci_jdk(vmToCheck='server')
+        env['JAVA_HOME'] = get_jvmci_jdk_dir(vmToCheck='server')
         env['MAVEN_OPTS'] = '-server -XX:-UseJVMCIClassLoader'
         cmd = ['mvn', 'install:install-file', '-DgroupId=' + groupId, '-DartifactId=' + artifactId,
                '-Dversion=1.0-SNAPSHOT', '-Dpackaging=jar', '-Dfile=' + d.path]
@@ -1440,7 +1440,7 @@
             else:
                 buildOutput.append(x)
         env = os.environ.copy()
-        env['JAVA_HOME'] = get_jvmci_jdk(vmToCheck='server')
+        env['JAVA_HOME'] = get_jvmci_jdk_dir(vmToCheck='server')
         env['MAVEN_OPTS'] = '-server -XX:-UseJVMCIClassLoader'
         mx.log("Building benchmarks...")
         cmd = ['mvn']