This is the intended change from commit cfd60254

Unintended changes were mixed into this commit; discussed with Tom

fix typo in bridge commands struct (missing) field for exit command

ok Tom

ok Tom

As reported by Tom, the following input sequence would lead to a segfault: !<space>?<tab><backspace><tab> Resetting the cursor position in makeargv's special-case handling of shell escapes fixes this. tested by myself and Tom ok Tom

fix conf_patch to avoid overwriting the current interface being shown I have reviewed the Code what stsp describes makes sense and the diff works and appears to solves the issue raised by ryan-sg OK Tom

Afer conf_patch() was called, subsequent configuration items were being shown for the paired interface, rather than the pair interface itself. This happened because conf_patch() wrote the name of the paired interface into the interface name buffer provided by the caller. Use a local buffer instead.

add su command alias for enable

and update manual page and tags associated with su doas sudo and enable

replace mv commands with cp comands

moving files files not copying them is not the best approach file name is way too long too deleting

add clear command to clear terminal screen (feedback from BSDCAN) tra… Ok by Stsp and leaving clear.1 manual page absence is resolved in current and by right should have been in previous releases

add clear command to clear terminal screen (feedback from BSDCAN) training adding clear command and manual entry i have checked and man clear does open a clear manpage in section 1 of the manual

removed binary /usr/sbin/snmpctl I would have love to been able to modify snmp trap command to use the snmp client binary but this requires further thought in the mean time to keep the code clean Im removing that defunct functionality thanks to ryan-sg who raised the issue https://github.com/yellowman/nsh/issues/146

ryan-sg oticed a missing \n, causing the loopback interface to appear on the same line as the header: % Name Status Link Routing-Domain Media lo0 up - 0 vio0 up active 0 Ethernet vio1 down active 0 Ethernet vio2 down active 0 Ethernet vio3 up active 0 Ethernet enc0 down active 0 pflog0 up - 0 pair0 up active 0 Ethernet pair1 up no carrier 2 Ethernet lo2 down - 2 pair2 up active 3 Ethernet lo3 down - 3 The following patch seems to be all that is needed. Troubles with github and ssh keys right now, not worth the time to try and make a pull request over 2 chars :-) index 4ce758e..48cb569 100644 --- a/if.c +++ b/if.c @@ -302,7 +302,7 @@ show_int(int argc, char **argv, FILE *outfile) close(ifs); return 0; } - fputs("% Name Status Link Routing-Domain Media", + fputs("% Name Status Link Routing-Domain Media\n", outfile); for (ifnp = ifn_list; ifnp->if_name != NULL; ifnp++) show_int_status(ifnp->if_name, ifs, outfile); Results: % Name Status Link Routing-Domain Media lo0 up - 0 vio0 up active 0 Ethernet vio1 down active 0 Ethernet vio2 down active 0 Ethernet vio3 up active 0 Ethernet enc0 down active 0 pflog0 up - 0 pair0 up active 0 Ethernet pair1 up no carrier 2 Ethernet lo2 down - 2 pair2 up active 3 Ethernet lo3 down - 3

Bump version to 1.2.1

preparing for new release

clarify how the changes to nsh-dist.txt appear