[debbugs-tracker] bug#11806: closed ((setq load-path ..) of elisp-comp)

[GNU bug Tracking System]

Your message dated Fri, 06 Jul 2012 21:39:15 +0200
with message-id <address@hidden>
and subject line Re: bug#11806: (setq load-path ..) of elisp-comp
has caused the debbugs.gnu.org bug report #11806,
regarding (setq load-path ..) of elisp-comp
to be marked as done.

(If you believe you have received this mail in error, please contact
address@hidden)


-- 
11806: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=11806
GNU Bug Tracking System
Contact address@hidden with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> (setq load-path ..) of elisp-comp
>
>
>
>
> Date: 
>
> Thu, 28 Jun 2012 13:47:17 +0900
>
>
>
>
> User-agent: 
>
> Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka)	FLIM/1.14.9 (Goj$(D+W(B) APEL/10.8	Emacs/24.0.94	(x86_64--netbsd) MULE/6.0 (花散里)
>
>
>
>
> We have following line in automake-1.12.1/lib/elisp-comp.
>
>     73  mkdir $tempdir
>     74  cp "$@" $tempdir
>     75  
>     76  (
>     77    cd $tempdir
>     78    echo "(setq load-path (cons nil load-path))" > script
>     79    $EMACS -batch -q -l script -f batch-byte-compile *.el || exit $?
>     80    mv *.elc ..
>     81  ) || exit $?
>     82  
>     83  (exit 0); exit 0
>
>  It seems to me the intention of line 78 is to set load-path
> to add default directory on top of existing load-path.
>
>  This 'script' is OK if the file to compile is only one in the
> directory.
>
>   But if some files are in the directory there and we will
> compile file by file on the same directory, there may be a
> possibility that some files load another file in the same
> directory. In that case this 'script' fails to read such ones.
>
> I do have problem compiling *.el files with tc-2.3.1 (svn version) 
>
> Following patch fixes this type of problem, thanks a lot.
>
> By the way, the same patch was once proposed as
>   http://osdir.com/ml/sysutils.automake.patches/2003-01/msg00004.html
> and fix seems to have made, but real line was dropped with
> unknown reason.
>
> --- lib/elisp-comp.orig 2012-06-01 22:47:10.000000000 +0900
> +++ lib/elisp-comp      2012-06-28 13:28:44.000000000 +0900
> @@ -75,7 +75,7 @@
>  
>  (
>    cd $tempdir
> -  echo "(setq load-path (cons nil load-path))" > script
> +  echo "(setq load-path (cons \"../\" (cons nil load-path)))" > script
>    $EMACS -batch -q -l script -f batch-byte-compile *.el || exit $?
>    mv *.elc ..
>  ) || exit $?
>
>
> Thanks again, 
> ---
> Makoto Fujiwara, 
> Chiba, Japan, Narita Airport and Disneyland prefecture.
>
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#11806: (setq load-path ..) of elisp-comp
>
>
>
>
> Date: 
>
> Fri, 06 Jul 2012 21:39:15 +0200
>
>
>
>
> On 07/06/2012 10:34 AM, Stefano Lattarini wrote:
> >
> > Thanks.  I'll push the patch by this evening or afternoon if nobody
> > else speaks out.
> > 
> Pushed now.  I'm thus closing this bug report.
>
> Regards,
>   Stefano
>
>
> --- End Message ---