clojure-mode icon indicating copy to clipboard operation
clojure-mode copied to clipboard

Sync clojure-mode-extra-font-locking.el version with clojure-mode.el

Open manphiz opened this issue 1 year ago • 4 comments

  • It looks like the version of clojure-mode-extra-font-locking.el has been out-of-sync with clojure-mode.el and has been stuck in 3.0.0 for a long time.
  • This patch sync the version with the current snapshot.

Note: it would be great to fix 5.18.x branch but not really needed I guess.

  • [X] The commits are consistent with our [contribution guidelines][1].
  • [ ] You've added tests (if possible) to cover your change(s). Bugfix, indentation, and font-lock tests are extremely important!
  • [X] You've run M-x checkdoc and fixed any warnings in the code you've written.
  • [X] You've updated the changelog (if adding/changing user-visible functionality).
  • [ ] You've updated the readme (if adding/changing user-visible functionality).

manphiz avatar Mar 25 '24 21:03 manphiz

That's intentional. It's a separate package, so it should have a different version number.

bbatsov avatar Mar 25 '24 21:03 bbatsov

I see. However, it looks like on stable.melpa.org the version of clojure-mode-extra-font-locking is the same as clojure-mode[1]. This may cause an issue that a locally built clojure-mode-extra-font-locking may get superseded by the version on stable.melpa.org, but they are actually the same version. Is there a way to let stable.melpa.org honor the version specified in the source file?

[1] https://stable.melpa.org/#/clojure-mode-extra-font-locking

manphiz avatar Mar 25 '24 22:03 manphiz

I don't think so, as MELPA uses the newest tag as the version instead of the version metadata. Probably the only solution would be to move clojure-mode-extra-font-locking to a separate repo. CIDER's dynamic font-locking made this package somewhat obsolete, so I haven't thought much about it recently.

bbatsov avatar Mar 26 '24 05:03 bbatsov

Ack. Please feel free to close this (or leave open as you like :)

manphiz avatar Mar 26 '24 06:03 manphiz

Additional semi-relevant question: did we lose font-locking in the mini-buffer with these changes in version 3.0?

jackrusher avatar Sep 24 '24 14:09 jackrusher

@jackrusher For CIDER's eval commands? I don't recall if we ever had font-locking there.

bbatsov avatar Sep 24 '24 14:09 bbatsov

@bbatsov I must have been hallucinating, but I swear there was syntax highlighting on results in the mini-buffer up until cider 3.0.0 🤔

jackrusher avatar Sep 24 '24 14:09 jackrusher

Digging in the history of CIDER I see this was the case about 10 years ago, but I'm not sure when and why we changed this. Definitely not related to any changes in clojure-mode, though.

bbatsov avatar Sep 24 '24 19:09 bbatsov

@jackrusher After digging a bit more - seems that's a recent regression caused by https://github.com/clojure-emacs/cider/pull/3661

bbatsov avatar Sep 24 '24 19:09 bbatsov

And it's now fixed in https://github.com/clojure-emacs/cider/pull/3743 Turned out to be a trivial issue of using the wrong local variable.

bbatsov avatar Sep 24 '24 20:09 bbatsov

@bbatsov great, thanks!

jackrusher avatar Sep 25 '24 18:09 jackrusher