fix: remove --exec option

Author: diohabara

CMakeLists.txt

@@ -2,7 +2,7 @@
 cmake_minimum_required(VERSION 3.22)
 project(scc
   LANGUAGES CXX
-  VERSION 0.1.0
+  VERSION 0.2.0
 )
 set(CMAKE_CXX_STANDARD 20)
 set(CMAKE_CXX_STANDARD_REQUIRED ON)

include/scc_app.h

@@ -17,7 +17,7 @@ struct State {
   std::vector<std::string> items;
   int selected = 0;
   bool quit = false;
-  bool do_exec = false;
+  
   bool no_print = false;
   std::optional<std::string> pending_cmd;
 };

src/main.cc

@@ -16,10 +16,7 @@ int main(int argc, char *argv[]) {
   program.add_argument("--config")
       .help("Path to ssh config file")
       .default_value(std::string{});
-  program.add_argument("--exec")
-      .help("Execute ssh after selection")
-      .default_value(false)
-      .implicit_value(true);
+  
   program.add_argument("--no-print")
       .help("Do not print the command to stdout")
       .default_value(false)
@@ -56,7 +53,6 @@ int main(int argc, char *argv[]) {
   st.items = std::move(items);
   st.selected = 0;
   st.quit = false;
-  st.do_exec = program.get<bool>("--exec");
   st.no_print = program.get<bool>("--no-print");
   st.pending_cmd.reset();
 
@@ -70,24 +66,22 @@ int main(int argc, char *argv[]) {
     if (!st.no_print) {
       std::cout << *st.pending_cmd << std::endl;
     }
-    if (st.do_exec) {
-      std::string dest = scc_app::BuildDest(st.connections[st.selected]);
-      std::vector<std::string> args_str = {"ssh", dest};
-      std::vector<char *> args;
-      args.reserve(args_str.size() + 1);
-      for (auto &s : args_str) {
-        args.push_back(s.data());
-      }
-      args.push_back(nullptr);
-      const char *ssh_path = "/usr/bin/ssh";
-      if (::access(ssh_path, X_OK) == 0) {
-        ::execv(ssh_path, args.data());
-      } else {
-        ::execvp("ssh", args.data());
-      }
-      std::perror("exec ssh failed");
-      return 127;
+    std::string dest = scc_app::BuildDest(st.connections[st.selected]);
+    std::vector<std::string> args_str = {"ssh", dest};
+    std::vector<char *> args;
+    args.reserve(args_str.size() + 1);
+    for (auto &s : args_str) {
+      args.push_back(s.data());
+    }
+    args.push_back(nullptr);
+    const char *ssh_path = "/usr/bin/ssh";
+    if (::access(ssh_path, X_OK) == 0) {
+      ::execv(ssh_path, args.data());
+    } else {
+      ::execvp("ssh", args.data());
     }
+    std::perror("exec ssh failed");
+    return 127;
   }
 
   return 0;