Bug#825809: unclutter: diff for NMU version 8-21.1
Hi Sean,
Sean Whitton wrote:
> I've prepared an NMU for unclutter (versioned as 8-21.1) and uploaded it
> to DELAYED/15. Please feel free to tell me if I should delay it longer.
Yes, please remove it completely. It is incomplete and has unsolved
issues which might be even harder to fix properly afterwards if the
upload succeeds.
I'll try to upload unclutter 8-22 (to experimental) before that NMU
passes throught, but for that, I need your feedback first:
Please check the master branch under
https://salsa.debian.org/debian/unclutter and especially the commits I
made after applying your NMU patch minus the unnecessary NMU stuff:
https://salsa.debian.org/debian/unclutter/commits/master
Main points:
* Make /etc/X11/Xsession.d/90unclutter a slave alternative, too.
* Rename /etc/default/unclutter to /etc/default/unclutter-classic.
* Make debian/unclutter.install executable. (Might have been lost when
you generated your patch as it clearly FTBFS else.)
I'd appreciate if you could look over these changes and tell me if you
agree (as we both should agree on at least the list of slave
alternatives), have alternative suggestions or otherwise comments.
Open issues and why I haven't uploaded the current state to
experimental yet:
* In the current setup, /etc/default/unclutter might need to be a
slave alternative, too. Not sure, though. Thoughts on this are
welcome!
* /etc/default/unclutter and /etc/X11/Xsession.d/90unclutter could be
implementation-independent files, but then they need to be shipped
either by both packages (which causes a file conflict) or we create
some unclutter-common packages with just the common files between
both implementations.
Looks like the least complex implementation to me, but is probably
also the one with the most coordination effort between the
maintainers of both packages. (I suggest to maintain such a package
together in git so that either maintainer can easily upload fixes.)
Regards, Axel
--
,''`. | Axel Beckert <abe@debian.org>, https://people.debian.org/~abe/
: :' : | Debian Developer, ftp.ch.debian.org Admin
`. `' | 4096R: 2517 B724 C5F6 CA99 5329 6E61 2FF9 CD59 6126 16B5
`- | 1024D: F067 EA27 26B9 C3FC 1486 202E C09E 1D89 9593 0EDE
Reply to: