fix build (jenkins): Check if 'targets' is None

The jenkins build script (jenkins_build_spark.cmd) specifies
'--target publish' but not '--configuration Release'. It was building
debug by default (not what we want). This fixes that as a side effect.

jenkins/spark.Jenkinsfile

@@ -0,0 +1,61 @@
+pipeline {
+   agent {
+      label 'os=windows && ram=8gb && installed=vs2022 && installed=python3'
+   }
+   stages {
+      stage('Build') {
+         steps {
+            echo 'Building..'
+            script {
+               if (fileExists('.venv')) {
+                  echo 'Using existing Python virtual env'
+               } else {
+                  echo 'Creating Python virtual env'
+                  bat 'tools\\ensure_venv.cmd'
+               }
+            }
+            bat """
+            call .venv\\Scripts\\activate
+            call pip install -r requirements.txt
+            call python tools\\py\\build_spark.py --configuration Release
+            """
+         }
+         post {
+            always {
+               echo '========Build Done========'
+            }
+            success {
+               echo '========Build executed successfully========'
+            }
+            failure {
+               echo '========Build execution failed========'
+            }
+         }
+      }
+      stage('Installing') {
+         steps {
+            echo 'Installing...'
+            bat """
+            "C:\\Program Files\\7-Zip\7z.exe" a -tzip Release.zip "build\\Release"
+            """
+         }
+      }
+      stage('Archiving') {
+         steps {
+            echo 'Archiving...'
+            archiveArtifacts artifacts: 'Release.zip', followSymlinks: false, onlyIfSuccessful: true
+         }
+      }
+   }
+   post {
+      always {
+         echo '========Pipeline Done========'
+      }
+      success {
+         echo '========Pipeline executed successfully ========'
+      }
+      failure {
+         echo '========Pipeline execution failed========'
+      }
+   }
+}

jenkins/sparkd.Jenkinsfile

@@ -0,0 +1,38 @@
+pipeline {
+   agent {
+      label 'lab-main'
+   }
+   
+   stages {
+      stage('Build') {
+         steps {
+            echo 'Building..'
+            if (!fileExists('.venv')) {
+               echo 'Creating python virtual env'
+               sh 'python3 -m venv .venv'
+            }
+            sh """
+            . .venv/bin/activate
+            pip install -r requirements.txt
+            python tools/py/build_sparkd.py --compiler gcc --configuration release
+            """
+         }
+      }
+      stage('Installing') {
+         steps {
+            echo 'Installing..'
+            sh """
+            cd build/gcc/release
+            cmake --build . --target install
+            """
+            sh 'tar -czf install.tar.gz install'
+         }
+      }
+      stage('Archiving') {
+         steps {
+            echo 'Archiving..'
+            archiveArtifacts artifacts: 'install.tar.gz', followSymlinks: false, onlyIfSuccessful: true
+         }
+      }
+   }
+}

tools/build_spark.cmd

@@ -1,4 +1,4 @@
-@echo off
+@echo on
 
 @call %~dp0..\tools\ensure_venv.cmd
 touch %~dp0..\src\lib\CMakeLists.txt

tools/jenkins_build_spark.cmd

@@ -0,0 +1,8 @@
+@echo off
+
+call "C:\\Program Files\\Microsoft Visual Studio\\2022\\Community\\Common7\\Tools\\VsDevCmd.bat"
+call .venv\\Scripts\\activate
+call set PYTHONPATH=%~dp0\..\.venv\Scripts\python
+call .venv\Scripts\python.exe tools\py\build_spark.py %*
+
+EXIT /B %ERRORLEVEL%

tools/py/build_spark.py

@@ -173,8 +173,7 @@ def _get_arg_parser():
         metavar='debug|release',
         type=str,
         nargs='?',
-        help='The build configuration',
-        default='Debug')
+        help='The build configuration')
 
     parser.add_argument(
         '-o',
@@ -336,9 +335,10 @@ def _main():
     link_build_dir = arg_to_path(get_arg(args_dict, 'link_output_dir'))
     targets = flatten_arglist(get_arg(args_dict, 'target'))
     verbosity = get_arg(args_dict, 'verbosity') or None
-
+    configuration = args.configuration or 'Release' if targets is not None and any(
+        filter(lambda t: t.lower() == 'publish', targets)) else 'Debug'
     build(
-        configuration=args.configuration,
+        configuration=configuration,
         clean=args.clean,
         restore=args.restore,
         build_dir=build_dir,