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

[Mosè Giordano]

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 ;-)

Bye,
Mosè