Fix dry run warnings (#59)

* Fix dry run warnings Signed-off-by: Lucas Gracioso <[email protected]> * Change order Signed-off-by: Lucas Gracioso <[email protected]> --------- Signed-off-by: Lucas Gracioso <[email protected]>
viniciusferrao · May 8, 2024 · af32d3f · af32d3f
1 parent 5639860
commit af32d3f
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 16 deletions.
diff --git a/src/main.cpp b/src/main.cpp
@@ -100,22 +100,6 @@ int main(int argc, const char** argv)
     LOG_INFO("{} Started", productName);
 
     try {
-        while (!unattended) {
-            char response = 'N';
-            fmt::print("{} will now modify your system, do you want to "
-                       "continue? [Y/N]\n",
-                cloyster::productName);
-            std::cin >> response;
-
-            if (response == 'Y' || response == 'y') {
-                LOG_INFO("Running {}.\n", cloyster::productName)
-                break;
-            } else if (response == 'N' || response == 'n') {
-                LOG_INFO("Stopping {}.\n", cloyster::productName);
-                return EXIT_SUCCESS;
-            }
-        }
-
         if (cloyster::showVersion) {
             fmt::print("{}: Version {}\n", productName, productVersion);
             return EXIT_SUCCESS;
@@ -127,6 +111,22 @@ int main(int argc, const char** argv)
 
         if (cloyster::dryRun) {
             fmt::print("Dry run enabled.\n");
+        } else {
+            while (!unattended) {
+                char response = 'N';
+                fmt::print("{} will now modify your system, do you want to "
+                           "continue? [Y/N]\n",
+                    cloyster::productName);
+                std::cin >> response;
+
+                if (response == 'Y' || response == 'y') {
+                    LOG_INFO("Running {}.\n", cloyster::productName)
+                    break;
+                } else if (response == 'N' || response == 'n') {
+                    LOG_INFO("Stopping {}.\n", cloyster::productName);
+                    return EXIT_SUCCESS;
+                }
+            }
         }
 
         //@TODO implement CLI feature

diff --git a/src/repos.cpp b/src/repos.cpp
@@ -236,6 +236,11 @@ std::vector<std::string> Repos::getxCATOSImageRepos() const
 
 void Repos::configureRepositories() const
 {
+    if (cloyster::dryRun) {
+        LOG_INFO("Would create and configure repositories")
+        return;
+    }
+
     LOG_INFO("Setting up repositories")
 
     createCloysterRepo();