Am Mittwoch, 20. Dezember 2017 um 16:06:18, schrieb Jean-Marc Lasgouttes 
<lasgout...@lyx.org>
> Le 20/12/2017 à 10:05, Jürgen Spitzmüller a écrit :
> >> Shall we add a special casing to DocumentClass:addLayoutIfNeeded, or
> >> just fix tex2lyx so that it creates correct files?
> > 
> > I'd suggest the latter.
> 
> tex2lyx is fixed in master at 40a3a44aa2. Of course this makes our 
> tex2lyx tests fail (with the expected issues), but I am not sure how I 
> am supposed to fix that. The test machniery is mysterious to me.
> 
> If nobody complains too much, I'll backport to 2.3.X.
> 
> JMarc

For instance the following command fails:
        #ctest -R tex2lyx/roundtrip/test-insets-basic.tex

Now look into "Testing/Temporary/LastTest.log"
Hier are the differences between created and provided test-files.
In this case

Created: 
/BUILD/BUILDMint17/BuildLyxGitQt5.8self-gcc7.1/src/tex2lyx/test/test-insets-basic.lyx
Provided (e.g. from repo): 
/usr2/src/lyx/lyx-git/src/tex2lyx/test/test-insets-basic.lyx.lyx
@@ -1,4 +1,4 @@
-#LyX file created by tex2lyx 2.3
+#LyX file created by tex2lyx 2.4
 \lyxformat 544
 \begin_document
 \begin_header
@@ -963,7 +963,7 @@

 \begin_layout Standard
 An URL:
-\begin_inset Flex Flex:URL
+\begin_inset Flex URL
 status collapsed

 \begin_layout Plain Layout
@@ -986,7 +986,7 @@

 \begin_layout Standard
 An URL with strange characters:
-\begin_inset Flex Flex:URL
+\begin_inset Flex URL
 status collapsed

 \begin_layout Plain Layout

Checking the diff, we can decide that the created file is OK, so we have to 
correct the repo.

        # cp Created Provided

I've done it locally.

        Kornel

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to