Am Sonntag, 20. März 2016 um 15:10:13, schrieb Guenter Milde 
<mi...@users.sf.net>
> 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.

Yes, I know. I have a patch here already.

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

Hm.

> 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

You have a point.

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

What to do if you want to check test with 'todo' _and_ 'mathmacros'?
ATM you can select
        'ctest -L "mathmacros.*todo"'
That was also a reason I (or we?) created combined labels.

I cite you (your mail from 24.11.2015, subject: Re: My current test failures)
.............................................................................................................
> > What about
> > 
> >       'Sublabel: nonstandard'
> > 
> > ?
> 
> I don't know how you want to implemement it, but if the output contains then
> lines like
> 
>  UNRELIABLE.NON_STANDARD.export/templates/ectaart_dvi3_systemF
>  
>  UNRELIABLE.ERRATIC.export/examples/seminar_dvi
> 
> or
> 
>  NON_STANDARD.UNRELIABLE.export/templates/ectaart_dvi3_systemF
>  
>  ERRATIC.UNRELIABLE.export/examples/seminar_dvi
> 
> this is fine.
> 
> Günter
............................................................................................................

>
> I prefer b), because it is much simpler.
> 


> Günter

        Kornel

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

Reply via email to