Package: global
Version: 6.6.2-2+b1
Severity: minor
Tags: patch

Emacs expects the source for byte-compiled files to be in the same
directory. If this is not the case `M-x find-function` is broken and
`M-x describe-function` doesn't display a link to the package source
code.

The `debian/global.emacsen-install` file seems to remove all *.el
files in the aforementioned directory (excerpt): 

    # remove the redundant .el files
    # presumes that any .el files in the <flavor> dir are trash.
    rm ${elc_dir}/*.el

If I remove this line the everything works as expected. I do see how
without this line files might accumulate in the $elc_dir if the set of
*.el files shipped by upstream changes.

Thus I suggest just cleaning up $elc_dir before copying/compiling the
*.el files into it (patch attached).

--Daniel

-- System Information:
Debian Release: buster/sid
  APT prefers testing
  APT policy: (990, 'testing'), (500, 'testing-debug'), (500, 'unstable'), (1, 
'experimental')
Architecture: amd64 (x86_64)
Foreign Architectures: arm64

Kernel: Linux 4.16.0-2-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8), 
LANGUAGE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled

Versions of packages global depends on:
ii  emacsen-common  2.0.8
ii  libc6           2.27-3
ii  libltdl7        2.4.6-2.1
ii  libncurses6     6.1+20180210-4
ii  libtinfo6       6.1+20180210-4

Versions of packages global recommends:
ii  python  2.7.15-3

Versions of packages global suggests:
ii  chromium [www-browser]  67.0.3396.79-2
ii  doxygen                 1.8.13-10
ii  elinks [www-browser]    0.12~pre6-13
ii  exuberant-ctags         1:5.9~svn20110310-12
ii  firefox [www-browser]   60.0.2-1
pn  id-utils                <none>
ii  lynx [www-browser]      2.8.9pre.1-1
ii  python-pygments         2.2.0+dfsg-1
ii  w3m [www-browser]       0.5.3-36+b1

-- no debconf information
>From 6be64a2b67fa862afd65eabb5b3f34e10ce36f83 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Gr=C3=B6ber?= <[email protected]>
Date: Mon, 2 Jul 2018 19:32:58 +0200
Subject: [PATCH] emacsen: Don't remove *.el files from site-lisp dir

Emacs expects the source for byte-compiled files to be in the same
directory. If this is not the case `M-x find-function` is broken and
`M-x describe-function` doesn't display a link to the package source
code.
---
 debian/global.emacsen-install | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/debian/global.emacsen-install b/debian/global.emacsen-install
index 09327b3..c7108d6 100644
--- a/debian/global.emacsen-install
+++ b/debian/global.emacsen-install
@@ -14,16 +14,16 @@ if [ ${FLAVOR} != emacs ]
 then
        echo install/${PACKAGE}: byte-compiling for ${FLAVOR}
 
-       # Copy the temp .el files
        install -m 755 -d ${elc_dir}
+
+        # remove the redundant files
+       rm ${elc_dir}/*.el ${elc_dir}/*.elc
+
+       # Copy the temp .el files
        cp ${el_path_list} ${elc_dir}
 
        # Byte compile them
        ${FLAVOR} ${byte_compile_options} ${elc_path_list}
-
-       # remove the redundant .el files
-       # presumes that any .el files in the <flavor> dir are trash.
-       rm ${elc_dir}/*.el
 fi
 exit 0;
 
-- 
2.17.1

Reply via email to