[FROG] readline 8.1 and bracketed pasting in vtysh

[mike tancsa]

On 2/2/2021 11:51 AM, mike tancsa wrote:
> On 2/1/2021 10:27 PM, Mark Kamichoff wrote:
>> tl;dr readline 8.1 turns on bracketed paste and appears to break
>> multi-line pasting in vtysh
>>
> Actually, it seems more broken than that for me. If I do something
> simple like
>
> Starting bgpd.
> Booting for integrated-vtysh-config...
> The route-map 'TESTOUT' does not exist.
> 0{testrouter-2021}#
>
I recompiled the port with readline 8.1, and its the same behaviour.
However, its not 100% clear if this is just a parsing issue at boot or not

0{testrouter-2021}# pkg info | grep -i readline
readline-8.1.0                 Library for editing command lines as they
are typed
0{testrouter-2021}#

0{testrouter-2021}# vtysh

Hello, this is FRRouting (version 7.5).
Copyright 1996-2005 Kunihiro Ishiguro, et al.

testrouter-2021# sh run
Building configuration...

Current configuration:
!
frr version 7.5
frr defaults traditional
hostname testrouter-2021
service integrated-vtysh-config
!
router bgp 65666
 neighbor 192.168.1.1 remote-as 65667
 !
 address-family ipv4 unicast
  neighbor 192.168.1.1 route-map TESTOUT out
 exit-address-family
!
route-map TESTOUT permit 10
 set as-path prepend 65666
!
line vty
!
end
testrouter-2021# conf t
testrouter-2021(config)# route-map TESTOUT permit 10
testrouter-2021(config-route-map)# set as-path prepend 65666

testrouter-2021# wr
Note: this version of vtysh never writes vtysh.conf

Warning: attempting direct configuration write without watchfrr.
File permissions and ownership may be incorrect, or write may fail.

Building Configuration...
Integrated configuration saved to /usr/local/etc/frr/frr.conf
[OK]
testrouter-2021# exit
0{testrouter-2021}# /usr/local/etc/rc.d/frr stop
Stopping bgpd.
Waiting for PIDS: 42176.
Stopping staticd.
Waiting for PIDS: 42171.
Stopping zebra.
Waiting for PIDS: 42166.
0{testrouter-2021}# /usr/local/etc/rc.d/frr start
Checking intergrated config...
Checking vtysh.conf
OK
Starting zebra.
2021/02/02 12:02:04 ZEBRA: [EC 4043309111] Disabling MPLS support (no
kernel support)
Starting staticd.
Starting bgpd.
Booting for integrated-vtysh-config...
The route-map 'TESTOUT' does not exist.
0{testrouter-2021}# vtysh

Hello, this is FRRouting (version 7.5).
Copyright 1996-2005 Kunihiro Ishiguro, et al.

testrouter-2021# show route-map
  <cr> 
  WORD  route-map name
     TESTOUT
testrouter-2021# show route-map TESTOUT
ZEBRA:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
  Action:
    Exit routemap
BGP:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
    as-path prepend 65666
  Call clause:
  Action:
    Exit routemap
testrouter-2021#

testrouter-2021# exit
0{testrouter-2021}# /usr/local/etc/rc.d/frr stop
Stopping bgpd.
Waiting for PIDS: 42238.
Stopping staticd.
Waiting for PIDS: 42233.
Stopping zebra.
Waiting for PIDS: 42228.
0{testrouter-2021}# /usr/local/etc/rc.d/frr start
Checking intergrated config...
Checking vtysh.conf
OK
Starting zebra.
2021/02/02 12:04:46 ZEBRA: [EC 4043309111] Disabling MPLS support (no
kernel support)
Starting staticd.
Starting bgpd.
Booting for integrated-vtysh-config...
The route-map 'TESTOUT' does not exist.
0{testrouter-2021}# vtysh

Hello, this is FRRouting (version 7.5).
Copyright 1996-2005 Kunihiro Ishiguro, et al.

testrouter-2021# sh route-map
ZEBRA:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
  Action:
    Exit routemap
BGP:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
    as-path prepend 65666
  Call clause:
  Action:
    Exit routemap
testrouter-2021# sh route-map TESTOUT
ZEBRA:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
  Call clause:
  Action:
    Exit routemap
BGP:
route-map: TESTOUT Invoked: 0 Optimization: enabled Processed Change: false
 permit, sequence 10 Invoked 0
  Match clauses:
  Set clauses:
    as-path prepend 65666
  Call clause:
  Action:
    Exit routemap
testrouter-2021#