1
0
mirror of https://github.com/profanity-im/profanity.git synced 2024-11-03 19:37:16 -05:00

Don't stop plugin installation if given path points to a directory

The command "/plugins install <path_to_folder> should install all
plugins in the given folder. This commit removes the check if the path
points to a file (which is repeated in the section installing a plugin
from file) and makes the existing code installing multiple plugins accessible
This commit is contained in:
Philip Flohr 2019-02-26 14:17:59 +01:00
parent 5b7f9dffbc
commit 73549dc751

View File

@ -6612,12 +6612,6 @@ cmd_plugins_install(ProfWin *window, const char *const command, gchar **args)
path = strdup(path); path = strdup(path);
} }
if (access(path, R_OK) != 0) {
cons_show("File not found: %s", path);
free(path);
return TRUE;
}
if (is_regular_file(path)) { if (is_regular_file(path)) {
if (!g_str_has_suffix(path, ".py") && !g_str_has_suffix(path, ".so")) { if (!g_str_has_suffix(path, ".py") && !g_str_has_suffix(path, ".so")) {
cons_show("Plugins must have one of the following extensions: '.py' '.so'"); cons_show("Plugins must have one of the following extensions: '.py' '.so'");
@ -6637,9 +6631,7 @@ cmd_plugins_install(ProfWin *window, const char *const command, gchar **args)
g_string_free(error_message, TRUE); g_string_free(error_message, TRUE);
free(path); free(path);
return TRUE; return TRUE;
} } else if (is_dir(path)) {
if (is_dir(path)) {
PluginsInstallResult* result = plugins_install_all(path); PluginsInstallResult* result = plugins_install_all(path);
if (result->installed || result->failed) { if (result->installed || result->failed) {
if (result->installed) { if (result->installed) {
@ -6666,9 +6658,9 @@ cmd_plugins_install(ProfWin *window, const char *const command, gchar **args)
free(path); free(path);
plugins_free_install_result(result); plugins_free_install_result(result);
return TRUE; return TRUE;
} else {
cons_show("Argument must be a file or directory.");
} }
cons_show("Argument must be a file or directory.");
return TRUE; return TRUE;
} }