AVRO-3932: [C] fix variable reference in CMakeLists.txt (#2713)

AVRO-3932: [C] fix variable reference in CMakeLists.txt Check that `pkg-config --libs avro-c` does not produce errors Signed-off-by: Martin Tzvetanov Grigorov <[email protected]> --------- Signed-off-by: Martin Tzvetanov Grigorov <[email protected]> Co-authored-by: Martin Tzvetanov Grigorov <[email protected]>
apache · Jan 29, 2024 · 4e03a4c · 4e03a4c
1 parent bb5ccc7
commit 4e03a4c
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/.github/workflows/test-lang-c.yml b/.github/workflows/test-lang-c.yml
@@ -47,6 +47,15 @@ jobs:
       - name: Test
         run: ./build.sh test
 
+      - name: Check pkg-config
+        run: |
+          mkdir -p build
+          cd build
+          cmake ..
+          cp /usr/lib/x86_64-linux-gnu/pkgconfig/snappy.pc src/libsnappy.pc
+          export PKG_CONFIG_PATH=./src
+          pkg-config --libs avro-c
+
       - name: Cache Local Maven Repository
         uses: actions/cache@v4
         with:

diff --git a/lang/c/CMakeLists.txt b/lang/c/CMakeLists.txt
@@ -176,7 +176,7 @@ else (LZMA_FOUND)
 endif (LZMA_FOUND)
 
 set(CODEC_LIBRARIES ${ZLIB_LIBRARIES} ${LZMA_LIBRARIES} ${SNAPPY_LIBRARIES})
-set(CODEC_PKG "@ZLIB_PKG@ @LZMA_PKG@ @SNAPPY_PKG@")
+set(CODEC_PKG "${ZLIB_PKG} ${LZMA_PKG} ${SNAPPY_PKG}")
 
 # Jansson JSON library
 pkg_check_modules(JANSSON jansson>=2.3)