DocBook v2

[Thibaut Cuvelier]

On Fri, 11 Sep 2020 at 06:14, Scott Kostyshak <skostysh at lyx.org> wrote:

> On Fri, Sep 11, 2020 at 01:58:42AM +0200, Thibaut Cuvelier wrote:
> > On Thu, 3 Sep 2020 at 20:02, Scott Kostyshak <skostysh at lyx.org> wrote:
> >
> > > Also, I  see warnings when exporting to docbook5:
> > >
> > > $ lyx -e docbook5 EmbeddedObjects.lyx
> > > xml.cpp (177): Tried to close `emphasis' when no tags were open!
> > > xml.cpp (177): Tried to close `emphasis' when no tags were open!
> > > xml.cpp (177): Tried to close `emphasis' when no tags were open!
> > >
> > > But as Riki mentioned, these documents are so complex that perhaps some
> > > warnings like the above are expected in some cases.
> > >
> >
> > That one was tougher than expected (especially to pinpoint the right area
> > of the file that caused troubles…), but it should be fixed with
> d2ef4a2012.
> > At least, on my computer, EmbeddedObjects.lyx only generates warnings
> when
> > exporting images (which it should not, normally), nothing related to
> > emphasis.
>
> Thanks, the crash from xhtml export seems gone.
>
> There seems to be a problem configure. I now get:
>
>   The layout file:
>   article
>   could not be found. A default textclass with default
>   layouts will be used. LyX will not be able to produce
>   correct output.
>
> and the terminal shows the following:
>
> checking LaTeX configuration...         auto
> Failed to find valid \Declare line for layout file
> `/home/scott/lyxbuilds/docbook/repo/lib/layouts/acmsiggraph.layout'.
>         => Skipping this file!
> Failed to find valid \Declare line for layout file
> `/home/scott/lyxbuilds/docbook/repo/lib/layouts/tufte-handout.layout'.
>         => Skipping this file!
> ...
>
> I made a commit that fixes things for me. I'm not sure if it's correct.
> could you check 4ac929f3?


I've made a better fix: actually, this was just a typo :). All the relevant
commits have been squashed into 0a7c29c9b567518d26a378759846e892b33d2925.

79712827c9bb4534c4649d3dc67c35ad64bc3b6e simplifies the code further (it's
basically an amendment to 0a7c29c9b567518d26a378759846e892b33d2925). I hope
it does not create new problems for you!

As expected, though, the old DocBook layouts produce warnings like this:

Layout file D:/LyX/lyx-unstable/lib\layouts\docbook-book.layout has no
\DeclareLaTeXClass line.
Layout file D:/LyX/lyx-unstable/lib\layouts\docbook-chapter.layout has no
\DeclareLaTeXClass line.
Layout file D:/LyX/lyx-unstable/lib\layouts\docbook-section.layout has no
\DeclareLaTeXClass line.

In my opinion, it's not really a problem, as the \DeclareDocBookClass has
no more effect: if a user had such a layout, they will get a warning about
it, which is nice.

By the way, what would you prefer I do in this
> case where I'm not sure the fix is correct? Since we're on a branch, I
> just committed so that way the tests work for me and Kornel (who I
> imagine is also affected by this configure issue). But if you prefer I
> post the patch to the list instead of pushing, that's fine too. In my
> opinion, since it's your branch you should choose the workflow.
>

 I have no problem with you pushing commits onto that branch. Just make
sure to send the hash on the list, in case I make one force push too many
:).
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.lyx.org/pipermail/lyx-devel/attachments/20200911/671145f1/attachment.html>