ci: update yaml-cpp (#11096)

coryan · web-flow · commit 974698045791 · 2023-03-23T18:45:15.000-04:00
diff --git a/ci/cloudbuild/dockerfiles/fedora-37-cmake.Dockerfile b/ci/cloudbuild/dockerfiles/fedora-37-cmake.Dockerfile
@@ -40,7 +40,7 @@ RUN dnf makecache && dnf install -y "dnf-command(debuginfo-install)"
 RUN dnf makecache && dnf debuginfo-install -y libstdc++
 
 # This is used by the docfx tool.
-RUN dnf makecache && dnf install -y pugixml-devel yaml-cpp-devel
+RUN dnf makecache && dnf install -y pugixml-devel
 
 # Sets root's password to the empty string to enable users to get a root shell
 # inside the container with `su -` and no password. Sudo would not work because
@@ -63,6 +63,20 @@ RUN curl -sSL https://pkgconfig.freedesktop.org/releases/pkg-config-0.29.2.tar.g
     ldconfig
 ENV PKG_CONFIG_PATH=/usr/local/lib64/pkgconfig:/usr/local/lib/pkgconfig:/usr/lib64/pkgconfig
 
+# Installs the yaml-cpp-devel from source. The version included with Fedora:37
+# leaves trailing whitespace in the output:
+#     https://github.com/jbeder/yaml-cpp/pull/1005
+WORKDIR /var/tmp/build
+RUN curl -sSL https://github.com/jbeder/yaml-cpp/archive/refs/tags/yaml-cpp-0.7.0.tar.gz | \
+    tar -xzf - --strip-components=1 && \
+    cmake \
+      -DCMAKE_BUILD_TYPE=Release \
+      -DBUILD_SHARED_LIBS=ON \
+      -DBUILD_TESTING=OFF \
+      -GNinja -S . -B cmake-out && \
+    cmake --build cmake-out --target install && \
+    ldconfig && cd /var/tmp && rm -fr build
+
 # We expose `absl::optional<>` in our public API. An Abseil LTS update will
 # break our `check-api` build, unless we disable the inline namespace.
 WORKDIR /var/tmp/build
