dandavison / xenops

An editing environment for LaTeX mathematical documents
MIT License
228 stars 14 forks source link

tex-buf.el has been merged into tex.el #48

Closed tarsius closed 2 years ago

tarsius commented 2 years ago

I've noticed this because I maintain the Emacsmirror and the tools, that I use to do so, inform me when a dependency of a mirrored package isn't satisfied (anymore).

In auctex tex-buf.el was merged into tex.el but your package still requires tex-buf. You can probably fix this by depending on tex instead, or if you already do that by dropping the additional (require 'tex-buf).

tarsius commented 2 years ago

Friendly ping!

dandavison commented 2 years ago

Hi @tarsius, thanks. I sort-of addressed this in https://github.com/dandavison/xenops/pull/40 which made it no longer an error if tex-buf was not available. I was thinking that leaving it in that state -- rather than deleting the require entirely -- was good for backwards compatibility, i.e. for people who still have the old version of auctex installed. It looks like my declared dependency is on auctex "12.2.0" which I think still has tex-buf.el. What do you think -- is that backwards-compatibility argument reasonable?

tarsius commented 2 years ago

Thanks!