On 2016-03-20, Kornel Benko wrote:
> Am  19. März 2016 um 23:12:32, schrieb Guenter Milde <mi...@users.sf.net>

Dear Kornel,

thanks for the patch.

>> I still suggest that labels for "mathmacros", "examples",
>> "templates", "manuals" are not required and without them the test system
>> would become easier to understand and use.

> Like the attached?

This is a step in the right direction: "location labels" dont interfere with
other labels in the test names and don't slip inbetween label and sublabel.

There are, however still problems:

#> ctest --print-labels
Test project /usr/local/src/lyxtest
All Labels:
  -1autotests:export
  -1autotests:export:inverted:multilingual
  -1autotests:export:inverted:todo
...

The -1 should not become part of the label.

Actually, IMV the location should not become part of a label:

A user trying to find out which labels exist is overwhelmed by the long list.
And there are so many duplications: 

For example, there are 12 different labels with "todo":

  autotests:export:inverted:todo
  autotests:export:load:lyx2lyx:inverted:todo
  autotests:suspended:todo
  examples:export:inverted:todo
  examples:suspended:todo
  manuals:export:inverted:todo
  mathmacros:export:inverted:todo
  mathmacros:export:load:lyx2lyx:inverted:todo
  mathmacros:suspended:todo
  templates:export:inverted:todo
  templates:suspended:todo
  load:inverted:todo

where 2 would do:

  inverted:todo
  suspended:todo

Suggestion:

a) use a separate label for the location, or
b) don't crate "location labels" at all.

I prefer b), because it is much simpler.


Günter

Reply via email to