search

JackSlateur / mellanox-rancid

Ugly Mellanox support for rancid
4 stars 3 forks source link

Update mlnxlogin #1

Closed bdbrassart closed 3 years ago

bdbrassart commented 3 years ago

Updated this bit, I don't really know what it does, I'm not much of a programmer.

Was getting a "couldn't compile regular expression pattern: parentheses () not balanced" error. Googled around a bit and saw people talking about different versions of tcl etc..., so I thought maybe my linux/rancid environment had newer dependencies or something.

Realizing that it's a bastardized clogin script, I went to the current version of clogin in my RANCID install, and found the corresponding bits and swapped them out and it worked. /shrug

JackSlateur commented 3 years ago

@bdbrassart I have no idea what this code do After some tests, it does not seems to break, so thank you