Re: [Bug-AUCTeX] auctex build fixes for xemacs

[Ralf Angeli]

Please keep the mailing list copied.  I won't cut anything from the
quoted message so that others can see what you wrote.

* Clemens Fruhwirth (2006-01-27) writes:

> Ralf Angeli <address@hidden> wrote:
>
>> > -       (let (reldir (dirs (append late-packages last-packages 
>> > early-packages)))
>> > +       (let (reldir (dirs (append late-package-hierarchies 
>> > last-package-hierarchies early-package-hierarchies)))
>> 
>> Are *-packages and *-package-hierarchies equivalent?  If they are, do
>> you know why XEmacs does not provide aliases for those variables?
>
> It seems so. From lisp/ChangeLog:
>
>         * update-elc.el (early-package-hierarchies)
>         (late-package-hierarchies)
>         (last-package-hierarchies): Renamed these from `early-packages',
>         `late-packages' and `last-packages'.
>
> Why they don't provide an alias? Maybe cause they didn't intend the
> variable to be used from "external" sources.

Hm, I've just asked on the xemacs-beta list.  Let's hear what they
have to say.

>> > The second one isn't xemacs specific and shouldn't cause any
>> > problems. 
>> 
>> Copied here from the patch:
>> 
>> >           for i in $(PACKAGE_INFO); do \
>> > -           test -r "info/$$i.info" && echo "info/$$i.info" >> 
>> > "pkginfo/MANIFEST.$(PACKAGE)" ; \
>> > +           test -r "info/$$i.info" && echo "info/$$i.info" >> 
>> > "pkginfo/MANIFEST.$(PACKAGE)" ; true; \
>> >           done && \
>> 
>> Why do you require this change?
>
> I just cut&pasted the commands executed by make install-metadata. 
>
> ghanima:/tmp/auctex-11.82# set -x
> + set -x
> ghanima:/tmp/auctex-11.82#  { \
>>   ./mkinstalldirs /usr/local/lib/xemacs/xemacs-packages/pkginfo || : ; \
>>   /usr/local/bin/xemacs -batch -q -no-site-file -no-init-file -l lpath.el -l 
>> preview/prv-install -f preview-make-package 
>> /usr/local/lib/xemacs/xemacs-packages "auctex" \"`echo 2005-12-17|sed 
>> 's/\(.*\)-\(.*\)-\(.*\)/\1.\2\3/'`\" \"11.82\" tex-site.el && \
>>   cd /usr/local/lib/xemacs/xemacs-packages && \
>>   find "etc/auctex" "lisp/auctex" -print >"pkginfo/MANIFEST.auctex" && \
>>   for i in auctex preview-latex; do \
>>     test -r "info/$i.info" && echo "info/$i.info" >> 
>> "pkginfo/MANIFEST.auctex"; \
>>   done && \
>>   echo "pkginfo/MANIFEST.auctex" >> "pkginfo/MANIFEST.auctex" ; \
>> }
> + ./mkinstalldirs /usr/local/lib/xemacs/xemacs-packages/pkginfo
> ++ sed 's/\(.*\)-\(.*\)-\(.*\)/\1.\2\3/'
> ++ echo 2005-12-17
> + /usr/local/bin/xemacs -batch -q -no-site-file -no-init-file -l lpath.el -l 
> preview/prv-install -f preview-make-package 
> /usr/local/lib/xemacs/xemacs-packages auctex '"2005.1217"' '"11.82"' 
> tex-site.el
>
> Processing /usr/local/lib/xemacs/xemacs-packages/lisp/auctex/
>
> [cutted]
> + cd /usr/local/lib/xemacs/xemacs-packages
> + find etc/auctex lisp/auctex -print
> + for i in auctex preview-latex
> + test -r info/auctex.info
> + echo info/auctex.info
> + for i in auctex preview-latex
> + test -r info/preview-latex.info
> ghanima:/usr/local/lib/xemacs/xemacs-packages# echo $?
> + echo 1
> 1
> ghanima:/usr/local/lib/xemacs/xemacs-packages# 
>
> This is because the return code of "test" becomes the return code of
> "for". My patch was of course to hackish. Better do:
>
> if test -r ...; then
>   echo ... > pkginfo;
> fi
>
> "if" does not propagate the value of its conditional, in contrast to the
> "conditional && then" construction.
>
> ghanima:/usr/local/lib/xemacs/xemacs-packages# if test -r 
> info/preview-latex.info; then echo asdss; fi
> + test -r info/preview-latex.info
> ghanima:/usr/local/lib/xemacs/xemacs-packages# echo $?
> + echo 0
> 0
> ghanima:/usr/local/lib/xemacs/xemacs-packages# test -r 
> info/preview-latex.info && echo x
> + test -r info/preview-latex.info
> ghanima:/usr/local/lib/xemacs/xemacs-packages# echo $?
> + echo 1
> 1
> ghanima:/usr/local/lib/xemacs/xemacs-packages# 

David can probably say more about why && is used instead of if..then
in the case at hand.

-- 
Ralf