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

update index.html and index.md tidy up basics.md to improve pagination add improved nsh manual examples

Update README.md to improve flow and readability of install instructions

Update README.md to reflect improvements to NSH make file and to reflect new integration scripts

fix handling of <ctrl> + <c> interruption that previously caused lock files to remain and prevent further editing The default SIGINT handler does the wrong thing when a file is currently locked for editing. Please try the diff below. ensure that edited files get unlocked if the user types Ctrl-C at prompt While editing files, temporarily disable the default SIGINT handler which does a long jump to displaying a command prompt. Otherwise we would jump away without removing our lock on the file and future attempts to edit the file would fail. Issue found by Tom Smyth. Fixed by stsp@ Thanks