summaryrefslogtreecommitdiff
path: root/src/nmcli/nmcli.c
Commit message (Collapse)AuthorAgeFilesLines
* nmcli: add --offline option for "add" and "modify"Lubomir Rintel2022-04-191-1/+6
| | | | | | | | | | | | | | | | | | | | | This adds a global "--offline" option and allows its use with "add" and "modify" commands. The "add" looks like this: $ nmcli --offline conn add type ethernet ens3 ipv4.dns 192.168.1.1 \ >output.nmconnection The "modify" is essentially implementing what's been suggested by Beniamino in bugzilla ticked (referred to below): $ nmcli --offline connection modify ens3 ipv4.dns 192.168.1.1 \ <input.nmconnection >output.nmconnection Other commands don't support the argument at the moment: $ nmcli --offline c up ens3 Error: 'up' command doesn't support --offline mode. https://bugzilla.redhat.com/show_bug.cgi?id=1361145
* nmcli/trivial: consistently order the options in process_command_line()Lubomir Rintel2022-04-191-2/+2
| | | | | | Make the order of nmc_complete_strings() arguments consistent with the multi-way conditional below. Doesn't have any effect, just ensures the ommisions and mistakes are hopefully easier to spot.
* cli: add color codes for deprecated featuresLubomir Rintel2022-03-131-1/+5
| | | | | | | Follow-up commits are going color Wi-Fi networks and connections that rely on deprecated features differently, to provide a visual cue. Add color definitions for those.
* style: fix clang-formatThomas Haller2022-02-171-1/+1
|
* nmcli: do not save/restore terminal attrsLubomir Rintel2022-02-161-6/+0
| | | | | | | | | | | | This is vestigal. It has been in place, because we'd be turning off echo ourselves when asking for password and needed to make sure we'd still terminal in original state upon unexpected termination. This shouldn't be necessary since commit 9d95e1f1753a ('clients/cli: use a nicer password prompt') we let readline take care of this and also clean up after itself in nmc_cleanup_readline(). https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/merge_requests/1100
* format: reformat source tree with clang-format 13.0Thomas Haller2021-11-291-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | We use clang-format for automatic formatting of our source files. Since clang-format is actively maintained software, the actual formatting depends on the used version of clang-format. That is unfortunate and painful, but really unavoidable unless clang-format would be strictly bug-compatible. So the version that we must use is from the current Fedora release, which is also tested by our gitlab-ci. Previously, we were using Fedora 34 with clang-tools-extra-12.0.1-1.fc34.x86_64. As Fedora 35 comes along, we need to update our formatting as Fedora 35 comes with version "13.0.0~rc1-1.fc35". An alternative would be to freeze on version 12, but that has different problems (like, it's cumbersome to rebuild clang 12 on Fedora 35 and it would be cumbersome for our developers which are on Fedora 35 to use a clang that they cannot easily install). The (differently painful) solution is to reformat from time to time, as we switch to a new Fedora (and thus clang) version. Usually we would expect that such a reformatting brings minor changes. But this time, the changes are huge. That is mentioned in the release notes [1] as Makes PointerAligment: Right working with AlignConsecutiveDeclarations. (Fixes https://llvm.org/PR27353) [1] https://releases.llvm.org/13.0.0/tools/clang/docs/ReleaseNotes.html#clang-format
* cli: Provide optional support for libedit instead of readlineLukasz Majewski2021-07-141-0/+4
| | | | | | | | | | | | | | | | | The libreadline starting from version 6 is licensed as GPLv3. For some use cases it is not acceptable to use this license. In the NetworkManager the libreadline is used by nmcli. This change allows using libedit instead of libreadline. Following adjustments were made: 1. The history_set_history_state() is not supported in the libedit. Instead, the where_history() with remove_history() were used to remove the history content if needed. 2. rl_complete_with_tilde_expansion - it is the binary flag used only when one wants to have the expansion support. The libedit is not supporting and hence exporting this flag.
* clang-format: use "IndentPPDirectives:None" instead of "BeforeHash"Thomas Haller2021-07-091-2/+2
| | | | Subjectively, I think this looks better.
* all: don't explicitly include <glib-unix.h>Thomas Haller2021-06-281-1/+0
| | | | We get it now always by "nm-macros-internal.h".
* cli: move from "clients/cli/" to "src/nmcli/"Thomas Haller2021-03-151-0/+1051