Re: new exporter

[Nicolas Goaziou <n.goaziou@gmail.com>]

Hello,

Achim Gratz <Stromeko@nexgo.de> writes:

>here's my first stab on fixing some of that:
>
> From d7ef1bfbaef873521731697d86112029f02cdc8b Mon Sep 17 00:00:00 2001
> From: Achim Gratz <Stromeko@Stromeko.DE>
> Date: Sat, 2 Jun 2012 18:38:24 +0200
> Subject: [PATCH 5/5] Make byte-compiler more happy
>
> * contrib/lisp/org-e-ascii.el: Remove declarations and replace with
>   require.
>
> * contrib/lisp/org-e-html.el: Remove declarations and replace with
>   require.
>
> * contrib/lisp/org-e-latex.el: Remove declarations and replace with
>   require.
>
> * contrib/lisp/org-e-publish.el: Remove declarations and replace with
>   require.
>
> * contrib/lisp/org-export.el: Remove declarations and replace with
>   require.  Prevent byte-compilation, it currently would produce an
>   error due to circular dependencies in the file.

declarations (instead of requires) are here to avoid circular
dependencies. Why do you want to revert that in the first place?

> The way org-export is structured unfortunately produces circular
> dependencies due to the dispatcher and that prevents it from being
> compiled properly.

I think that adding autoload cookies on export functions and removing
requires in the dispatcher should solve most problems.

> One of test fails (but independently of compiling or not compiling the
> rest of the new exporter).  The failing test is
> test-org-element/src-block-interpreter, it appears that the test
> expects the block to be indented by two spaces, but it starts at the
> beginning of line for me.  Maybe some configuration is missing, but in
> any case that should be provided by the test.

It should be fixed now. Thank you.


Regards,

-- 
Nicolas Goaziou