tag 694676 +patch
thanks

On Sat, Dec 01, 2012 at 06:13:25PM +0100, Agustin Martin wrote:
> Loading /usr/share/emacs22/site-lisp/speechd-el/speechd-braille.el (source)...
> Cannot open load file: eieio
> make: *** [braille.elc] Error 255
> ERROR: install script from speechd-el package failed
> 
> diffferent than in original bug report, but reproducible here. I am
> afraid that the simple solution is to make speechd-el not build for
> emacs22.

I am attaching a patch that may help here with speechd-el. For emacs21 and
emacs22 it checks if a standalone eieio package is installed and otherwise
warns and skips byte-compilation for that flavour.

On Sat, Dec 01, 2012 at 08:50:59PM +0100, Andreas Beckmann wrote:
> PS: if you upload packages that are supposed to fix these issues
> somewhere I can run them through my piuparts tests before they get
> uploaded to the archive

Thanks,

I have put an experimental package with that change at

http://people.debian.org/~agmartin/debian-store/experimental/

or if you prefer in repo way, at

deb http://people.debian.org/~agmartin/debian-store/experimental/ ./

Repo is signed with my key. 

Although the experimental package is created as a NMU, I'd prefer a regular
upload by speechd-el maintainer (Hi, Milan). I will only consider uploading
it if there is no reply in a reasonable time period.

Regards,

-- 
Agustin
diff --git a/debian/speechd-el.emacsen-install b/debian/speechd-el.emacsen-install
index fb7dfd7..d940e07 100755
--- a/debian/speechd-el.emacsen-install
+++ b/debian/speechd-el.emacsen-install
@@ -13,6 +13,23 @@ ELC_DIR=/usr/share/${FLAVOR}/site-lisp/$PACKAGE
 if [ "${FLAVOR}" = emacs23 ] || [ "${FLAVOR}" = emacs22 ] || [ "${FLAVOR}" = emacs21 ] || [ "${FLAVOR}" = emacs-snapshot ]; then
   echo "install/$PACKAGE: byte-compiling for ${FLAVOR}..."
 
+  # eieio is integrated in emacs23, conflicting with standalone eieio
+  # package. This makes impossible to use this package with
+  # e.g. emacs22 and a standalone eieio package if emacs23 is
+  # installed (#694676). Check if eieio sources are available when
+  # byte-compiling for emacs21 or emacs22. This should help use of this
+  # package in ancient boxes.
+  case "${FLAVOR}" in
+      emacs21|emacs22)
+	  if [ ! -d "/usr/share/emacs/site-lisp/eieio" ]; then
+	      echo "install/$PACKAGE: No standalone eieio package available for ${FLAVOR}. Skipping ..."
+	      exit 0
+	  fi
+	  ;;
+      *)
+
+  esac
+
   # Copy the temp .el files
   if [ ! -d ${ELC_DIR} ]; then
     mkdir ${ELC_DIR}

Reply via email to