Am Donnerstag, 17. Januar 2019 09:15:03 CET schrieb Kornel Benko 
<kor...@lyx.org>:
> Am Mittwoch, 16. Januar 2019 14:20:37 CET schrieb Scott Kostyshak 
> <skost...@lyx.org>:
> > On Wed, Jan 16, 2019 at 08:10:39PM +0100, Kornel Benko wrote:
> > > Am Mittwoch, 16. Januar 2019 13:58:41 CET schrieb Scott Kostyshak 
> > > <skost...@lyx.org>:
> > > > On Wed, Jan 16, 2019 at 07:55:43PM +0100, Kornel Benko wrote:
> > > > 
> > > > > I don't remember why we do not test for lyx20x.
> > > > 
> > > > It is because of the \origin tag (which was introduced later). I think
> > > > the test scripts rely on this tag existing for dealing with the
> > > > dependencies when copying to the temporary directory.
> > > > 
> > > > Scott
> > > > 
> > > 
> > > Hm, the 'script' ExportTests.cmake:335 has the line
> > >   foreach(_lyx_format_num 16 21 22 23)
> > > 
> > > This is independent of origin or not.
> > > What I meant is that we do not test for 20x versions.
> > > (I did not mean the compilation of the converted lyx file)
> > 
> > Ah now I see. I don't remember the reason either.
> 
> The first entry for lyx21-format is done at e3cd6db4a78. No special reason
> to exclude lyx20. I think, we could add this value now.
> 
> > Probably we wanted to
> > reduce the time of running all of the tests.
> 
> Looks like we did not take this format as important.
> 
> > We should at some point
> > come up with a guideline. For example, we could test validity of .lyx
> > files and convergence for the previous 4 version file formats, and test
> > (in addition to validity and convergence) compilation of only the
> > previous 2. Or something like that. I don't have any strong opinion on
> > this. I guess we have to think about what is the best way to catch
> > important regressions.
> 
> +1
> 
> > Scott
> > 
> 

In fact, adding lyx20 to the tests leads to:
The following tests FAILED:
        3562 - export/examples/beamer_lyx20 (Failed)
        3571 - export/examples/beamerlyxexample1_lyx20 (Failed)
        3847 - export/examples/localization_test_lyx20 (Failed)
        3891 - export/examples/modernCV_lyx20 (Failed)
        4891 - export/examples/de/beamer_lyx20 (Failed)
        5170 - export/examples/es/modernCV_lyx20 (Failed)
        5431 - export/examples/fr/beamer_lyx20 (Failed)
        5845 - export/examples/ja/beamer_lyx20 (Failed)
        6650 - export/templates/attic/latex8_lyx20 (Failed)
        6673 - export/templates/beamer-conference-ornate-20min_lyx20 (Failed)
        6710 - export/templates/de_beamer-conference-ornate-20min_lyx20 (Failed)
        6800 - export/templates/es_beamer-conference-ornate-20min_lyx20 (Failed)
        6818 - export/templates/fr_beamer-conference-ornate-20min_lyx20 (Failed)
        6890 - export/templates/ja_beamer-conference-ornate-20min_lyx20 (Failed)
        6977 - export/templates/maa-monthly_lyx20 (Failed)
        7175 - export/templates/revtex4-1_lyx20 (Failed)

This is not so bad. Committing now (at 5435b900)

        Kornel

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

Reply via email to