Re: pkg/44485: EMACSLOADPATH is not valid for editores/emacs21

["OBATA Akio" <obache%netbsd.org@localhost>]

The following reply was made to PR pkg/44485; it has been noted by GNATS.

From: "OBATA Akio" <obache%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/44485: EMACSLOADPATH is not valid for editores/emacs21
Date: Tue, 01 Feb 2011 17:46:45 +0900

 On Mon, 31 Jan 2011 15:45:02 +0900, David Holland 
<dholland-pbugs%netbsd.org@localhost> wrote:
 
 > The following reply was made to PR pkg/44485; it has been noted by GNATS.
 >
 > From: David Holland <dholland-pbugs%netbsd.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc:
 > Subject: Re: pkg/44485: EMACSLOADPATH is not valid for editores/emacs21
 > Date: Mon, 31 Jan 2011 06:42:28 +0000
 >
 >  On Sat, Jan 29, 2011 at 05:15:00PM +0000, ryoon%NetBSD.org@localhost wrote:
 >   > Index: modules.mk
 >   > ===================================================================
 >   > RCS file: /cvsroot/pkgsrc/editors/emacs/modules.mk,v
 >   > retrieving revision 1.13
 >   > diff -u -r1.13 modules.mk
 >   > --- modules.mk   25 Oct 2010 00:03:03 -0000      1.13
 >   > +++ modules.mk   29 Jan 2011 17:11:49 -0000
 >   > @@ -348,7 +348,7 @@
 >   >          ${SED} -e 's/emacs-//' |                                        
 > \
 >   >          ${SED} -e 's/\.[0-9]\{8,\}//' |                                 
 > \
 >   >          ${SED} -e 's/nb.*//'
 >   > -ALL_ENV+=       
 > EMACSLOADPATH=${_EMACS_DIR}/${_EMACS_VERSION_DIR}/lisp:${_EMACS_DIR}/site-lisp
 >   > +ALL_ENV+=       
 > EMACSLOADPATH=${_EMACS_DIR}/${_EMACS_VERSION_MAJOR}.${_EMACS_VERSION_MINOR}/lisp:${_EMACS_DIR}/site-lisp
 >   >  .include        "${_EMACS_PKGDIR}/buildlink3.mk"
 >   >  .endif
 > Looks ok to me...
 
 If so, _EMACS_VERSION_DIR will no need to be defined dynamically,
 then, it should be gone away, or defined statically instead.
 If not, SED patterns for _EMACS_VERSION_DIR should be added to
 strip the tail of 'a' for emacs21.
 
 -- 
 OBATA Akio / obache%NetBSD.org@localhost