Use more secure mailutils instead of internal movemail
authorRob Browning <rlb@defaultvalue.org>
Sun, 16 Dec 2018 23:11:00 +0000 (17:11 -0600)
committerRob Browning <rlb@defaultvalue.org>
Mon, 17 Dec 2018 19:23:31 +0000 (13:23 -0600)
See the 26.1 NEWS for more information.

debian/control
debian/emacs-bin-common.lintian-overrides [new file with mode: 0644]
debian/emacs-bin-common.lintian-overrides.in [deleted file]
debian/rules

index 6d4a533ea2096c9b556ca14d874480c96c96c7d4..c419686ff09015ac9a7537abf15dc237de8db56c 100644 (file)
@@ -119,6 +119,7 @@ Description: Debugging symbols for emacs-gtk
 Package: emacs-bin-common
 Architecture: any
 Depends: emacs-common (= ${source:Version}), ${shlibs:Depends}, ${misc:Depends}
+ mailutils
 Description: GNU Emacs editor's shared, architecture dependent files
  GNU Emacs is the extensible self-documenting text editor.
  This package contains the architecture dependent infrastructure
diff --git a/debian/emacs-bin-common.lintian-overrides b/debian/emacs-bin-common.lintian-overrides
new file mode 100644 (file)
index 0000000..fc6ca28
--- /dev/null
@@ -0,0 +1,7 @@
+emacs-bin-common binary: binary-without-manpage usr/bin/b2m.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/ctags.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/ebrowse.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/emacsclient.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/etags.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/grep-changelog.emacs
+emacs-bin-common binary: binary-without-manpage usr/bin/rcs-checkin.emacs
diff --git a/debian/emacs-bin-common.lintian-overrides.in b/debian/emacs-bin-common.lintian-overrides.in
deleted file mode 100644 (file)
index ca90881..0000000
+++ /dev/null
@@ -1,8 +0,0 @@
-emacs-bin-common binary: setgid-binary @MOVEMAIL_BIN@ 2755 root/mail
-emacs-bin-common binary: binary-without-manpage usr/bin/b2m.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/ctags.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/ebrowse.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/emacsclient.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/etags.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/grep-changelog.emacs
-emacs-bin-common binary: binary-without-manpage usr/bin/rcs-checkin.emacs
index d16e07dd92912f2fdd043049d245373a92b35daf..8f985b74e04f463345755d3d0100cccbe71395b9 100755 (executable)
@@ -112,9 +112,6 @@ ifeq (,$(joblimit))
   joblimit := 1
 endif
 
-target := $(DEB_HOST_GNU_TYPE)
-movemail_bin := usr/lib/emacs/$(runtime_ver)/$(target)/movemail
-
 # Info files that are going to show up in the main dir.
 main_dir_info_files := efaq.info
 
@@ -126,21 +123,17 @@ nonpersistent_autogen_build_files :=
 persistent_autogen_install_files :=
 
 nonpersistent_autogen_install_files := \
-  debian/emacs-bin-common.lintian-overrides \
   debian/emacs-bin-common.postinst \
   debian/emacs-bin-common.prerm \
   debian/emacs-common.README.Debian \
   debian/emacs-common.postinst \
   debian/emacs-common.prerm \
-  debian/emacs-nox.lintian-overrides \
   debian/emacs-nox.postinst \
   debian/emacs-nox.preinst \
   debian/emacs-nox.prerm \
-  debian/emacs-lucid.lintian-overrides \
   debian/emacs-lucid.postinst \
   debian/emacs-lucid.preinst \
   debian/emacs-lucid.prerm \
-  debian/emacs-gtk.lintian-overrides \
   debian/emacs-gtk.preinst \
   debian/emacs-gtk.postinst \
   debian/emacs-gtk.prerm
@@ -204,7 +197,6 @@ define deb_sub
     -e "s|\@MENU_VERSION\@|$(menu_ver)|go;" \
     -e "s|\@X_SUPPORT\@|$(xsupport)|go;" \
     -e "s|\@BIN_PRIORITY\@|$(bin_priority)|go;" \
-    -e "s|\@MOVEMAIL_BIN\@|$(movemail_bin)|go;" \
       < $(1) > $(2)
 endef
 
@@ -220,7 +212,6 @@ check-vars:
        @echo "runtime_ver: $(runtime_ver)"
        @echo "major_ver: $(major_ver)"
        @echo "minor_ver: $(minor_ver)"
-       @echo "movemail_bin: $(movemail_bin)"
 
 .PHONY: check-autogen
 check-autogen: $(persistent_autogen_files) $(nonpersistent_autogen_files)
@@ -280,6 +271,7 @@ confflags += --with-pop=yes
 confflags += --enable-locallisppath=$(local_lpath)
 confflags += --with-sound=alsa
 confflags += --without-gconf
+confflags += --with-mailutils
 
 # x configure flags
 confflags_gtk := $(confflags)
@@ -313,8 +305,6 @@ define build_cmd
   $(MAKE) -C $(1) -j $(joblimit)
   # If we don't use bootstrap, we need to explicitly build info.
   $(MAKE) -C $(1) -j $(joblimit) info
-  # Make sure we chose liblockfile.
-  ldd $(1)/lib-src/movemail | grep liblockfile
 endef
 
 # Don't mess with the source tree (we build in debian/build-*)
@@ -459,10 +449,6 @@ override_dh_auto_install: $(autogen_install_files)
          test -f $(pkgdir_bin_common)/usr/bin/emacs-*
          rm $(pkgdir_bin_common)/usr/bin/{emacs,emacs-*}
 
-         # Set up movemail.
-         chown root.mail $(pkgdir_bin_common)/$(movemail_bin)
-         chmod g+s $(pkgdir_bin_common)/$(movemail_bin)
-
          # Set up alternatives.
          alternatives=$$(ls $(pkgdir_bin_common)/usr/bin) \
            && set -x \
@@ -574,9 +560,6 @@ override_dh_strip_nondeterminism:
 override_dh_testdir:
        dh_testdir debian/emacsVAR.postinst
 
-override_dh_fixperms:
-       dh_fixperms -X$(movemail_bin)
-
 override_dh_auto_clean: $(persistent_autogen_files)
        true