[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Mosè Giordano
Subject: Re: [AUCTeX-devel] [PATCH v2]: Merge /preview/ into top-level
Date: Sun, 16 Nov 2014 11:18:56 +0100

Hi Stefan,

2014-11-16 4:38 GMT+01:00 Stefan Monnier <address@hidden>:
>>> I'm not against this change, I'd only like to know if we can avoid to
>>> move all files around.  For ELPA, I understand the problem is that
>>> preview.el isn't in the `load-path', we could add "preview/" (actually
>>> (expand-file-name "preview" TeX-lisp-directory) ) to the `load-path'
>>> in tex-site.el (is it ok?), but then we'd have to adapt our install
>>> tree accordingly.
>> I'm not exactly sure if having a preview/ subdir is only a load-path
>> issue.
> ELPA packages are generally expected to put their .el files at the
> top-level.  This affects various things, such as the load-path, the
> autoloads, and more.  You can hack you way around most of those problems
> one way or another, but it's just simpler to move preview to the
> top-level and forget about the whole issue.

I've tried to move all files in ELPA to a "preview" subdirectory and
add it to `load-path' and indeed it works, but I wasn't able to test
whether autoloads would have been generated properly (likely not?).

Then, ok, moving preview to top level is probably the best solution,
but, Matthew, your patch has to be improved ;-)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]