[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [AUCTeX-devel] [Patch] support xreader
From: |
Mosè Giordano |
Subject: |
Re: [AUCTeX-devel] [Patch] support xreader |
Date: |
Sat, 17 Jun 2017 01:29:04 +0200 |
Hi Antoine,
2017-06-15 15:11 GMT+02:00 Antoine Levitt <address@hidden>:
> So apparently forking GNOME once was not enough, and now the linux mint
> people have a new name for their pdf viewer, xreader (see e.g.
> https://www.linuxmint.com/rel_sarah_cinnamon_whatsnew.php).
Oh my gosh.
> Here's a
> patch for auctex to support it. I had to remove a check for the
> executable, because xreader has DE "x" and APP "reader" while the
> executable is named "xreader", but I don't think that check was useful
> in the first place (might as well try to register with dbus, even if the
> executable does not exist). I also reworded the doc to speak of
> "evince-compatible viewers" (who knows how many of these there will be
> in the future...)
Thanks for the patch! Looks good to me, I installed it. I see that
you already assigned copyright for Emacs, that's good for AUCTeX as
well.
For next contributions, git patches with proper commit message (with
ChangeLog-like body) would be much appreciated ;-)
Bye,
Mosè