Package: haskell-mode Version: 2.7.0-2.1 Severity: wishlist X-Debbugs-Cc: era+deb...@iki.fi Tags: patch
The patch for #568579 would appear to also drop support for e.g. emacs-snapshot. I use neither emacs-snapshot nor haskell-mode, so I'm not going to push this any further, but if you publish another update at some point, you might want to change the logic to explicitly ignore xemacs21 and support everything else, instead of the other way around. The patch also changes the behavior for the case when ${FLAVOR} is just 'emacs' -- I'm not intimately familiar with this part of the Debian emacsen infrastructure, but you might want to reinstate the quiet behavior that existed before the NMU. For context, here is the patch from that bug report: -if [ ${FLAVOR} = emacs ]; then exit 0; fi +case ${FLAVOR} in + emacs2[2-9]*) + echo install/${PACKAGE}: Handling install for emacsen flavor ${FLAVOR} + ;; + *) + echo install/${PACKAGE}: Ignoring emacsen flavor ${FLAVOR} -echo install/${PACKAGE}: Handling install for emacsen flavor ${FLAVOR} + exit 0 + ;; +esac Here is my proposal for a revised patch: -if [ ${FLAVOR} = emacs ]; then exit 0; fi +case ${FLAVOR} in + emacs) + exit 0 + ;; + xemacs21) + echo install/${PACKAGE}: Ignoring emacsen flavor ${FLAVOR} + exit 0 + ;; + *) + echo install/${PACKAGE}: Handling install for emacsen flavor ${FLAVOR} + ;; +esac Finally, shouldn't "Depends: whatever | xemacs21" be dropped from debian/control? /* era */ -- If this were a real .signature, it would suck less. Well, maybe not. -- To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org