Change in ...osmo-bsc[master]: vty: Fix typo in VTY command descrption -> description

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

laforge gerrit-no-reply at lists.osmocom.org
Fri Jul 19 10:41:07 UTC 2019


laforge has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-bsc/+/14837 )

Change subject: vty: Fix typo in VTY command descrption -> description
......................................................................

vty: Fix typo in VTY command descrption -> description

Let's add deprecated aliases for backwards compatibility

Change-Id: I0e5da9d702910cf2571486e22a56f3ec17d0d67b
---
M doc/examples/osmo-bsc/osmo-bsc.cfg
M doc/examples/osmo-bsc/osmo-bsc_custom-sccp.cfg
M doc/manuals/vty/bsc_vty_reference.xml
M doc/manuals/vty/libbsc_vty_additions.xml
M src/osmo-bsc/bsc_vty.c
5 files changed, 42 insertions(+), 20 deletions(-)

Approvals:
  Jenkins Builder: Verified
  fixeria: Looks good to me, but someone else must approve
  pespin: Looks good to me, approved



diff --git a/doc/examples/osmo-bsc/osmo-bsc.cfg b/doc/examples/osmo-bsc/osmo-bsc.cfg
index 926951b..77b6094 100644
--- a/doc/examples/osmo-bsc/osmo-bsc.cfg
+++ b/doc/examples/osmo-bsc/osmo-bsc.cfg
@@ -32,9 +32,9 @@
   channel allocator ascending
   rach tx integer 9
   rach max transmission 7
-  channel-descrption attach 1
-  channel-descrption bs-pa-mfrms 5
-  channel-descrption bs-ag-blks-res 1
+  channel-description attach 1
+  channel-description bs-pa-mfrms 5
+  channel-description bs-ag-blks-res 1
   early-classmark-sending forbidden
   ipa unit-id 0 0
   oml ipa stream-id 255 line 0
diff --git a/doc/examples/osmo-bsc/osmo-bsc_custom-sccp.cfg b/doc/examples/osmo-bsc/osmo-bsc_custom-sccp.cfg
index 80a38ba..c250fac 100644
--- a/doc/examples/osmo-bsc/osmo-bsc_custom-sccp.cfg
+++ b/doc/examples/osmo-bsc/osmo-bsc_custom-sccp.cfg
@@ -31,9 +31,9 @@
   channel allocator ascending
   rach tx integer 9
   rach max transmission 7
-  channel-descrption attach 1
-  channel-descrption bs-pa-mfrms 5
-  channel-descrption bs-ag-blks-res 1
+  channel-description attach 1
+  channel-description bs-pa-mfrms 5
+  channel-description bs-ag-blks-res 1
   early-classmark-sending forbidden
   ipa unit-id 0 0
   oml ipa stream-id 255 line 0
diff --git a/doc/manuals/vty/bsc_vty_reference.xml b/doc/manuals/vty/bsc_vty_reference.xml
index db4f2c1..85c0cb7 100644
--- a/doc/manuals/vty/bsc_vty_reference.xml
+++ b/doc/manuals/vty/bsc_vty_reference.xml
@@ -2983,24 +2983,24 @@
         <param name='7' doc='Maximum number of 7 RACH burst transmissions' />
       </params>
     </command>
-    <command id='channel-descrption attach (0|1)'>
+    <command id='channel-description attach (0|1)'>
       <params>
-        <param name='channel-descrption' doc='Channel Description' />
+        <param name='channel-description' doc='Channel Description' />
         <param name='attach' doc='Set if attachment is required' />
         <param name='0' doc='Attachment is NOT required' />
         <param name='1' doc='Attachment is required (standard)' />
       </params>
     </command>
-    <command id='channel-descrption bs-pa-mfrms <2-9>'>
+    <command id='channel-description bs-pa-mfrms <2-9>'>
       <params>
-        <param name='channel-descrption' doc='Channel Description' />
+        <param name='channel-description' doc='Channel Description' />
         <param name='bs-pa-mfrms' doc='Set number of multiframe periods for paging groups' />
         <param name='<2-9>' doc='Number of multiframe periods for paging groups' />
       </params>
     </command>
-    <command id='channel-descrption bs-ag-blks-res <0-7>'>
+    <command id='channel-description bs-ag-blks-res <0-7>'>
       <params>
-        <param name='channel-descrption' doc='Channel Description' />
+        <param name='channel-description' doc='Channel Description' />
         <param name='bs-ag-blks-res' doc='Set number of blocks reserved for access grant' />
         <param name='<0-7>' doc='Number of blocks reserved for access grant' />
       </params>
diff --git a/doc/manuals/vty/libbsc_vty_additions.xml b/doc/manuals/vty/libbsc_vty_additions.xml
index cf61961..dbf4080 100644
--- a/doc/manuals/vty/libbsc_vty_additions.xml
+++ b/doc/manuals/vty/libbsc_vty_additions.xml
@@ -34,7 +34,7 @@
 	  Only applies to Ericsson OML2000 based BTSs.
 	</description>
       </command>
-      <command id='channel-descrption attach (0|1)'>
+      <command id='channel-description attach (0|1)'>
         <description>
 	  Configure whether the IMSI ATTACH (and DETACH) procedures
 	  shall be used by MS in this cell.  The default should be enabled.
@@ -160,7 +160,7 @@
 	  0) - which is also the default configuration.</warning>
 	</description>
       </command>
-      <command id='channel-descrption bs-ag-blks-res <0-7>'>
+      <command id='channel-description bs-ag-blks-res <0-7>'>
 	<description>
 	  Using this command, you can specify how many blocks of the
 	  downlink CCCH should be reserved for exclusive usage as AGCH.
diff --git a/src/osmo-bsc/bsc_vty.c b/src/osmo-bsc/bsc_vty.c
index 24f8313..b19c950 100644
--- a/src/osmo-bsc/bsc_vty.c
+++ b/src/osmo-bsc/bsc_vty.c
@@ -841,11 +841,11 @@
 		rach_max_trans_raw2val(bts->si_common.rach_control.max_trans),
 		VTY_NEWLINE);
 
-	vty_out(vty, "  channel-descrption attach %u%s",
+	vty_out(vty, "  channel-description attach %u%s",
 		bts->si_common.chan_desc.att, VTY_NEWLINE);
-	vty_out(vty, "  channel-descrption bs-pa-mfrms %u%s",
+	vty_out(vty, "  channel-description bs-pa-mfrms %u%s",
 		bts->si_common.chan_desc.bs_pa_mfrms + 2, VTY_NEWLINE);
-	vty_out(vty, "  channel-descrption bs-ag-blks-res %u%s",
+	vty_out(vty, "  channel-description bs-ag-blks-res %u%s",
 		bts->si_common.chan_desc.bs_ag_blks_res, VTY_NEWLINE);
 
 	if (bts->ccch_load_ind_thresh != 10)
@@ -2420,7 +2420,7 @@
 
 DEFUN(cfg_bts_chan_desc_att,
       cfg_bts_chan_desc_att_cmd,
-      "channel-descrption attach (0|1)",
+      "channel-description attach (0|1)",
 	CD_STR
       "Set if attachment is required\n"
       "Attachment is NOT required\n"
@@ -2430,10 +2430,17 @@
 	bts->si_common.chan_desc.att = atoi(argv[0]);
 	return CMD_SUCCESS;
 }
+ALIAS_DEPRECATED(cfg_bts_chan_desc_att,
+		 cfg_bts_chan_dscr_att_cmd,
+		 "channel-descrption attach (0|1)",
+		 CD_STR
+		 "Set if attachment is required\n"
+		 "Attachment is NOT required\n"
+		 "Attachment is required (standard)\n");
 
 DEFUN(cfg_bts_chan_desc_bs_pa_mfrms,
       cfg_bts_chan_desc_bs_pa_mfrms_cmd,
-      "channel-descrption bs-pa-mfrms <2-9>",
+      "channel-description bs-pa-mfrms <2-9>",
 	CD_STR
       "Set number of multiframe periods for paging groups\n"
       "Number of multiframe periods for paging groups\n")
@@ -2444,10 +2451,16 @@
 	bts->si_common.chan_desc.bs_pa_mfrms = bs_pa_mfrms - 2;
 	return CMD_SUCCESS;
 }
+ALIAS_DEPRECATED(cfg_bts_chan_desc_bs_pa_mfrms,
+		 cfg_bts_chan_dscr_bs_pa_mfrms_cmd,
+		 "channel-descrption bs-pa-mfrms <2-9>",
+		 CD_STR
+		 "Set number of multiframe periods for paging groups\n"
+		 "Number of multiframe periods for paging groups\n");
 
 DEFUN(cfg_bts_chan_desc_bs_ag_blks_res,
       cfg_bts_chan_desc_bs_ag_blks_res_cmd,
-      "channel-descrption bs-ag-blks-res <0-7>",
+      "channel-description bs-ag-blks-res <0-7>",
 	CD_STR
       "Set number of blocks reserved for access grant\n"
       "Number of blocks reserved for access grant\n")
@@ -2458,6 +2471,12 @@
 	bts->si_common.chan_desc.bs_ag_blks_res = bs_ag_blks_res;
 	return CMD_SUCCESS;
 }
+ALIAS_DEPRECATED(cfg_bts_chan_desc_bs_ag_blks_res,
+		 cfg_bts_chan_dscr_bs_ag_blks_res_cmd,
+		 "channel-descrption bs-ag-blks-res <0-7>",
+		 CD_STR
+		 "Set number of blocks reserved for access grant\n"
+		 "Number of blocks reserved for access grant\n");
 
 #define CCCH_STR "Common Control Channel\n"
 
@@ -5273,8 +5292,11 @@
 	install_element(BTS_NODE, &cfg_bts_rach_tx_integer_cmd);
 	install_element(BTS_NODE, &cfg_bts_rach_max_trans_cmd);
 	install_element(BTS_NODE, &cfg_bts_chan_desc_att_cmd);
+	install_element(BTS_NODE, &cfg_bts_chan_dscr_att_cmd);
 	install_element(BTS_NODE, &cfg_bts_chan_desc_bs_pa_mfrms_cmd);
+	install_element(BTS_NODE, &cfg_bts_chan_dscr_bs_pa_mfrms_cmd);
 	install_element(BTS_NODE, &cfg_bts_chan_desc_bs_ag_blks_res_cmd);
+	install_element(BTS_NODE, &cfg_bts_chan_dscr_bs_ag_blks_res_cmd);
 	install_element(BTS_NODE, &cfg_bts_ccch_load_ind_thresh_cmd);
 	install_element(BTS_NODE, &cfg_bts_rach_nm_b_thresh_cmd);
 	install_element(BTS_NODE, &cfg_bts_rach_nm_ldavg_cmd);

-- 
To view, visit https://gerrit.osmocom.org/c/osmo-bsc/+/14837
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Change-Id: I0e5da9d702910cf2571486e22a56f3ec17d0d67b
Gerrit-Change-Number: 14837
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <axilirator at gmail.com>
Gerrit-Reviewer: laforge <laforge at gnumonks.org>
Gerrit-Reviewer: pespin <pespin at sysmocom.de>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20190719/ec590fbc/attachment.htm>


More information about the gerrit-log mailing list