Avoid temporary file for listing GPUs.

1 files changed, 17 insertions, 17 deletions

diff --git a/src/lib/grok/util.cc b/src/lib/grok/util.cc
index e9b535df8..8a6d2c4f9 100644
--- a/src/lib/grok/util.cc
+++ b/src/lib/grok/util.cc
@@ -20,6 +20,8 @@
 
 
 #include "util.h"
+#include <boost/process.hpp>
+#include <future>
 
 
 using std::string;
@@ -27,24 +29,22 @@ using std::vector;
 
 
 vector<string>
-get_gpu_names(boost::filesystem::path binary, boost::filesystem::path filename)
+get_gpu_names(boost::filesystem::path binary)
 {
-    // Execute the GPU listing program and redirect its output to a file
-    if (std::system((binary.string() + " > " + filename.string()).c_str()) < 0) {
-	    return {};
-    }
-
-    std::vector<std::string> gpu_names;
-    std::ifstream file(filename.c_str());
-    if (file.is_open())
-    {
-        std::string line;
-        while (std::getline(file, line))
-            gpu_names.push_back(line);
-        file.close();
-    }
-
-    return gpu_names;
+	namespace bp = boost::process;
+
+	bp::ipstream stream;
+	bp::child child(binary, bp::std_out > stream);
+
+	string line;
+	vector<string> gpu_names;
+	while (child.running() && std::getline(stream, line) && !line.empty()) {
+		gpu_names.push_back(line);
+	}
+
+	child.wait();
+
+	return gpu_names;
 }