search

trptcolin / reply

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

Display JVM version information on startup #151

Closed bbatsov closed 10 years ago

bbatsov commented 10 years ago

It'd be nice if the startup banner of reply included some JVM version information. Perhaps something like:

Java(TM) SE Runtime Environment (build 1.7.0_17-b02)

I guess you're simply using the information provided by nREPL's describe-session at this point; I've also suggested this information for including there, although it's trivially obtainable anyways.

As someone who has to deal with nREPL frequently I'd appreciate displaying its version in the banner as well, although most users probably don't care about it.

trptcolin commented 10 years ago

I'd be open to a patch adding JVM version info on its own line, and the nREPL version on the same line as the REPLy version if anybody wants to patch this. This would go here: https://github.com/trptcolin/reply/blob/d9abc320d227b9524d930e9b24c04822e3038238/src/clj/reply/initialization.clj#L7-L9

bbatsov commented 10 years ago

OK, I'll take care of this.