Bug#915190: xemacs21-support: infinite loop in /etc/xemacs21/site-start.d

2018-12-24 Thread Mark Brown
On Sun, Dec 02, 2018 at 02:20:17PM +0900, Tatsuya Kinoshita wrote:
> reassign 909381 xemacs21-support
> forcemerge 915190 909381
> tags 915190 + patch
> thanks
> 
> See the following patch to fix this bug.

Which I didn't see because the mail only went to the control bot and the
pre-merged bug - it isn't even visble in the web view of the bug unless
you explicitly expand the message :(  It's better to add an explicit CC
to the maintainer to make sure the BTS does the right thing, the
defaults really aren't altogether helpful for reassignments and control
mail.  I'll just apply this just now, thanks...

> 
> ```
> --- a/debian/00debian.el
> +++ b/debian/00debian.el
> @@ -94,8 +94,4 @@
> )
> load-path))
>  
> -;; should now load from one of the /etc dirs since they are first in
> -;; the path now.
> -(load "site-start" t)
> -
>  ;;; end 00debian.el
> ```
> 
> This `(load "site-start" t)` was intended to load `/etc/emacs/site-start.el`,
> but unneeded now (dropped by emacsen-common 3.x).
> 
> cf. https://lists.debian.org/debian-emacsen/2018/06/msg00093.html
> > Date: Sat, 16 Jun 2018 11:04:27 -0500
> > From: Rob Browning 
> > Subject: [PATCH 3/4] Given new policy and emacsXY unversioning drop shared 
> > dirs
> > To: debian-emac...@lists.debian.org
> > Cc: Mark Brown 
> > 
> > ---
> >  debian/00debian.el | 7 +--
> >  1 file changed, 1 insertion(+), 6 deletions(-)
> > 
> > diff --git a/debian/00debian.el b/debian/00debian.el
> > index 87508d5..fd06986 100644
> > --- a/debian/00debian.el
> > +++ b/debian/00debian.el
> > @@ -83,10 +83,6 @@ starting with a '.'"
> > (dir-and-all-good-subs
> >  (expand-file-name "~/.xemacs/packages"))
> > (list (concat "/etc/xemacs" debian-xemacs-major-version))
> > -   '("/etc/emacs")
> > -   (list (concat "/usr/local/share/emacs/xemacs-" debian-xemacs-version
> > - "/site-lisp"))
> > -   '("/usr/local/share/emacs/site-lisp")
> > `(,@(dir-and-all-good-subs "/usr/local/lib/xemacs/site-lisp")
> > ,@(dir-and-all-good-subs
> >(concat "/usr/share/xemacs/site-lisp-"
> > @@ -96,8 +92,7 @@ starting with a '.'"
> >(concat "/usr/share/xemacs" debian-xemacs-major-version
> >"/site-lisp/"))
> > )
> > -   load-path
> > -   '("/usr/share/emacs/site-lisp")))
> > +   load-path))
> >  
> >  ;; should now load from one of the /etc dirs since they are first in
> >  ;; the path now.
> > -- 
> > 2.15.1
> 
> Thanks,
> -- 
> Tatsuya Kinoshita




signature.asc
Description: PGP signature


Bug#915190: xemacs21-support: infinite loop in /etc/xemacs21/site-start.d

2018-12-01 Thread Tatsuya Kinoshita
reassign 909381 xemacs21-support
forcemerge 915190 909381
tags 915190 + patch
thanks

See the following patch to fix this bug.

```
--- a/debian/00debian.el
+++ b/debian/00debian.el
@@ -94,8 +94,4 @@
)
load-path))
 
-;; should now load from one of the /etc dirs since they are first in
-;; the path now.
-(load "site-start" t)
-
 ;;; end 00debian.el
```

This `(load "site-start" t)` was intended to load `/etc/emacs/site-start.el`,
but unneeded now (dropped by emacsen-common 3.x).

cf. https://lists.debian.org/debian-emacsen/2018/06/msg00093.html
> Date: Sat, 16 Jun 2018 11:04:27 -0500
> From: Rob Browning 
> Subject: [PATCH 3/4] Given new policy and emacsXY unversioning drop shared 
> dirs
> To: debian-emac...@lists.debian.org
> Cc: Mark Brown 
> 
> ---
>  debian/00debian.el | 7 +--
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/debian/00debian.el b/debian/00debian.el
> index 87508d5..fd06986 100644
> --- a/debian/00debian.el
> +++ b/debian/00debian.el
> @@ -83,10 +83,6 @@ starting with a '.'"
> (dir-and-all-good-subs
>  (expand-file-name "~/.xemacs/packages"))
> (list (concat "/etc/xemacs" debian-xemacs-major-version))
> -   '("/etc/emacs")
> -   (list (concat "/usr/local/share/emacs/xemacs-" debian-xemacs-version
> - "/site-lisp"))
> -   '("/usr/local/share/emacs/site-lisp")
> `(,@(dir-and-all-good-subs "/usr/local/lib/xemacs/site-lisp")
> ,@(dir-and-all-good-subs
>(concat "/usr/share/xemacs/site-lisp-"
> @@ -96,8 +92,7 @@ starting with a '.'"
>(concat "/usr/share/xemacs" debian-xemacs-major-version
>"/site-lisp/"))
> )
> -   load-path
> -   '("/usr/share/emacs/site-lisp")))
> +   load-path))
>  
>  ;; should now load from one of the /etc dirs since they are first in
>  ;; the path now.
> -- 
> 2.15.1

Thanks,
-- 
Tatsuya Kinoshita


pgp_uYEO_5g9M.pgp
Description: PGP signature