Merge 25c629f5ed into 577caa4793

.gitignore

@@ -18,3 +18,6 @@ build
 .build
 .deps
 CMakeSettings.json
+# third_party exclusions
+third_party/tinyexif
+third_party/tinyxml2

.gitmodules

@@ -75,9 +75,3 @@
 [submodule "third_party/qoi"]
 	path = third_party/qoi
 	url = https://github.com/aseprite/qoi.git
-[submodule "third_party/tinyxml2"]
-	path = third_party/tinyxml2
-	url = https://github.com/aseprite/tinyxml2.git
-[submodule "third_party/TinyEXIF"]
-	path = third_party/TinyEXIF
-	url = https://github.com/aseprite/TinyEXIF.git

CMakeLists.txt

@@ -54,6 +54,9 @@ if(NOT EXISTS "${CMAKE_CURRENT_SOURCE_DIR}/laf/CMakeLists.txt" OR
     "  https://github.com/aseprite/aseprite/blob/main/INSTALL.md#get-the-source-code")
 endif()
 
+set(CPM_SOURCE_CACHE third_party)
+include(cmake/CPM.cmake)
+
 # This is required for KDE/Qt destop integration, which sets
 # BUILD_SHARED_LIBS to TRUE by default
 set(BUILD_SHARED_LIBS off)
@@ -262,23 +265,25 @@ add_definitions(-DPNG_NO_MMX_CODE) # Do not use MMX optimizations in PNG code
 
 # tinyxml2
 if(USE_SHARED_TINYXML)
-  find_library(TINYXML_LIBRARY NAMES tinyxml2)
-  find_path(TINYXML_INCLUDE_DIR NAMES tinyxml2.h)
+  find_package(tinyxml2 REQUIRED)
 else()
-  set(TINYXML_LIBRARY tinyxml2)
-  set(TINYXML_INCLUDE_DIR ${TINYXML_DIR})
+  CPMAddPackage(
+    URI "gh:aseprite/tinyxml2#312a8092245df393db14a0b2427457ed2ba75e1b"
+    OPTIONS "tinyxml2_BUILD_TESTING OFF"
+  )
 endif()
-include_directories(${TINYXML_INCLUDE_DIR})
 
 # TinyEXIF
 if(USE_SHARED_TINYEXIF)
-  find_library(TINYEXIF_LIBRARY NAMES TinyEXIF)
-  find_path(TINYEXIF_INCLUDE_DIR NAMES TinyEXIF.h)
+  find_package(TinyEXIF REQUIRED)
+  set(TINYEXIF_LIBRARY TinyEXIF)
 else()
+  CPMAddPackage(
+    URI "gh:aseprite/TinyEXIF#21d6cf762724260f8e5bcee4abdee5411ae47813"
+    OPTIONS "BUILD_SHARED_LIBS OFF" "BUILD_STATIC_LIBS ON" "LINK_CRT_STATIC_LIBS OFF" "BUILD_DEMO OFF"
+  )
   set(TINYEXIF_LIBRARY TinyEXIFstatic)
-  set(TINYEXIF_INCLUDE_DIR ${TINYEXIF_DIR})
 endif()
-include_directories(${TINYEXIF_INCLUDE_DIR})
 
 # pixman
 if(USE_SHARED_PIXMAN)

docs/LICENSES.md

@@ -2,6 +2,7 @@ Aseprite uses the following open source projects:
 
 * [Allegro 4](http://liballeg.org/)
 * [Bresenham algorithm implementations by Alois Zingl](http://members.chello.at/easyfilter/bresenham.html)
+* [CPM.cmake](https://github.com/cpm-cmake/CPM.cmake)
 * [cityhash](https://github.com/google/cityhash)
 * [cmark](https://github.com/jgm/cmark)
 * [curl](http://curl.haxx.se/)
@@ -82,6 +83,30 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 SOFTWARE.
 ```
 
+# [CPM.cmake](https://github.com/cpm-cmake/CPM.cmake)
+
+```
+Copyright (c) 2019-2022 Lars Melchior and contributors
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.
+```
+
 # [cityhash](https://github.com/google/cityhash)
 
 ```

src/app/CMakeLists.txt

@@ -768,7 +768,7 @@ target_link_libraries(app-lib
   updater-lib
   undo
   ${CMARK_LIBRARIES}
-  ${TINYXML_LIBRARY}
+  tinyxml2::tinyxml2
   ${TINYEXIF_LIBRARY}
   ${GIF_LIBRARIES}
   ${PNG_LIBRARIES}

src/gen/CMakeLists.txt

@@ -18,4 +18,4 @@ endif()
 target_link_libraries(gen
   laf-base
   cfg-lib
-  ${TINYXML_LIBRARY})
+  tinyxml2::tinyxml2)

src/updater/CMakeLists.txt

@@ -11,5 +11,5 @@ target_link_libraries(updater-lib laf-base ver-lib)
 # portion of the library.
 if(ENABLE_UPDATER)
   target_sources(updater-lib PRIVATE check_update.cpp)
-  target_link_libraries(updater-lib net-lib ${TINYXML_LIBRARY})
+  target_link_libraries(updater-lib net-lib tinyxml2::tinyxml2)
 endif()

third_party/CMakeLists.txt

@@ -51,19 +51,6 @@ if(ENABLE_WEBP AND NOT LAF_BACKEND STREQUAL "skia")
   endif()
 endif()
 
-if(NOT USE_SHARED_TINYXML)
-  set(tinyxml2_BUILD_TESTING OFF CACHE BOOL "Build tests for tinyxml2")
-  add_subdirectory(tinyxml2)
-endif()
-
-if(NOT USE_SHARED_TINYEXIF)
-  set(BUILD_SHARED_LIBS OFF CACHE BOOL "build as shared library")
-  set(BUILD_STATIC_LIBS ON CACHE BOOL "build as static library")
-  set(LINK_CRT_STATIC_LIBS OFF CACHE BOOL "link CRT static library")
-  set(BUILD_DEMO OFF CACHE BOOL "build demo binary")
-  add_subdirectory(TinyEXIF)
-endif()
-
 if(REQUIRE_CURL AND NOT USE_SHARED_CURL)
   set(BUILD_RELEASE_DEBUG_DIRS ON BOOL)
   set(BUILD_CURL_EXE OFF CACHE BOOL "Set to ON to build curl executable.")

third_party/TinyEXIF

@@ -1 +0,0 @@
-Subproject commit 21d6cf762724260f8e5bcee4abdee5411ae47813

third_party/tinyxml2

@@ -1 +0,0 @@
-Subproject commit 312a8092245df393db14a0b2427457ed2ba75e1b