You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by sa...@apache.org on 2012/09/24 17:57:15 UTC

svn commit: r1389448 - /lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py

Author: sarowe
Date: Mon Sep 24 15:57:14 2012
New Revision: 1389448

URL: http://svn.apache.org/viewvc?rev=1389448&view=rev
Log:
Make smoke tester work on cygwin when JAVAX_HOME environment variable values are Windows style (X:/blah or X:\blah...)

Modified:
    lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py

Modified: lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py
URL: http://svn.apache.org/viewvc/lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py?rev=1389448&r1=1389447&r2=1389448&view=diff
==============================================================================
--- lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py (original)
+++ lucene/dev/branches/lucene_solr_4_0/dev-tools/scripts/smokeTestRelease.py Mon Sep 24 15:57:14 2012
@@ -38,6 +38,9 @@ import checkJavadocLinks
 # must have a working gpg, tar, unzip in your path.  This has been
 # tested on Linux and on Cygwin under Windows 7.
 
+cygwin = platform.system().lower().startswith('cygwin')
+cygwinWindowsRoot = os.popen('cygpath -w /').read().strip().replace('\\','/') if cygwin else ''
+
 def unshortenURL(url):
   parsed = urllib.parse.urlparse(url)
   if parsed[0] in ('http', 'https'):
@@ -55,6 +58,8 @@ def javaExe(version):
     path = JAVA7_HOME
   else:
     raise RuntimeError("unknown Java version '%s'" % version)
+  if cygwin:
+    path = os.popen('cygpath -u "%s"' % path).read().strip()
   return 'export JAVA_HOME="%s" PATH="%s/bin:$PATH"' % (path, path)
 
 def verifyJavaVersion(version):
@@ -77,9 +82,6 @@ except KeyError:
 verifyJavaVersion('1.6')
 verifyJavaVersion('1.7')
 
-cygwin = platform.system().lower().startswith('cygwin')
-cygwinWindowsRoot = os.popen('cygpath -w /').read().strip().replace('\\','/') if cygwin else ''
-
 # TODO
 #   + verify KEYS contains key that signed the release
 #   + make sure changes HTML looks ok