Skip to content

Commit

Permalink
Merge pull request #1193 from janhq/j/fix-install-engine-empty-engine…
Browse files Browse the repository at this point in the history
…-name

fix: empty engine name while install
  • Loading branch information
namchuai authored Sep 11, 2024
2 parents 4cc7528 + 79fdc15 commit c67d281
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions engine/controllers/command_line_parser.cc
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
#include "commands/run_cmd.h"
#include "commands/server_stop_cmd.h"
#include "config/yaml_config.h"
#include "httplib.h"
#include "services/engine_service.h"
#include "utils/file_manager_utils.h"
#include "utils/logging_utils.h"
Expand Down Expand Up @@ -219,7 +218,7 @@ void CommandLineParser::EngineInstall(CLI::App* parent,
std::string& version) {
auto install_engine_cmd = parent->add_subcommand(engine_name, "");

install_engine_cmd->callback([&] {
install_engine_cmd->callback([=] {
commands::EngineInitCmd eic(engine_name, version);
eic.Exec();
});
Expand All @@ -229,7 +228,7 @@ void CommandLineParser::EngineUninstall(CLI::App* parent,
const std::string& engine_name) {
auto uninstall_engine_cmd = parent->add_subcommand(engine_name, "");

uninstall_engine_cmd->callback([&] {
uninstall_engine_cmd->callback([=] {
commands::EngineUninstallCmd cmd(engine_name);
cmd.Exec();
});
Expand Down

0 comments on commit c67d281

Please sign in to comment.