search

trptcolin / reply

REPL-y: A fitter, happier, more productive REPL for Clojure.
Eclipse Public License 1.0
337 stars 44 forks source link

Jline error on blink-matching-paren #197

Closed trptcolin closed 5 years ago

trptcolin commented 5 years ago

See https://github.com/technomancy/leiningen/issues/2533 and https://github.com/jline/jline2/issues/298

trptcolin commented 5 years ago

If folks who are experiencing this can help verify that it's specifically due to the blink-matching-paren feature, that'll inform the fix.

Steps:

  1. Validate the badness in the reply repo, via (lein trampoline run)
  2. Change this line to make blink-parens false
  3. Validate whether that causes goodness in the reply repo (via lein trampoline run)

It seems like to make this configurable, this should default to the value in $HOME/.inputrc and true only if that's not present - that's where other config like this lives. Not sure why I didn't implement it that way - definitely a mistake.

And then of course actually-ideally, the feature should work properly in jline2 (also my fault, it looks like).

trptcolin commented 5 years ago

FWIW I haven't been able to reproduce the issue on a VM with Ubuntu 18.04.2 / Leiningen 2.8.1 / OpenJDK 8 (1.8.0_191), so I'll definitely need help.

trptcolin commented 5 years ago

Per @bjaglin on upstream & downstream tickets (thanks!) this appears to be a bad kernel update (https://www.spigotmc.org/threads/ubuntu-18-04-bad-kernel-update.357871/). I'm looking for a non-Minecraft-forum source for this, but... Wowsers!

trptcolin commented 5 years ago

Ah, here's the official stuff:

bbatsov commented 5 years ago

Wow! This doesn't happen every day!

trptcolin commented 5 years ago

Calling this closed - workaround is to upgrade/downgrade kernel to not be Ubuntu 4.15.0-44 or 4.15.0-45.