?Glitch in Makefile: Release org-mode 5.10b

?Glitch in Makefile: Release org-mode 5.10b

[Eric J Haywiser]

I think the Makefile in the current release, and other
5. versions, has a glitch.

If a new user were to download org.tgz and expand it in
a directory other than the one which becomes $(lispdir) in
the Makefile (e.g. ~/otherdir), they would, upon typing make,
encounter:

While compiling toplevel forms in file 
~/otherdir/org-5.10b/org-mouse.el:
   !! File error (("Cannot open load file" "org"))
Done
make: *** [org-mouse.elc] Error 1

This small change seems to resolve the problem:

#BATCH=$(EMACS) -batch -q -eval "(add-to-list (quote load-path) \"$(lispdir)\")"
BATCH=$(EMACS) -batch -q -eval "(add-to-list (quote load-path) \"$(cwd)\")"

This could be confusing to a newbie, who is the most likely
person to encounter this problem.

Anyone already using org appears to be compiling against the older org
already in their lispdir. They would not necessarily encounter a problem or 
notice this glitch.

Re: ?Glitch in Makefile: Release org-mode 5.10b

[Steven E. Harris]

Eric J Haywiser <ejh1@MIT.EDU> writes:

> This could be confusing to a newbie, who is the most likely person to
> encounter this problem.

I just ran into this problem while installing org-mode last night, and
came up with a similar fix.

Also, for XEmacs users, the eval form needs to add both the base
directory and the "xemacs" directory in order to pick up noutline.el. I
wound up using something like

  BATCH=$(EMACS) -batch -q -eval \
    "(progn (add-to-list (quote load-path) \"$(pwd)\") \
            (add-to-list (quote load-path) \"$(pwd)/xemacs\"))"

as a quick cut-and-paste fix.

-- 
Steven E. Harris

Re: Re: ?Glitch in Makefile: Release org-mode 5.10b

[Eric J Haywiser]

On Thu, 27 Sep 2007, Steven E. Harris wrote:

> Eric J Haywiser <ejh1@MIT.EDU> writes:
>
>> This could be confusing to a newbie, who is the most likely person to
>> encounter this problem.
>
> I just ran into this problem while installing org-mode last night, and
> came up with a similar fix.
>
>  BATCH=$(EMACS) -batch -q -eval \
>    "(progn (add-to-list (quote load-path) \"$(pwd)\") \
>            (add-to-list (quote load-path) \"$(pwd)/xemacs\"))"
>
For the record:

The appropriate adjective may be "better" not "similar", particularly for a 
newbie.  I think pwd is a more portable, specifically shell agnostic, and 
therefore better choice.