refactor(packages): emit dependency build params only when the dep is present

Same ifDep guard as the gdal recipe for geotiff, tiff, spatialite,
proj and curl: a dependency removed via excludedDependencies or an
override no longer crashes getBuildParams; its params (including
feature toggles like -Djpeg=ON) are simply omitted. Full-dependency
output is byte-identical.

Author: bugra9

cppjs-packages/cppjs-package-curl/cppjs-package-curl/build.mjs

@@ -12,9 +12,13 @@ export default {
     buildType: 'cmake',
     getBuildParams: (target, depPaths) => [
         ...(platformBuild[target.platform] || []),
-        `-DOPENSSL_INCLUDE_DIR=${depPaths.ssl.header}`,
-        `-DOPENSSL_SSL_LIBRARY=${depPaths.ssl.lib}`,
-        `-DOPENSSL_CRYPTO_LIBRARY=${depPaths.crypto.lib}`,
+        ...(depPaths.ssl && depPaths.crypto
+            ? [
+                `-DOPENSSL_INCLUDE_DIR=${depPaths.ssl.header}`,
+                `-DOPENSSL_SSL_LIBRARY=${depPaths.ssl.lib}`,
+                `-DOPENSSL_CRYPTO_LIBRARY=${depPaths.crypto.lib}`,
+            ]
+            : []),
         '-DBUILD_EXAMPLES=OFF', '-DBUILD_CURL_EXE=OFF', '-DBUILD_LIBCURL_DOCS=OFF',
         '-DBUILD_TESTING=OFF',
         '-DENABLE_CURL_MANUAL=OFF',

cppjs-packages/cppjs-package-geotiff/cppjs-package-geotiff/build.mjs

@@ -12,13 +12,17 @@ const platformExtraLibs = {
     'ios': ['-lc++'],
 };
 
+const ifDep = (dep, params) => (dep ? params(dep) : []);
+
 export default {
     getURL: (version) => `https://download.osgeo.org/geotiff/libgeotiff/libgeotiff-${version}.tar.gz`,
     buildType: 'configure',
     getBuildParams: (target, depPaths) => [
         ...(platformBuild[target.platform] || platformBuild[`${target.platform}-${target.arch}`] || []),
-        `--with-proj=${depPaths.proj.root}`, `--with-libtiff=${depPaths.tiff.root}`,
-        `--with-zlib=${depPaths.z.root}`, `--with-jpeg=${depPaths.jpeg.root}`,
+        ...ifDep(depPaths.proj, (d) => [`--with-proj=${d.root}`]),
+        ...ifDep(depPaths.tiff, (d) => [`--with-libtiff=${d.root}`]),
+        ...ifDep(depPaths.z, (d) => [`--with-zlib=${d.root}`]),
+        ...ifDep(depPaths.jpeg, (d) => [`--with-jpeg=${d.root}`]),
     ],
     getExtraLibs: (target) => platformExtraLibs[target.platform] || [],
     replaceList: [

cppjs-packages/cppjs-package-proj/cppjs-package-proj/build.mjs

@@ -1,11 +1,17 @@
+const ifDep = (dep, params) => (dep ? params(dep) : []);
+
 export default {
     getURL: (version) => `https://download.osgeo.org/proj/proj-${version}.tar.gz`,
     buildType: 'cmake',
     getBuildParams: (target, depPaths) => [
         '-DENABLE_CURL=OFF', '-DBUILD_TESTING=OFF', '-DBUILD_APPS=OFF',
-        `-DSQLite3_INCLUDE_DIR=${depPaths.sqlite3.header}`,
-        `-DSQLite3_LIBRARY=${depPaths.sqlite3.lib}`,
-        `-DTIFF_INCLUDE_DIR=${depPaths.tiff.header}`,
-        `-DTIFF_LIBRARY_RELEASE=${depPaths.tiff.lib}`,
+        ...ifDep(depPaths.sqlite3, (d) => [
+            `-DSQLite3_INCLUDE_DIR=${d.header}`,
+            `-DSQLite3_LIBRARY=${d.lib}`,
+        ]),
+        ...ifDep(depPaths.tiff, (d) => [
+            `-DTIFF_INCLUDE_DIR=${d.header}`,
+            `-DTIFF_LIBRARY_RELEASE=${d.lib}`,
+        ]),
     ],
 };

cppjs-packages/cppjs-package-spatialite/cppjs-package-spatialite/build.mjs

@@ -23,6 +23,8 @@ const platformSourceReplaceList = {
     ],
 };
 
+const ifDep = (dep, params) => (dep ? params(dep) : []);
+
 export default {
     getURL: (version) => `https://www.gaia-gis.it/gaia-sins/libspatialite-sources/libspatialite-${version}.tar.gz`,
     copyToSource: { 'config.sub': 'config.sub' },
@@ -32,9 +34,11 @@ export default {
         ...(platformBuild[target.platform] || platformBuild[`${target.platform}-${target.arch}`] || []),
         '--enable-geosadvanced=yes', '--enable-geopackage=yes', '--enable-examples=no', '--enable-minizip=no',
         '--enable-libxml2=no', '--enable-freexl=no', '--disable-rttopo', '--disable-gcp',
-        `--with-geosconfig=${depPaths.geos.bin}/geos-config`,
-        `SQLITE3_CFLAGS=-I${depPaths.sqlite3.header}`,
-        `SQLITE3_LIBS=-L${depPaths.sqlite3.libPath}`,
+        ...ifDep(depPaths.geos, (d) => [`--with-geosconfig=${d.bin}/geos-config`]),
+        ...ifDep(depPaths.sqlite3, (d) => [
+            `SQLITE3_CFLAGS=-I${d.header}`,
+            `SQLITE3_LIBS=-L${d.libPath}`,
+        ]),
     ],
     getExtraLibs: (target) => platformLibs[target.platform] || [],
 };

cppjs-packages/cppjs-package-tiff/cppjs-package-tiff/build.mjs

@@ -1,17 +1,25 @@
+const ifDep = (dep, params) => (dep ? params(dep) : []);
+
 export default {
     getURL: (version) => `https://download.osgeo.org/libtiff/tiff-${version}.tar.gz`,
     buildType: 'cmake',
     getBuildParams: (target, depPaths) => [
         '-Dtiff-tools=OFF', '-Dtiff-tests=OFF', '-Dtiff-contrib=OFF',
         '-Dtiff-docs=OFF', '-Dld-version-script=OFF',
-        '-Djpeg=ON',
-        `-DJPEG_INCLUDE_DIR=${depPaths.jpeg.header}`,
-        `-DJPEG_LIBRARY=${depPaths.jpeg.lib}`,
-        '-Dzstd=ON',
-        `-DZSTD_INCLUDE_DIR=${depPaths.zstd.header}`,
-        `-DZSTD_LIBRARY=${depPaths.zstd.lib}`,
-        '-Dlerc=ON',
-        `-DLERC_INCLUDE_DIR=${depPaths.Lerc.header}`,
-        `-DLERC_LIBRARY=${depPaths.Lerc.lib}`,
+        ...ifDep(depPaths.jpeg, (d) => [
+            '-Djpeg=ON',
+            `-DJPEG_INCLUDE_DIR=${d.header}`,
+            `-DJPEG_LIBRARY=${d.lib}`,
+        ]),
+        ...ifDep(depPaths.zstd, (d) => [
+            '-Dzstd=ON',
+            `-DZSTD_INCLUDE_DIR=${d.header}`,
+            `-DZSTD_LIBRARY=${d.lib}`,
+        ]),
+        ...ifDep(depPaths.Lerc, (d) => [
+            '-Dlerc=ON',
+            `-DLERC_INCLUDE_DIR=${d.header}`,
+            `-DLERC_LIBRARY=${d.lib}`,
+        ]),
     ],
 };