summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThomas Haller <thaller@redhat.com>2023-03-14 08:31:40 +0100
committerThomas Haller <thaller@redhat.com>2023-03-27 11:22:45 +0200
commitcec89aa2e4f360a0212967e8aa40334d7e9883e8 (patch)
tree7f1ab6ce85af68265019e085e9dc37ba092f8927
parent75c63f7a6c6c071240ac90ee028a51c41fcf696a (diff)
downloadNetworkManager-cec89aa2e4f360a0212967e8aa40334d7e9883e8.tar.gz
cli: drop unused readonly properties "wifi.{rate,tx-power}"
These properties were never implemented. Also, they were not settable via nmcli. Drop them from being shown. This is an API break, but hopefully something that does not affect anybody in a bad way.
-rw-r--r--src/libnmc-setting/nm-meta-setting-desc.c8
-rw-r--r--src/nmcli/gen-metadata-nm-settings-nmcli.xml.in4
-rw-r--r--src/tests/client/test-client.check-on-disk/test_004.expected12
3 files changed, 4 insertions, 20 deletions
diff --git a/src/libnmc-setting/nm-meta-setting-desc.c b/src/libnmc-setting/nm-meta-setting-desc.c
index 1770a2c753..6addbd55ce 100644
--- a/src/libnmc-setting/nm-meta-setting-desc.c
+++ b/src/libnmc-setting/nm-meta-setting-desc.c
@@ -8027,14 +8027,6 @@ static const NMMetaPropertyInfo *const property_infos_WIRELESS[] = {
PROPERTY_INFO_WITH_DESC (NM_SETTING_WIRELESS_BSSID,
.property_type = &_pt_gobject_mac,
),
- PROPERTY_INFO_WITH_DESC (NM_SETTING_WIRELESS_RATE,
- /* Do not allow setting 'rate'. It is not implemented in NM core. */
- .property_type = &_pt_gobject_readonly,
- ),
- PROPERTY_INFO_WITH_DESC (NM_SETTING_WIRELESS_TX_POWER,
- /* Do not allow setting 'tx-power'. It is not implemented in NM core. */
- .property_type = &_pt_gobject_readonly,
- ),
PROPERTY_INFO_WITH_DESC (NM_SETTING_WIRELESS_MAC_ADDRESS,
.property_type = &_pt_gobject_mac,
.is_cli_option = TRUE,
diff --git a/src/nmcli/gen-metadata-nm-settings-nmcli.xml.in b/src/nmcli/gen-metadata-nm-settings-nmcli.xml.in
index 8ac924c829..ab79fb8263 100644
--- a/src/nmcli/gen-metadata-nm-settings-nmcli.xml.in
+++ b/src/nmcli/gen-metadata-nm-settings-nmcli.xml.in
@@ -30,10 +30,6 @@
description="Wireless channel to use for the Wi-Fi connection. The device will only join (or create for Ad-Hoc networks) a Wi-Fi network on the specified channel. Because channel numbers overlap between bands, this property also requires the &quot;band&quot; property to be set." />
<property name="bssid"
description="If specified, directs the device to only associate with the given access point. This capability is highly driver dependent and not supported by all devices. Note: this property does not control the BSSID used when creating an Ad-Hoc network and is unlikely to in the future. Locking a client profile to a certain BSSID will prevent roaming and also disable background scanning. That can be useful, if there is only one access point for the SSID." />
- <property name="rate"
- description="This property is not implemented and has no effect." />
- <property name="tx-power"
- description="This property is not implemented and has no effect." />
<property name="mac-address"
alias="mac"
description="If specified, this connection will only apply to the Wi-Fi device whose permanent MAC address matches. This property does not change the MAC address of the device (i.e. MAC spoofing)." />
diff --git a/src/tests/client/test-client.check-on-disk/test_004.expected b/src/tests/client/test-client.check-on-disk/test_004.expected
index b69eb1d9d6..af377f2b5b 100644
--- a/src/tests/client/test-client.check-on-disk/test_004.expected
+++ b/src/tests/client/test-client.check-on-disk/test_004.expected
@@ -58,12 +58,12 @@ location: src/tests/client/test-client.py:test_004()/7
cmd: $NMCLI connection mod con-xx1 ipv4.addresses 192.168.77.5/24 ipv4.routes '2.3.4.5/32 192.168.77.1' ipv6.addresses 1:2:3:4::6/64 ipv6.routes 1:2:3:4:5:6::5/128
lang: C
returncode: 0
-size: 5164
+size: 5080
location: src/tests/client/test-client.py:test_004()/8
cmd: $NMCLI con s con-xx1
lang: C
returncode: 0
-stdout: 5033 bytes
+stdout: 4949 bytes
>>>
connection.id: con-xx1
connection.uuid: UUID-con-xx1-REPLACED-REPLACED-REPLA
@@ -97,8 +97,6 @@ connection.wait-activation-delay: -1
802-11-wireless.band: a
802-11-wireless.channel: 0
802-11-wireless.bssid: --
-802-11-wireless.rate: 0
-802-11-wireless.tx-power: 0
802-11-wireless.mac-address: --
802-11-wireless.cloned-mac-address: --
802-11-wireless.generate-mac-address-mask:--
@@ -174,12 +172,12 @@ proxy.pac-url: --
proxy.pac-script: --
<<<
-size: 5200
+size: 5116
location: src/tests/client/test-client.py:test_004()/9
cmd: $NMCLI con s con-xx1
lang: pl_PL.UTF-8
returncode: 0
-stdout: 5059 bytes
+stdout: 4975 bytes
>>>
connection.id: con-xx1
connection.uuid: UUID-con-xx1-REPLACED-REPLACED-REPLA
@@ -213,8 +211,6 @@ connection.wait-activation-delay: -1
802-11-wireless.band: a
802-11-wireless.channel: 0
802-11-wireless.bssid: --
-802-11-wireless.rate: 0
-802-11-wireless.tx-power: 0
802-11-wireless.mac-address: --
802-11-wireless.cloned-mac-address: --
802-11-wireless.generate-mac-address-mask:--