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
