search

casacore / casacore

Suite of C++ libraries for radio astronomy data processing
http://casacore.github.io/casacore
Other
118 stars 87 forks source link

Compiler errors with g++ 4.7.2 #58

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Hi,

When compiling on ubuntu 12.10, I encountered a few compiler errors. The diff 
below should fix them.

Best regards,
Nico

diff -r ./casa/Arrays/ArrayBase.h ../orig/casacore-1.5.0/casa/Arrays/ArrayBase.h
30a31
> 
34,35d34
< 
< #include <unistd.h>
diff -r ./casa/Arrays/IPosition.h ../orig/casacore-1.5.0/casa/Arrays/IPosition.h
37d36
< #include <unistd.h>
diff -r ./casa/BasicMath/Math.h ../orig/casacore-1.5.0/casa/BasicMath/Math.h
300c300
<   return ( std::isnan(val) );

---
>   return ( isnan(val) );
diff -r ./casa/System/Casarc.cc ../orig/casacore-1.5.0/casa/System/Casarc.cc
37d36
< #include <unistd.h>

Original issue reported on code.google.com by nicokrui...@gmail.com on 24 Apr 2013 at 12:12

GoogleCodeExporter commented 9 years ago 
I can confirm these errors and that the patch is working.

Apparently, there is also an alternative to this patch. New "#include 
<unistd.h> lines can be added to casa/aips.h. and casa/System/Casarc.h 
according to 
http://lofarathamburg.wordpress.com/2013/01/11/things-needed-to-install-lofar-pa
ckage/
I confirm that this other patch also works.

Original comment by nudomari...@gmail.com on 11 May 2013 at 1:29

GoogleCodeExporter commented 9 years ago 
Yes, sure. I guess that casa/aips.h is included in the other files. I tried to 
keep the includes as local as possible. Either solution is fine with me.

Original comment by nicokrui...@gmail.com on 13 May 2013 at 7:41

GoogleCodeExporter commented 9 years ago 
Perhaps it is duplicated to issue 51?

http://code.google.com/p/casacore/issues/detail?id=51

Original comment by hero...@gmail.com on 24 May 2014 at 6:45

GoogleCodeExporter commented 9 years ago 
It is fixed on the trunk. A new release will be made soon by Malte.

Original comment by gervandi...@gmail.com on 26 May 2014 at 5:46

GoogleCodeExporter commented 9 years ago 
casacore-1.7.0 has been released. See the front page of the project.
casacore-2.0.0 will follow soon.

Original comment by Malte.Marquarding on 4 Jun 2014 at 3:12

GoogleCodeExporter commented 9 years ago

Original comment by gervandi...@gmail.com on 18 Nov 2014 at 7:07