Le 20/06/2015 15:58, Enrico Forestieri a écrit :
On Sat, Jun 20, 2015 at 08:14:03AM +0100, Guillaume M-M wrote:
Le 20/06/2015 01:11, Enrico Forestieri a écrit :
On Fri, Jun 19, 2015 at 04:50:43PM +0100, Guillaume M-M wrote:
Lastly I see that you prevent loading the microtype package by redefining
\usepackage as you described earlier in the thread. While your redefinition
of \usepackage seems to be very careful, I still think that passing the
option draft to microtype is more reliable in case the user actually uses
macros from microtype (in addition there's no need to test whether the user
loads microtype).
I am concerned with the fact that in dvi mode the microtype package
considerably slows down the processing and prefer to not allow it.
Yes, and the draft option disables the microtype features completely
according to the manual. That's also why I mentioned it. But it's your call.
In this case I will reconsider. Please, let me perform some checks first.
Please, try again with master and report back. The required changes to
address these glitches are straightforward and maybe Richard will allow
backporting them to stable.
I simply applied your patch 98a5072a to stable directly (quicker to
recompile).
You were missing the amendment at b610c13f that solves the issue with
lyx-bug-undefined5.lyx.
Well yes unfortunately b610c13f did not exist when I pulled and
recompiled. But I think I should also test master when I have more time.
lyx-bug-undefined3.lyx: macros inside nested macro definitions (!) are not
taken into account.
I knew you would have been able to spot another corner case :)
This should also be solved at cabc7c4b.
lyx-bug-undefined4.lyx: a combination of two bugs I think. In particular,
our friend \renewcommandx again. More detailed comments in the files.
Here it was going as follows. The script detected a postscript special
in the second preview snippet and thus decided to use the legacy route
through dvips for obtaining the preview instead of directly using dvipng.
In order to do that, the affected snippets are extracted from the original
latex file. Now, in the original file the second snippet was correctly
using \renewcommandx because the macro was already defined in the first
snippet, but now it was incorrect, of course. Also solved at cabc7c4b.
lyx-bug-undefined5.lyx: the arguments are only scanned on the first
occurrence of a macro.
This had already been taken into account at b610c13f.
For your convenience I am attaching the overall set of patches for
the stable branch. Please, let me know if you are able to find some
other remaining glitches :)
Thank you for the convenience.
I have no other issue to report for stable, Enrico wins :)
