Remove some unneeded libs & headers

Based on some hints by the automatic dependency checking in Debian
packaging[0], it appears that, at least on Debian GNU/Linux, quite a few
of the libs that lugaru links against are not needed.

Remove linking to those libs, and use of related unneeded headers.

* For ogg/vorbis, it seems like libogg is not needed unless handling
  container metadata?

* For opengl, there seems to be only a subset of the libs that are
  required. Notably the glext.h header seems to only be needed on
  windows for GL_GENERATE_MIPMAP, but not otherwise.

* For zlib it seems to be unused, but removing zlib.h exposed a
  dependency on unistd.h which is now included directly on non-windows.

[0]
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libz.so.1 (it uses
none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libSM.so.6 (it
uses none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libX11.so.6 (it
uses none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libogg.so.0 (it
uses none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libXext.so.6 (it
uses none of the library's symbols)
dpkg-shlibdeps: warning: package could avoid a useless dependency if
debian/lugaru/usr/games/lugaru was not linked against libICE.so.6 (it
uses none of the library's symbols)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index ae389fe54eeba7937c02ea9d223adb17a46fd515..30616f4398ca71bd2d59b171e19654558a24971c 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -219,22 +219,19 @@ endif(APPLE)
 
 find_package(PNG REQUIRED)
 find_package(JPEG REQUIRED)
-find_package(ZLIB REQUIRED)
 find_package(OggVorbis REQUIRED)
 
 include_directories(
     ${OPENAL_INCLUDE_DIR}
     ${JPEG_INCLUDE_DIR}
     ${PNG_INCLUDE_DIR}
-    ${ZLIB_INCLUDE_DIR}
     ${OPENGL_INCLUDE_DIR}
     ${SDL2_INCLUDE_DIRS}
     ${VORBISFILE_INCLUDE_DIR}
-    ${OGG_INCLUDE_DIR}
     ${CMAKE_SOURCE_DIR}/Source
 )
 
-set(LUGARU_LIBS ${OPENAL_LIBRARY} ${PNG_LIBRARY} ${JPEG_LIBRARY} ${ZLIB_LIBRARIES} ${SDL2_LIBRARIES} ${OPENGL_LIBRARIES} ${VORBISFILE_LIBRARY} ${OGG_LIBRARY} ${PLATFORM_LIBS})
+set(LUGARU_LIBS ${OPENAL_LIBRARY} ${PNG_LIBRARY} ${JPEG_LIBRARY} ${SDL2_LIBRARIES} ${OPENGL_gl_LIBRARY} ${OPENGL_glu_LIBRARY} ${VORBISFILE_LIBRARY} ${PLATFORM_LIBS})
 
 
 ### Definitions

diff --git a/Source/Audio/openal_wrapper.hpp b/Source/Audio/openal_wrapper.hpp
index 379949fd6e0d3f88c1cbe3006b05d4e64aa7c40b..99c4c2fa7f6c8d250fe8a128fc07bb48207c6235 100644 (file)
--- a/Source/Audio/openal_wrapper.hpp
+++ b/Source/Audio/openal_wrapper.hpp
@@ -32,7 +32,6 @@ along with Lugaru.  If not, see <http://www.gnu.org/licenses/>.
 #include <AL/alc.h>
 #endif
 
-#include <ogg/ogg.h>
 #include <vorbis/vorbisfile.h>
 
 #ifdef _WIN32

diff --git a/Source/Utils/ImageIO.cpp b/Source/Utils/ImageIO.cpp
index c19a63e5f6e65878b15b95e54477787a11196c26..689224ea7888a25add17bfe6c56749e48552807b 100644 (file)
--- a/Source/Utils/ImageIO.cpp
+++ b/Source/Utils/ImageIO.cpp
@@ -26,7 +26,10 @@ along with Lugaru.  If not, see <http://www.gnu.org/licenses/>.
 #include <jpeglib.h>
 #include <png.h>
 #include <stdio.h>
-#include <zlib.h>
+
+#ifndef WIN32
+#include <unistd.h>
+#endif
 
 /* These two are needed for screenshot */
 extern int kContextWidth;

diff --git a/Source/main.cpp b/Source/main.cpp
index 3593c691c82b7588f501a6f3e63235fea9784746..e0270b66dcf27219ba0bf63a8e9573107ee4a31f 100644 (file)
--- a/Source/main.cpp
+++ b/Source/main.cpp
@@ -32,13 +32,14 @@ along with Lugaru.  If not, see <http://www.gnu.org/licenses/>.
 #include <set>
 #include <stdio.h>
 #include <string.h>
-#include <zlib.h>
 
 using namespace Game;
 
 #ifdef WIN32
 #include <shellapi.h>
 #include <windows.h>
+#else
+#include <unistd.h>
 #endif
 
 extern float multiplier;