NOISSUE clean up some old cmake messes

23 files changed, 78 insertions, 164 deletions

diff --git a/depends/LogicalGui/CMakeLists.txt b/depends/LogicalGui/CMakeLists.txt
index 9fb03a03..12665738 100644
--- a/depends/LogicalGui/CMakeLists.txt
+++ b/depends/LogicalGui/CMakeLists.txt
@@ -1,7 +1,4 @@
 project(LogicalGui)
-
-# Set the include dir path.
-set(LOGICALGUI_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}" PARENT_SCOPE)
-
 add_library(LogicalGui STATIC LogicalGui.h)
+target_include_directories(LogicalGui PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}")
 qt5_use_modules(LogicalGui Core)
diff --git a/depends/classparser/CMakeLists.txt b/depends/classparser/CMakeLists.txt
index 68ae7b73..a6c3fa14 100644
--- a/depends/classparser/CMakeLists.txt
+++ b/depends/classparser/CMakeLists.txt
@@ -16,9 +16,8 @@ find_package(Qt5Core REQUIRED)
 include_directories(${Qt5Base_INCLUDE_DIRS})
 
 set(CLASSPARSER_HEADERS
-include/classparser_config.h
-
 # Public headers
+include/classparser_config.h
 include/javautils.h
 
 # Private headers
@@ -35,14 +34,8 @@ src/javautils.cpp
 src/annotations.cpp
 )
 
-# Set the include dir path.
-set(LIBGROUPVIEW_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include" PARENT_SCOPE)
-
-# Include self.
-include_directories(${CMAKE_CURRENT_SOURCE_DIR}/include)
-include_directories(${CMAKE_BINARY_DIR}/include)
-
 add_definitions(-DCLASSPARSER_LIBRARY)
 
 add_library(classparser SHARED ${CLASSPARSER_SOURCES} ${CLASSPARSER_HEADERS})
+target_include_directories(classparser PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
 qt5_use_modules(classparser Core)
diff --git a/depends/hoedown/CMakeLists.txt b/depends/hoedown/CMakeLists.txt
index b4ef8ef9..7902e734 100644
--- a/depends/hoedown/CMakeLists.txt
+++ b/depends/hoedown/CMakeLists.txt
@@ -1,14 +1,14 @@
 # hoedown 3.0.2 - https://github.com/hoedown/hoedown/archive/3.0.2.tar.gz
-project(hoedown)
+project(hoedown LANGUAGES C VERSION 3.0.2)
 
 set(HOEDOWN_SOURCES
-hoedown/autolink.h
-hoedown/buffer.h
-hoedown/document.h
-hoedown/escape.h
-hoedown/html.h
-hoedown/stack.h
-hoedown/version.h
+include/hoedown/autolink.h
+include/hoedown/buffer.h
+include/hoedown/document.h
+include/hoedown/escape.h
+include/hoedown/html.h
+include/hoedown/stack.h
+include/hoedown/version.h
 src/autolink.c
 src/buffer.c
 src/document.c
@@ -20,10 +20,7 @@ src/stack.c
 src/version.c
 )
 
-# Set the include dir path.
-set(HOEDOWN_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}" PARENT_SCOPE)
-
 # Include self.
-include_directories(${CMAKE_CURRENT_SOURCE_DIR})
-
 add_library(hoedown STATIC ${HOEDOWN_SOURCES})
+
+target_include_directories(hoedown PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/include)
diff --git a/depends/hoedown/hoedown/autolink.h b/depends/hoedown/include/hoedown/autolink.h
index 528885c9..528885c9 100644
--- a/depends/hoedown/hoedown/autolink.h
+++ b/depends/hoedown/include/hoedown/autolink.h
diff --git a/depends/hoedown/hoedown/buffer.h b/depends/hoedown/include/hoedown/buffer.h
index d7703f8d..d7703f8d 100644
--- a/depends/hoedown/hoedown/buffer.h
+++ b/depends/hoedown/include/hoedown/buffer.h
diff --git a/depends/hoedown/hoedown/document.h b/depends/hoedown/include/hoedown/document.h
index a8178fec..a8178fec 100644
--- a/depends/hoedown/hoedown/document.h
+++ b/depends/hoedown/include/hoedown/document.h
diff --git a/depends/hoedown/hoedown/escape.h b/depends/hoedown/include/hoedown/escape.h
index d7659c27..d7659c27 100644
--- a/depends/hoedown/hoedown/escape.h
+++ b/depends/hoedown/include/hoedown/escape.h
diff --git a/depends/hoedown/hoedown/html.h b/depends/hoedown/include/hoedown/html.h
index e46e7fd6..e46e7fd6 100644
--- a/depends/hoedown/hoedown/html.h
+++ b/depends/hoedown/include/hoedown/html.h
diff --git a/depends/hoedown/hoedown/stack.h b/depends/hoedown/include/hoedown/stack.h
index bf9b439b..bf9b439b 100644
--- a/depends/hoedown/hoedown/stack.h
+++ b/depends/hoedown/include/hoedown/stack.h
diff --git a/depends/hoedown/hoedown/version.h b/depends/hoedown/include/hoedown/version.h
index 4938cae5..4938cae5 100644
--- a/depends/hoedown/hoedown/version.h
+++ b/depends/hoedown/include/hoedown/version.h
diff --git a/depends/iconfix/CMakeLists.txt b/depends/iconfix/CMakeLists.txt
index 53d8c28e..4dfc39a9 100644
--- a/depends/iconfix/CMakeLists.txt
+++ b/depends/iconfix/CMakeLists.txt
@@ -1,12 +1,9 @@
-cmake_minimum_required(VERSION 2.8.11)
+cmake_minimum_required(VERSION 3.1)
 project(iconfix)
 
 find_package(Qt5Core REQUIRED QUIET)
 find_package(Qt5Widgets REQUIRED QUIET)
 
-include_directories(${Qt5Core_INCLUDE_DIRS})
-include_directories(${Qt5Widgets_INCLUDE_DIRS})
-
 set(ICONFIX_SOURCES
 xdgicon.h
 xdgicon.cpp
@@ -15,6 +12,7 @@ internal/qiconloader.cpp
 internal/qiconloader_p.h
 )
 
-set(ICONFIX_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}" PARENT_SCOPE)
 add_library(iconfix STATIC ${ICONFIX_SOURCES})
+target_include_directories(iconfix PUBLIC ${CMAKE_CURRENT_SOURCE_DIR})
+
 qt5_use_modules(iconfix Core Widgets)
diff --git a/depends/javacheck/CMakeLists.txt b/depends/javacheck/CMakeLists.txt
index d4f29ff7..9768650e 100644
--- a/depends/javacheck/CMakeLists.txt
+++ b/depends/javacheck/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 2.8.6)
+cmake_minimum_required(VERSION 3.1)
 project(launcher Java)
 find_package(Java 1.6 REQUIRED COMPONENTS Development)
 
diff --git a/depends/launcher/CMakeLists.txt b/depends/launcher/CMakeLists.txt
index 7564161d..b62805e0 100644
--- a/depends/launcher/CMakeLists.txt
+++ b/depends/launcher/CMakeLists.txt
@@ -1,4 +1,4 @@
-cmake_minimum_required(VERSION 2.8.6)
+cmake_minimum_required(VERSION 3.1)
 project(launcher Java)
 find_package(Java 1.6 REQUIRED COMPONENTS Development)
 
diff --git a/depends/libnbtplusplus b/depends/libnbtplusplus
-Subproject 9d17b0a9b93e684a5ac6cac8f70d9a7909a877c
+Subproject 5d0ffb50a526173ce58ae57136bf5d79a7e1920
diff --git a/depends/pack200/CMakeLists.txt b/depends/pack200/CMakeLists.txt
index 4624b483..19a7643c 100644
--- a/depends/pack200/CMakeLists.txt
+++ b/depends/pack200/CMakeLists.txt
@@ -1,24 +1,11 @@
-cmake_minimum_required(VERSION 2.6)
-
-if(WIN32)
-	# In Qt 5.1+ we have our own main() function, don't autolink to qtmain on Windows
-	cmake_policy(SET CMP0020 OLD)
-endif()
+cmake_minimum_required(VERSION 3.1)
 
 project(unpack200)
 
+option(PACK200_BUILD_BINARY "Build a tiny utility that decompresses pack200 streams" OFF)
+
 # Find ZLIB for quazip
-# Use system zlib on unix and Qt ZLIB on Windows
-if(UNIX)
-	find_package(ZLIB REQUIRED)
-else(UNIX)
-	get_filename_component(ZLIB_FOUND_DIR "${Qt5Core_DIR}/../../../include/QtZlib" ABSOLUTE)
-	set(ZLIB_INCLUDE_DIRS ${ZLIB_FOUND_DIR} CACHE PATH "Path to ZLIB headers of Qt")
-	set(ZLIB_LIBRARIES "")
-	if(NOT EXISTS "${ZLIB_INCLUDE_DIRS}/zlib.h")
-		message("Please specify a valid zlib include dir")
-	endif(NOT EXISTS "${ZLIB_INCLUDE_DIRS}/zlib.h")
-endif(UNIX)
+find_package(ZLIB REQUIRED)
 
 set(PACK200_SRC
 	include/unpack200.h
@@ -41,19 +28,14 @@ set(PACK200_SRC
 
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
-set(PACK200_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include" PARENT_SCOPE)
-include_directories(
-	include
-	${ZLIB_INCLUDE_DIRS}
-)
 add_library(unpack200 STATIC ${PACK200_SRC})
+target_include_directories(unpack200
+	PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include"
+	PRIVATE ${ZLIB_INCLUDE_DIRS})
 
-if(UNIX)
-	target_link_libraries(unpack200 ${ZLIB_LIBRARIES})
-else()
-	# zlib is part of Qt on windows. use it.
-	qt5_use_modules(unpack200 Core)
-endif()
+target_link_libraries(unpack200 ${ZLIB_LIBRARIES})
 
-add_executable(anti200 anti200.cpp)
-target_link_libraries(anti200 unpack200)
+if(PACK200_BUILD_BINARY)
+	add_executable(anti200 anti200.cpp)
+	target_link_libraries(anti200 unpack200)
+endif()
diff --git a/depends/rainbow/CMakeLists.txt b/depends/rainbow/CMakeLists.txt
index 6eeb2adc..01342d1b 100644
--- a/depends/rainbow/CMakeLists.txt
+++ b/depends/rainbow/CMakeLists.txt
@@ -1,20 +1,15 @@
-cmake_minimum_required(VERSION 2.8.11)
+cmake_minimum_required(VERSION 3.1)
 project(rainbow)
 
 find_package(Qt5Core REQUIRED QUIET)
 find_package(Qt5Gui REQUIRED QUIET)
 
-include_directories(${Qt5Core_INCLUDE_DIRS})
-include_directories(${Qt5Gui_INCLUDE_DIRS})
-
 set(RAINBOW_SOURCES
-include/rainbow_config.h
-include/rainbow.h
 src/rainbow.cpp
 )
 
 add_definitions(-DRAINBOW_LIBRARY)
-set(RAINBOW_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include" PARENT_SCOPE)
 add_library(rainbow SHARED ${RAINBOW_SOURCES})
+target_include_directories(rainbow PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
 
 qt5_use_modules(rainbow Core Gui)
diff --git a/depends/util/CMakeLists.txt b/depends/util/CMakeLists.txt
index 40adae7e..e8646e1c 100644
--- a/depends/util/CMakeLists.txt
+++ b/depends/util/CMakeLists.txt
@@ -1,42 +1,28 @@
-project(libUtil)
+project(MultiMC_util)
 
-include(UseCXX14)
 include(Coverage)
 
 # Find Qt
 find_package(Qt5Core REQUIRED)
-
-# Include Qt headers.
-include_directories(${Qt5Base_INCLUDE_DIRS})
+find_package(Qt5Gui REQUIRED)
 
 set(LIBUTIL_SOURCES
-	include/libutil_config.h
-
-	include/pathutils.h
 	src/pathutils.cpp
-
-	include/osutils.h
-
-	include/userutils.h
 	src/userutils.cpp
-
-	include/cmdutils.h
 	src/cmdutils.cpp
-
-	include/modutils.h
 	src/modutils.cpp
+	include/cmdutils.h
+	include/modutils.h
+	include/osutils.h
+	include/pathutils.h
+	include/userutils.h
 )
 
-# Set the include dir path.
-set(LIBUTIL_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include" PARENT_SCOPE)
-
-# Static link!
-add_definitions(-DLIBUTIL_STATIC)
-
-add_definitions(-DLIBUTIL_LIBRARY)
-
 set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 
-add_library(libUtil STATIC ${LIBUTIL_SOURCES})
-qt5_use_modules(libUtil Core)
-target_link_libraries(libUtil)
+add_library(MultiMC_util STATIC ${LIBUTIL_SOURCES})
+
+qt5_use_modules(MultiMC_util Core Gui)
+generate_export_header(MultiMC_util)
+target_include_directories(MultiMC_util PUBLIC ${CMAKE_CURRENT_BINARY_DIR} ${CMAKE_CURRENT_SOURCE_DIR}/include)
+set_property(TARGET MultiMC_util PROPERTY CXX_STANDARD 14)
diff --git a/depends/util/include/cmdutils.h b/depends/util/include/cmdutils.h
index 56ae4edb..8bf39eaa 100644
--- a/depends/util/include/cmdutils.h
+++ b/depends/util/include/cmdutils.h
@@ -25,7 +25,7 @@
 #include <QHash>
 #include <QStringList>
 
-#include "libutil_config.h"
+#include "multimc_util_export.h"
 
 /**
  * @file libutil/include/cmdutils.h
@@ -42,7 +42,7 @@ namespace Commandline
  * @param args the argument string
  * @return a QStringList containing all arguments
  */
-LIBUTIL_EXPORT QStringList splitArgs(QString args);
+MULTIMC_UTIL_EXPORT QStringList splitArgs(QString args);
 
 /**
  * @brief The FlagStyle enum
@@ -85,7 +85,7 @@ enum Enum
 /**
  * @brief The ParsingError class
  */
-class LIBUTIL_EXPORT ParsingError : public std::runtime_error
+class MULTIMC_UTIL_EXPORT ParsingError : public std::runtime_error
 {
 public:
 	ParsingError(const QString &what);
@@ -94,7 +94,7 @@ public:
 /**
  * @brief The Parser class
  */
-class LIBUTIL_EXPORT Parser
+class MULTIMC_UTIL_EXPORT Parser
 {
 public:
 	/**
diff --git a/depends/util/include/libutil_config.h b/depends/util/include/libutil_config.h
deleted file mode 100644
index 422001b8..00000000
--- a/depends/util/include/libutil_config.h
+++ /dev/null
@@ -1,28 +0,0 @@
-/* Copyright 2013-2015 MultiMC Contributors
- *
- * Licensed under the Apache License, Version 2.0 (the "License");
- * you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS,
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- * See the License for the specific language governing permissions and
- * limitations under the License.
- */
-
-#pragma once
-
-#include <QtCore/QtGlobal>
-
-#ifdef LIBUTIL_STATIC
-#define LIBUTIL_EXPORT
-#else
-#ifdef LIBUTIL_LIBRARY
-#define LIBUTIL_EXPORT Q_DECL_EXPORT
-#else
-#define LIBUTIL_EXPORT Q_DECL_IMPORT
-#endif
-#endif
diff --git a/depends/util/include/modutils.h b/depends/util/include/modutils.h
index bc608f96..fde51881 100644
--- a/depends/util/include/modutils.h
+++ b/depends/util/include/modutils.h
@@ -3,7 +3,7 @@
 #include <QString>
 #include <QList>
 
-#include "libutil_config.h"
+#include "multimc_util_export.h"
 
 class QUrl;
 
@@ -107,7 +107,7 @@ private:
 	void parse();
 };
 
-LIBUTIL_EXPORT bool versionIsInInterval(const QString &version, const QString &interval);
-LIBUTIL_EXPORT bool versionIsInInterval(const Version &version, const QString &interval);
+MULTIMC_UTIL_EXPORT bool versionIsInInterval(const QString &version, const QString &interval);
+MULTIMC_UTIL_EXPORT bool versionIsInInterval(const Version &version, const QString &interval);
 }
 
diff --git a/depends/util/include/pathutils.h b/depends/util/include/pathutils.h
index 82d55cd7..6d52097f 100644
--- a/depends/util/include/pathutils.h
+++ b/depends/util/include/pathutils.h
@@ -18,12 +18,12 @@
 #include <QString>
 #include <QDir>
 
-#include "libutil_config.h"
+#include "multimc_util_export.h"
 
-LIBUTIL_EXPORT QString PathCombine(QString path1, QString path2);
-LIBUTIL_EXPORT QString PathCombine(QString path1, QString path2, QString path3);
+MULTIMC_UTIL_EXPORT QString PathCombine(QString path1, QString path2);
+MULTIMC_UTIL_EXPORT QString PathCombine(QString path1, QString path2, QString path3);
 
-LIBUTIL_EXPORT QString AbsolutePath(QString path);
+MULTIMC_UTIL_EXPORT QString AbsolutePath(QString path);
 
 /**
  * Normalize path
@@ -35,37 +35,37 @@ LIBUTIL_EXPORT QString AbsolutePath(QString path);
  */
 QString NormalizePath(QString path);
 
-LIBUTIL_EXPORT QString RemoveInvalidFilenameChars(QString string, QChar replaceWith = '-');
+MULTIMC_UTIL_EXPORT QString RemoveInvalidFilenameChars(QString string, QChar replaceWith = '-');
 
-LIBUTIL_EXPORT QString DirNameFromString(QString string, QString inDir = ".");
+MULTIMC_UTIL_EXPORT QString DirNameFromString(QString string, QString inDir = ".");
 
 /**
  * Creates all the folders in a path for the specified path
  * last segment of the path is treated as a file name and is ignored!
  */
-LIBUTIL_EXPORT bool ensureFilePathExists(QString filenamepath);
+MULTIMC_UTIL_EXPORT bool ensureFilePathExists(QString filenamepath);
 
 /**
  * Creates all the folders in a path for the specified path
  * last segment of the path is treated as a folder name and is created!
  */
-LIBUTIL_EXPORT bool ensureFolderPathExists(QString filenamepath);
+MULTIMC_UTIL_EXPORT bool ensureFolderPathExists(QString filenamepath);
 
 /**
  * Copy a folder recursively
  */
-LIBUTIL_EXPORT bool copyPath(const QString &src, const QString &dst, bool follow_symlinks = true);
+MULTIMC_UTIL_EXPORT bool copyPath(const QString &src, const QString &dst, bool follow_symlinks = true);
 
 /**
  * Delete a folder recursively
  */
-LIBUTIL_EXPORT bool deletePath(QString path);
+MULTIMC_UTIL_EXPORT bool deletePath(QString path);
 
 /// Opens the given file in the default application.
-LIBUTIL_EXPORT void openFileInDefaultProgram(QString filename);
+MULTIMC_UTIL_EXPORT void openFileInDefaultProgram(QString filename);
 
 /// Opens the given directory in the default application.
-LIBUTIL_EXPORT void openDirInDefaultProgram(QString dirpath, bool ensureExists = false);
+MULTIMC_UTIL_EXPORT void openDirInDefaultProgram(QString dirpath, bool ensureExists = false);
 
 /// Checks if the a given Path contains "!"
-LIBUTIL_EXPORT bool checkProblemticPathJava(QDir folder);
+MULTIMC_UTIL_EXPORT bool checkProblemticPathJava(QDir folder);
diff --git a/depends/util/include/userutils.h b/depends/util/include/userutils.h
index 6ce08bce..aadc112e 100644
--- a/depends/util/include/userutils.h
+++ b/depends/util/include/userutils.h
@@ -2,16 +2,16 @@
 
 #include <QString>
 
-#include "libutil_config.h"
+#include "multimc_util_export.h"
 
 namespace Util
 {
 // Get the Directory representing the User's Desktop
-LIBUTIL_EXPORT QString getDesktopDir();
+MULTIMC_UTIL_EXPORT QString getDesktopDir();
 
 // Create a shortcut at *location*, pointing to *dest* called with the arguments *args*
 // call it *name* and assign it the icon *icon*
 // return true if operation succeeded
-LIBUTIL_EXPORT bool createShortCut(QString location, QString dest, QStringList args,
+MULTIMC_UTIL_EXPORT bool createShortCut(QString location, QString dest, QStringList args,
 								   QString name, QString iconLocation);
 }
diff --git a/depends/xz-embedded/CMakeLists.txt b/depends/xz-embedded/CMakeLists.txt
index bc1759dc..5f744671 100644
--- a/depends/xz-embedded/CMakeLists.txt
+++ b/depends/xz-embedded/CMakeLists.txt
@@ -1,32 +1,26 @@
-cmake_minimum_required(VERSION 2.6)
-project(xz-embedded)
+cmake_minimum_required(VERSION 3.1)
+project(xz-embedded LANGUAGES C)
 
 option(XZ_BUILD_BCJ "Build xz-embedded with BCJ support (native binary optimization)" OFF)
 option(XZ_BUILD_CRC64 "Build xz-embedded with CRC64 checksum support" ON)
 option(XZ_BUILD_MINIDEC "Build a tiny utility that decompresses xz streams" OFF)
 
-set(CMAKE_C_FLAGS "-std=c99")
-
-include_directories(include)
-set(XZ_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include" PARENT_SCOPE)
-
 # See include/xz.h for manual feature configuration
 # tweak this list and xz.h to fit your needs
 
 set(XZ_SOURCES
-	include/xz.h
-	src/xz_config.h
 	src/xz_crc32.c
 	src/xz_crc64.c
 	src/xz_dec_lzma2.c
 	src/xz_dec_stream.c
-	src/xz_lzma2.h
-	src/xz_private.h
-	src/xz_stream.h
 #	src/xz_dec_bcj.c
 )
-# TODO: look into what would be needed for plain old lzma
-
 add_library(xz-embedded STATIC ${XZ_SOURCES})
-add_executable(xzminidec xzminidec.c)
-target_link_libraries(xzminidec xz-embedded)
+target_include_directories(xz-embedded PUBLIC "${CMAKE_CURRENT_SOURCE_DIR}/include")
+set_property(TARGET xz-embedded PROPERTY C_STANDARD 99)
+
+if(${XZ_BUILD_MINIDEC})
+	add_executable(xzminidec xzminidec.c)
+	target_link_libraries(xzminidec xz-embedded)
+	set_property(TARGET xzminidec PROPERTY C_STANDARD 99)
+endif()