[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: kdelibs (at least) : test building by default
From:       "Andreas Hartmetz" <ahartmetz () gmail ! com>
Date:       2007-10-06 9:11:35
Message-ID: f3642e6b0710060211p16bad6en721b434117d20946 () mail ! gmail ! com
[Download RAW message or body]

Well, tests not running without X are a a big problem if you want to
run them by default. Everybody who has a separate user for development
can tell you that. Otherwise I'd be all for running the tests by
default.

2007/10/6, Rafael Fernández López <ereslibre@kde.org>:
> Hi all,
>
> On Saturday 06 October 2007 10:40:50 Andreas Hartmetz wrote:
> > That is *not* the point. Some people do build tests by default and for
> > these people, a unit test that doesn't compile means a module that
> > doesn't compile. Developers that don't build unit tests quite often
> > introduce compile breakers  for those that do build tests, and that
> > has to stop.
> > Technically it might be possible to make (c)make ignore errors
> > building the test as nothing depends on them. If you think through
> > what that would lead to you should see that that is also a bad idea.
> > Basically, those who create problems would not be those who suffer
> > from them, we'd have even more confusion and so on.
> > So, sane option: Always build unit tests and fail if they don't build.
>
> But even that is not the point... as Thomas has said on a message before the
> important thing is that tests should be ran for keeping up the quality of
> the
> final product (is not a "for making them compile" only discussion).
>
> Getting the case to the extreme is like if you were saying that you don't
> want
> kdelibs to compile because it takes too many time, and you want directly to
> go to kdelibs... some kind of :) and you will say, yeah but kdelibs is
> necessary !! it is as tests are.
>
>
> Bye,
> Rafael Fernández López.
>

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic