Re: [AUCTeX-devel] [PATCH v2]: Merge /preview/ into top-level

[Matthew Leach]

Hi Mosè,

Thanks for the review.

Mosè Giordano <address@hidden> writes:
> Hi Matthew,
>
> there are still some references to files in "preview/" directory
> inside Makefile.in, rules "xemacs-package" 

I've just tried to run:

$ make xemacs-package TAG=11.88

and failed with:

--8<---------------cut here---------------start------------->8---
Compiling /home/matthew/Development/auctex/xemacs-build/tex-jp.el...
While compiling toplevel forms in file 
/home/matthew/Development/auctex/xemacs-build/tex-jp.el:
  !! Symbol's value as variable is void ((hack-one-local-variable))
>>Error occurred processing tex-jp.el: Symbol's value as variable is void: 
>>hack-one-local-variable

Done
Makefile:158: recipe for target 'lisp' failed
make[1]: *** [lisp] Error 1
--8<---------------cut here---------------end--------------->8---

I get the same error with and without my patches applied.  Did you have
any problems making an xemacs package for the 11.88 release?

> and "preview-ball" fail due to this, and the fix isn't as easy as
> stripping "preview/" away.  In addition, in "EXCLUDEDFILES" all
> occurrences of "preview/" should be removed, right?

Ah, yes.  Sorry I missed those.  I should have done the entire release
process to test everything but I missed the instructions in makefile.in.

I have everything working now besides the xemacs-package target.
-- 
Matt