[Pkg-emacsen-addons] Bug#850630: Bug#850630: elpa-company: Fails to upgrade from 0.8.12-4 when xemacs21 installed

[David Bremner]

Sean Whitton <spwhitton at spwhitton.name> writes:



> +++ b/debian/elpa-company.prerm
> @@ -0,0 +1,16 @@
> +#!/bin/sh
> +
> +set -e
> +
> +# fix upgrade from 0.8.12-5 and below, which can fail if xemacs21 is

I think you mean -4 and below
> +# also installed.  We manually upgrade the emacsen-common remove
> +# script so that it won't exit non-zero, which blocks the upgrade

> +if [ "$1" = "failed-upgrade" ] && dpkg --compare-versions "$2" lt 0.8.12-5; then
> +    broken_remove_script="/usr/lib/emacsen-common/packages/remove/elpa-company"
> +    sed --quiet -i "$broken_remove_script" \
> +        -e 's/find ${elc_dir} -type l -delete/[ -d ${elc_dir} ] && find ${elc_dir} -type l -delete/'

this is OK, but it might be simpler to add the 'exit 0' that dh-elpa 1.6
added to the xemacs case.

> +    sed --quiet -i "$broken_remove_script" \
> +        -e 's/emacs23)/emacs2[0123]*)/'

this seems unrelated to the problem in the comments. Did you mean to say
something like "xemacs21 or gnu emacs version 20-22"