Use new pointerlist and envmapHEADmaster

2 files changed, 7 insertions, 30 deletions

diff --git a/src/bootstrap.cc b/src/bootstrap.cc
index 42706c1..be1b5ed 100644
--- a/src/bootstrap.cc
+++ b/src/bootstrap.cc
@@ -19,6 +19,7 @@
 #include "tasks.cc"
 #include "build.cc"
 #include "tools.cc"
+#include "pointerlist.cc"
 
 const std::filesystem::path configurationFile("configuration.cc");
 const std::filesystem::path configHeaderFile("config.h");
diff --git a/src/execute.cc b/src/execute.cc
index a905efc..ad6c2a2 100644
--- a/src/execute.cc
+++ b/src/execute.cc
@@ -4,6 +4,7 @@
 #include "execute.h"
 
 #include "ctor.h"
+#include "pointerlist.h"
 
 #include <unistd.h>
 #include <cstring>
@@ -22,27 +23,6 @@ https://stackoverflow.com/questions/4259629/what-is-the-difference-between-fork-
 
 namespace
 {
-class Env
-	: public std::vector<char*>
-{
-public:
-	Env(const std::vector<std::string>& args)
-	{
-		for(const auto& arg : args)
-		{
-			push_back(strdup(arg.data()));
-		}
-		push_back(nullptr);
-	}
-
-	~Env()
-	{
-		for(auto ptr : *this)
-		{
-			free(ptr);
-		}
-	}
-};
 
 int parent_waitpid(pid_t pid)
 {
@@ -122,19 +102,15 @@ int execute(const ctor::settings& settings,
 	auto pid = vfork();
 	if(pid == 0)
 	{
-		std::vector<std::string> venv;
+		EnvMap envmap((const char**)environ);
 		for(const auto& [key, value] : env)
 		{
-			venv.push_back(key + "=" + value);
-		}
-
-		for(auto current = environ; *current; ++current)
-		{
-			venv.emplace_back(*current);
+			envmap.insert(key + "=" + value);
 		}
 
-		Env penv(venv);
-		execve(command.data(), (char**)argv.data(), penv.data());
+		auto [_, envv] = envmap.get();
+		execve(command.data(), const_cast<char* const *>(argv.data()),
+		       const_cast<char* const *>(envv));
 		std::cout << "Could not execute " << command << ": " <<
 			strerror(errno) << "\n";
 		_exit(1); // execve only returns if an error occurred