Skip to content

Commit 1fd1850

Browse files
committed
Successfully fail when running cmkr subcommands
1 parent 8b19441 commit 1fd1850

File tree

2 files changed

+3
-15
lines changed

2 files changed

+3
-15
lines changed

include/arguments.hpp

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,5 +7,3 @@ const char *handle_args(int argc, char **argv);
77

88
} // namespace args
99
} // namespace cmkr
10-
11-
const char *cmkr_args_handle_args(int, char **);

src/arguments.cpp

Lines changed: 3 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -37,31 +37,21 @@ const char *handle_args(int argc, char **argv) {
3737
} else if (main_arg == "build") {
3838
auto ret = build::run(argc, argv);
3939
if (ret)
40-
return "CMake build error!";
40+
throw std::runtime_error("CMake build failed!");
4141
return "CMake build completed!";
4242
} else if (main_arg == "install") {
4343
auto ret = build::install();
4444
if (ret)
45-
return "CMake install error!";
45+
throw std::runtime_error("CMake install failed!");
4646
return "CMake install completed!";
4747
} else if (main_arg == "clean") {
4848
auto ret = build::clean();
4949
if (ret)
50-
return "CMake clean error!";
50+
throw std::runtime_error("CMake clean failed!");
5151
return "Cleaned build directory!";
5252
} else {
5353
throw std::runtime_error(cmkr::help::message());
5454
}
5555
}
5656
} // namespace args
5757
} // namespace cmkr
58-
59-
const char *cmkr_args_handle_args(int argc, char **argv) {
60-
try {
61-
return cmkr::args::handle_args(argc, argv);
62-
} catch (const std::exception &e) {
63-
return e.what();
64-
} catch (...) {
65-
return "Unknown error!";
66-
}
67-
}

0 commit comments

Comments
 (0)