[Insight-developers] Sun-CC changes heads up

[Luis Ibanez]

Gaetan,

That will be great.

Thanks for the offer.


      Luis

---------------------------------------------------------------
2009/4/8 Gaëtan Lehmann <gaetan.lehmann at jouy.inra.fr>:
>
> Hi Luis,
>
> We should have 2 new 3 GHz 8-core servers under Solaris 10 next month.
> I was thinking to run the daily tests on one of them, but if needed, it
> shouldn't be a problem to set up a continuous build instead.
>
> Regards,
>
> Gaëtan
>
>
> Le 7 avr. 09 à 23:45, Luis Ibanez a écrit :
>
>> Bill,
>>
>> Agree,...
>>
>> unfortunately... this Sun machine is as slow as Glaciers used to be
>> before Global Warming started.  :-/
>>
>> It takes four hours to build ITK, and two hours to run the testing suite.
>> If we setup a continuous build we will only see one or two submission
>> during the day...  :-/
>>
>> It will be nice to run it as a virtual machine in some more modern
>> platform though...
>>
>> From a quick Google search, it seems that this may be possible...
>> http://www.vmware.com/appliances/directory/1246
>>
>>
>>   Luis
>>
>> --------------------------------------------------------------------
>> On Tue, Apr 7, 2009 at 4:35 PM, Bill Lorensen <bill.lorensen at gmail.com>
>> wrote:
>>>
>>> Luis,
>>>
>>> Maybe we need a continuous build on this platform.
>>>
>>> Bill
>>>
>>> On Tue, Apr 7, 2009 at 4:28 PM, Luis Ibanez <luis.ibanez at kitware.com>
>>> wrote:
>>>>
>>>> Bill, Tom,
>>>>
>>>>
>>>> The "Sun CC compiler + -stlport4 option" Saga seems
>>>> to be finally over.
>>>>
>>>>
>>>> After retouching several hundreds of files, it seems
>>>> that the build in dashsun1 is green again (as far as
>>>> compilation errors go, note that we still have a bunch
>>>> of warnings to deal with...)
>>>>
>>>>
>>>> As Bill pointed out the major change is from now on,
>>>> we should use the vcl_ prefix for (almost) all math
>>>> functions (cos, sin, atan, pow, ceil, floor,...).
>>>>
>>>>
>>>>  Luis
>>>>
>>>>
>>>>
>>>> -----------------------
>>>> Bill Lorensen wrote:
>>>>>
>>>>> Consistent use of the vcl functions versus the native functions is
>>>>> probably a good side effect of this process.
>>>>>
>>>>> On Sun, Apr 5, 2009 at 10:34 PM, Luis Ibanez <luis.ibanez at kitware.com>
>>>>> wrote:
>>>>>
>>>>>> Hi Bill,
>>>>>>
>>>>>> You are right, I should be adding the <string.h> header
>>>>>> instead of replacing the existing <string>.
>>>>>>
>>>>>> What started as a quick fix is turning out to be a major
>>>>>> revisiting of all the files with string and math functions.
>>>>>>
>>>>>> I have been replacing:
>>>>>>
>>>>>>  sin, cos, atan, acos, asin, pow, exp, log, sqrt
>>>>>>
>>>>>> with their vcl_ versions: vcl_sin, vcl_cos... etc.
>>>>>>
>>>>>> and adding header for the functions
>>>>>>
>>>>>> atoi
>>>>>> atof
>>>>>> memset
>>>>>> memcpy
>>>>>>
>>>>>>
>>>>>> I have been doing this by compiling simultaneously
>>>>>> in Linux with gcc 4.2 and with CC under Sun.
>>>>>>
>>>>>> We still have to see how this goes with our usual friends:
>>>>>> Borland and Visual Studio 6.0.
>>>>>>
>>>>>> I'll continue with this tomorrow...
>>>>>>
>>>>>>
>>>>>>  Luis
>>>>>>
>>>>>>
>>>>>> ---------------------
>>>>>> Bill Lorensen wrote:
>>>>>>
>>>>>>> Luis,
>>>>>>>
>>>>>>> I noticed that you are replacing <string> with <string.h>. I
>>>>>>> understand why you need to add string.h, but I thought <string> was
>>>>>>> the include file for the stl string class. Shouldn't you just be
>>>>>>> adding <string.h> and not removing <string>. I see that the
>>>>>>> continuous
>>>>>>> compiler seem to accept your changes, so maybe I'm off base here.
>>>>>>>
>>>>>>> Bill
>>>>>>>
>>>>>>>
>>>>>>> On Sun, Apr 5, 2009 at 7:35 AM, Luis Ibanez <luis.ibanez at kitware.com>
>>>>>>> wrote:
>>>>>>>
>>>>>>>
>>>>>>>> Hi Mathieu,
>>>>>>>>
>>>>>>>> Thanks for tracking the problem with file writing in the Sun
>>>>>>>> compiler.
>>>>>>>>
>>>>>>>> Just to give you a heads up: Related to the changes that Tom made
>>>>>>>> in order to support derived ImportImageContainer, we introduced a
>>>>>>>> compiler flag: -stlport4 for the SunCC build, because the
>>>>>>>> std::allocator
>>>>>>>> in this compiler was missing the "destroy" method.
>>>>>>>>
>>>>>>>> This new flag triggered a chain of additional changes (and still
>>>>>>>> more
>>>>>>>> to come). Hopefully they will not interfere with your exploration,
>>>>>>>> but...
>>>>>>>> just in case, if you see some dramatic (good or bad) change, it may
>>>>>>>> be related to these other ongoing modifications.
>>>>>>>>
>>>>>>>>
>>>>>>>>  Luis
>>>>>>>> _______________________________________________
>>>>>>>> Powered by www.kitware.com
>>>>>>>>
>>>>>>>> Visit other Kitware open-source projects at
>>>>>>>> http://www.kitware.com/opensource/opensource.html
>>>>>>>>
>>>>>>>> Please keep messages on-topic and check the ITK FAQ at:
>>>>>>>> http://www.itk.org/Wiki/ITK_FAQ
>>>>>>>>
>>>>>>>> Follow this link to subscribe/unsubscribe:
>>>>>>>> http://www.itk.org/mailman/listinfo/insight-developers
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>
>>>>
>>>
>> _______________________________________________
>> Powered by www.kitware.com
>>
>> Visit other Kitware open-source projects at
>> http://www.kitware.com/opensource/opensource.html
>>
>> Please keep messages on-topic and check the ITK FAQ at:
>> http://www.itk.org/Wiki/ITK_FAQ
>>
>> Follow this link to subscribe/unsubscribe:
>> http://www.itk.org/mailman/listinfo/insight-developers
>
> --
> Gaëtan Lehmann
> Biologie du Développement et de la Reproduction
> INRA de Jouy-en-Josas (France)
> tel: +33 1 34 65 29 66    fax: 01 34 65 29 09
> http://voxel.jouy.inra.fr  http://www.mandriva.org
> http://www.itk.org  http://www.clavier-dvorak.org
>
>