[Kst] Re: ready for release?

[Christian Trippe]

Am Donnerstag, 17. März 2011, 08:22:12 schrieb Brisset, Nicolas:
> > It needs this change and another one to build the cfitsio plugin with
> 
> the
> 
> > libcfitsio0 package from openSUSE.
> > 
> > The updated patch is attached. I do not know if it should get
> 
> included.
> I believe the cmake check could be integrated.
> For the other issue (the include path) I guess if we detect include
> paths well in the cmake check we could keep #include <fitsio.h> in the
> code and it will work.

Sorry for not really checking earlier. With the new cmake build the second 
part is no longer needed, only the first part 

Index: cmake/modules/FindCFITSIO.cmake
===================================================================
--- cmake/modules/FindCFITSIO.cmake	(Revision 1223149)
+++ cmake/modules/FindCFITSIO.cmake	(Arbeitskopie)
@@ -10,7 +10,7 @@
 	FIND_PATH(CFITSIO_INCLUDEDIR fitsio.h
 		HINTS
 		ENV CFITSIO_DIR
-		PATH_SUFFIXES include/cfitsio include
+		PATH_SUFFIXES include/libcfitsio0 include
 		PATHS ${kst_3rdparty_dir}
 		)
 	FIND_LIBRARY(CFITSIO_LIBRARIES cfitsio 

is needed as it otherwise does not detect the presence of fitsio.h

Christian