mirror of
https://gitlab.freedesktop.org/NetworkManager/NetworkManager.git
synced 2025-12-28 21:30:10 +01:00
cli: allow completing filenames
Allow the completion function to indicate that the word should be
completed as a filename by the shell.
(cherry picked from commit eb724293c2)
This commit is contained in:
parent
a9728a6112
commit
09f9831fc6
5 changed files with 24 additions and 8 deletions
|
|
@ -3785,6 +3785,7 @@ _meta_abstract_complete (const NMMetaAbstractInfo *abstract_info, const char *te
|
|||
nmc_meta_environment_arg,
|
||||
&ctx,
|
||||
text,
|
||||
NULL,
|
||||
&values_to_free);
|
||||
if (values)
|
||||
return values_to_free ?: g_strdupv ((char **) values);
|
||||
|
|
@ -4525,11 +4526,12 @@ run_rl_generator (rl_compentry_func_t *generator_func, const char *prefix)
|
|||
}
|
||||
|
||||
static gboolean
|
||||
complete_option (const NMMetaAbstractInfo *abstract_info, const char *prefix, NMConnection *context_connection)
|
||||
complete_option (NmCli *nmc, const NMMetaAbstractInfo *abstract_info, const char *prefix, NMConnection *context_connection)
|
||||
{
|
||||
const OptionInfo *candidate;
|
||||
const char *const*values;
|
||||
gs_strfreev char **values_to_free = NULL;
|
||||
gboolean complete_filename = FALSE;
|
||||
const NMMetaOperationContext ctx = {
|
||||
.connection = context_connection,
|
||||
};
|
||||
|
|
@ -4539,7 +4541,12 @@ complete_option (const NMMetaAbstractInfo *abstract_info, const char *prefix, NM
|
|||
nmc_meta_environment_arg,
|
||||
&ctx,
|
||||
prefix,
|
||||
&complete_filename,
|
||||
&values_to_free);
|
||||
if (complete_filename) {
|
||||
nmc->return_value = NMC_RESULT_COMPLETE_FILE;
|
||||
return TRUE;
|
||||
}
|
||||
if (values) {
|
||||
for (; values[0]; values++)
|
||||
g_print ("%s\n", values[0]);
|
||||
|
|
@ -4556,13 +4563,13 @@ complete_option (const NMMetaAbstractInfo *abstract_info, const char *prefix, NM
|
|||
}
|
||||
|
||||
static void
|
||||
complete_property (const char *setting_name, const char *property, const char *prefix, NMConnection *connection)
|
||||
complete_property (NmCli *nmc, const char *setting_name, const char *property, const char *prefix, NMConnection *connection)
|
||||
{
|
||||
const NMMetaPropertyInfo *property_info;
|
||||
|
||||
property_info = nm_meta_property_info_find_by_name (setting_name, property);
|
||||
if (property_info)
|
||||
complete_option ((const NMMetaAbstractInfo *) property_info, prefix, connection);
|
||||
complete_option (nmc, (const NMMetaAbstractInfo *) property_info, prefix, connection);
|
||||
}
|
||||
|
||||
/*****************************************************************************/
|
||||
|
|
@ -4652,8 +4659,10 @@ nmc_read_connection_properties (NmCli *nmc,
|
|||
if (!get_value (&value, argc, argv, option, error))
|
||||
return FALSE;
|
||||
|
||||
if (!*argc && nmc->complete)
|
||||
complete_property (setting, strv[1], value ?: "", connection);
|
||||
if (!*argc && nmc->complete) {
|
||||
complete_property (nmc, setting, strv[1], value ?: "", connection);
|
||||
return TRUE;
|
||||
}
|
||||
|
||||
if (!set_property (nmc->client, connection, setting_name, strv[1], value, modifier, error))
|
||||
return FALSE;
|
||||
|
|
@ -4734,7 +4743,7 @@ nmc_read_connection_properties (NmCli *nmc,
|
|||
return FALSE;
|
||||
|
||||
if (!*argc && nmc->complete)
|
||||
complete_option (chosen, value ?: "", connection);
|
||||
complete_option (nmc, chosen, value ?: "", connection);
|
||||
|
||||
if (!set_option (nmc, connection, chosen, value, error))
|
||||
return FALSE;
|
||||
|
|
@ -8861,7 +8870,7 @@ do_connection_import (NmCli *nmc, int argc, char **argv)
|
|||
if ( argc == 1
|
||||
&& nmc->complete) {
|
||||
nmc_complete_strings (*argv, "wireguard");
|
||||
complete_option ((const NMMetaAbstractInfo *) nm_meta_property_info_vpn_service_type,
|
||||
complete_option (nmc, (const NMMetaAbstractInfo *) nm_meta_property_info_vpn_service_type,
|
||||
*argv,
|
||||
NULL);
|
||||
}
|
||||
|
|
|
|||
|
|
@ -273,6 +273,7 @@ nm_meta_abstract_info_complete (const NMMetaAbstractInfo *abstract_info,
|
|||
gpointer environment_user_data,
|
||||
const NMMetaOperationContext *operation_context,
|
||||
const char *text,
|
||||
gboolean *out_complete_filename,
|
||||
char ***out_to_free)
|
||||
{
|
||||
const char *const*values;
|
||||
|
|
@ -292,6 +293,7 @@ nm_meta_abstract_info_complete (const NMMetaAbstractInfo *abstract_info,
|
|||
environment_user_data,
|
||||
operation_context,
|
||||
text,
|
||||
out_complete_filename,
|
||||
out_to_free);
|
||||
|
||||
nm_assert (!*out_to_free || values == (const char *const*) *out_to_free);
|
||||
|
|
|
|||
|
|
@ -69,6 +69,7 @@ const char *const*nm_meta_abstract_info_complete (const NMMetaAbstractInfo *abst
|
|||
gpointer environment_user_data,
|
||||
const NMMetaOperationContext *operation_context,
|
||||
const char *text,
|
||||
gboolean *out_complete_filename,
|
||||
char ***out_to_free);
|
||||
|
||||
/*****************************************************************************/
|
||||
|
|
|
|||
|
|
@ -788,7 +788,7 @@ _env_warn_fcn (const NMMetaEnvironment *environment,
|
|||
const NMMetaPropertyInfo *property_info, const NMMetaEnvironment *environment, gpointer environment_user_data, NMSetting *setting, const char *value, GError **error
|
||||
|
||||
#define ARGS_COMPLETE_FCN \
|
||||
const NMMetaPropertyInfo *property_info, const NMMetaEnvironment *environment, gpointer environment_user_data, const NMMetaOperationContext *operation_context, const char *text, char ***out_to_free
|
||||
const NMMetaPropertyInfo *property_info, const NMMetaEnvironment *environment, gpointer environment_user_data, const NMMetaOperationContext *operation_context, const char *text, gboolean *out_complete_filename, char ***out_to_free
|
||||
|
||||
#define ARGS_VALUES_FCN \
|
||||
const NMMetaPropertyInfo *property_info, char ***out_to_free
|
||||
|
|
@ -8159,6 +8159,7 @@ _meta_type_property_info_complete_fcn (const NMMetaAbstractInfo *abstract_info,
|
|||
gpointer environment_user_data,
|
||||
const NMMetaOperationContext *operation_context,
|
||||
const char *text,
|
||||
gboolean *out_complete_filename,
|
||||
char ***out_to_free)
|
||||
{
|
||||
const NMMetaPropertyInfo *info = (const NMMetaPropertyInfo *) abstract_info;
|
||||
|
|
@ -8171,6 +8172,7 @@ _meta_type_property_info_complete_fcn (const NMMetaAbstractInfo *abstract_info,
|
|||
environment_user_data,
|
||||
operation_context,
|
||||
text,
|
||||
out_complete_filename,
|
||||
out_to_free);
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -221,6 +221,7 @@ struct _NMMetaPropertyType {
|
|||
gpointer environment_user_data,
|
||||
const NMMetaOperationContext *operation_context,
|
||||
const char *text,
|
||||
gboolean *out_complete_filename,
|
||||
char ***out_to_free);
|
||||
|
||||
/* Whether set_fcn() supports the '-' modifier. That is, whether the property
|
||||
|
|
@ -444,6 +445,7 @@ struct _NMMetaType {
|
|||
gpointer environment_user_data,
|
||||
const NMMetaOperationContext *operation_context,
|
||||
const char *text,
|
||||
gboolean *out_complete_filename,
|
||||
char ***out_to_free);
|
||||
};
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue