On Tue, 7 Jul 2020 at 01:21, Pavel Sanda <sa...@lyx.org> wrote:

> On Tue, Jul 07, 2020 at 12:21:58AM +0200, Thibaut Cuvelier wrote:
> > So, here is the new (final?) version of that patch.
>
>
> Hmm, this layout2layout snippet:
>
> +        if format == 81:
> +            match = re.compile(b'(\\s*Header\\s+)(\\w+)',
> re.IGNORECASE).match(lines[i])
> +            if match:
> +                del lines[i]
> +                continue
>
> Do I get correctly that you want to catch stuff like:
> Header          "PUBLIC &quot;-//OASIS//DTD DocBook V4.3//EN&quot;"
> (which won't work because " is not \w)
> and avoid stuff like
> layouts/foils.layout:# Header definitions:
> layouts/moderncv.layout:          Tooltip       "Header content"
> which will be caught on contrary by the regexp?
>

I tweaked that regex to include optional " and force Header to be at the
beginning of the line. Is this sufficient?

b'^(\\s*Header\\s+)("?\\w+"?)'



> >I also include another
> > patch for AGU, which can be merged at any time, now.
>
> Makefile part is missing :)
>

Corrected!

Attachment: 0013-Remove-AGU-DTD-style-probably-doesn-t-work-impossibl.patch
Description: Binary data

Attachment: 0004-Add-required-elements-in-the-layouts-to-support-DocB.patch
Description: Binary data

-- 
lyx-devel mailing list
lyx-devel@lists.lyx.org
http://lists.lyx.org/mailman/listinfo/lyx-devel

Reply via email to