[Insight-developers] Re: WrapITK warning message

[Gaëtan Lehmann]

Hi Luis,

Selon Luis Ibanez <luis.ibanez at kitware.com>:

>
>
> Hi Gaetan,
>
>
> Please feel free to modify the message for one that you find more
> appropriate.
>
>
> Please note that anecdotic reports that the system "has been build"
> in platform X by person Y are not satisfactory enough for our quality
> control standards.
>
>
> The definition of a "system being supported" in ITK is the following:
>
>
>          "  There is a green submission from
>            that system to the Dashboard today.  "
>
>
> We can only guarantee that we support a particular platform if we
> can show that there is a green Dashboard submission from that
> specific platform.
>
>
> Quality control, just as science, is all about *Reproducibility*.
>

That's also true for the negative cases: that's not because some build are
failing on windows on the dashboard, that a program "is known not to work on
Windows platforms".
And I'm submitting the testing result with mac os from time to time - that's a
laptop, so it's difficult to do it daily. All the tests are green for the 3
languages.

>
> So, please, respecting that definition, modify the message to some
> less dissuasive text.
>
>
> Please do so soon.
> We will be closing the CVS repository in a couple of hours.
>

I away until next tuesday, so I can't do that, and I suppose it's too late now.
I'll try to live with that until the next release :-)

Gaetan


>
>
>     Thanks
>
>
>        Luis
>
>
>
> =====================
> Gaetan Lehmann wrote:
> >
> > Hi Luis,
> >
> > The message for WrapITK:
> >
> >   MESSAGE("WrapITK is an experimental system for wrapping ITK. It has
> > been  tested only on Linux and it is known not to work on Windows
> > platforms.")
> >
> > is not inexact: I'm also running the tests on macos X on my side, and
> > Zach  has entirely developped WrapITK on that system. Also, several
> > users has  reported using it on several systems, including solaris and
> > ... windows.  Charl P. Botha is using it on that OS, with VS.NET 7.1 -
> > so bad nobody  fixed the windows build on the dashboard before the
> > release, at least for  that compiler.
> >
> > By having WrapITK in ITK, I hope there will be more users who want to
> > test  it, and hopefully report the problems. With this message, and the
> > message  about the Review directory, I'm afraid most of the potential
> > users will be  discouraged to try it.
> >
> > Can you change it for a less dissuasive message ?
> >
> > Thanks,
> >
> > Gaetan
> >
> >
> >
> >
>