You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mesos.apache.org by an...@apache.org on 2018/02/06 22:49:39 UTC

[2/2] mesos git commit: Removed workaround in ZooKeeper test.

Removed workaround in ZooKeeper test.

Now that `fs::list` returns the full path, this code can be fixed.

Review: https://reviews.apache.org/r/65398/


Project: http://git-wip-us.apache.org/repos/asf/mesos/repo
Commit: http://git-wip-us.apache.org/repos/asf/mesos/commit/8e7afafb
Tree: http://git-wip-us.apache.org/repos/asf/mesos/tree/8e7afafb
Diff: http://git-wip-us.apache.org/repos/asf/mesos/diff/8e7afafb

Branch: refs/heads/master
Commit: 8e7afafb9056c04d2ff849011a9e8f79a2f9ce6a
Parents: e71ed38
Author: Andrew Schwartzmeyer <an...@schwartzmeyer.com>
Authored: Tue Feb 6 14:48:33 2018 -0800
Committer: Andrew Schwartzmeyer <an...@schwartzmeyer.com>
Committed: Tue Feb 6 14:48:33 2018 -0800

----------------------------------------------------------------------
 src/tests/zookeeper.cpp | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/mesos/blob/8e7afafb/src/tests/zookeeper.cpp
----------------------------------------------------------------------
diff --git a/src/tests/zookeeper.cpp b/src/tests/zookeeper.cpp
index 54cf9c6..722ea0b 100644
--- a/src/tests/zookeeper.cpp
+++ b/src/tests/zookeeper.cpp
@@ -72,12 +72,11 @@ void ZooKeeperTest::SetUpTestCase()
 
     foreach (const string& jar, jars.get()) {
 #ifdef __WINDOWS__
-      // TODO(andschwa): `fs::list` on Windows only returns the file names, not
-      // the full paths. Remove this work-around when MESOS-7803 is resolved.
-      classpath += ";" + path::join(zkHome, "lib", jar);
+      classpath += ";";
 #else
-      classpath += ":" + jar;
+      classpath += ":";
 #endif
+      classpath += jar;
     }
 
     LOG(INFO) << "Using Java classpath: " << classpath;