added comments to cross.toml, fixed getting the tags and labels in the

push steps of the build pipeline

Author: Benedikt-Illich

.github/workflows/containerize-and-push.yaml

@@ -76,8 +76,8 @@ jobs:
           file: "Dockerfile.zenoh-mqtt-arm"
           push: true
           platforms: linux/arm64
-          tags: ${{ steps.meta.outputs.tags }}
-          labels: ${{ steps.meta.outputs.labels }}
+          tags: ${{ needs.setup.outputs.tags }}
+          labels: ${{ needs.setup.outputs.labels }}
 
   build-amd64:
     needs: setup
@@ -111,5 +111,5 @@ jobs:
           file: "Dockerfile.zenoh-mqtt-amd"
           push: true
           platforms: linux/amd64
-          tags: ${{ steps.meta.outputs.tags }}
-          labels: ${{ steps.meta.outputs.labels }}
+          tags: ${{ needs.setup.outputs.tags }}
+          labels: ${{ needs.setup.outputs.labels }}

Cross.toml

@@ -12,6 +12,8 @@
 # *******************************************************************************/
 
 [build.env]
+# This overrides the CMake toolchain file that Cross natively wants to use,
+# since it causes problems with the vendored OpenSSL version that Paho wants to use
 passthrough = [
     "CMAKE_TOOLCHAIN_FILE_aarch64_unknown_linux_musl",
     "CMAKE_TOOLCHAIN_FILE_x86_64_unknown_linux_musl",