9f56bb608e
Add CI checker to compare `libcurl.pc` and `curl-config` files
generated by autotools and cmake builds.
Fix differences and apply tiny cleanups:
- curl-config: use single-quotes for literals.
- curl-config: quote all variables.
- curl-config: replace double with single quotes in a substituted value
that's always literal (`@prefix@`).
- libcurl.pc: spelling in `Description:`.
- libcurl.pc: avoid substitution in a comment.
- cmake: fill `libdir` with `${exec_prefix}` instead of a literal.
To sync with './configure'.
- configure: fix `CURL_CA_BUNDLE` value to not generate nested quotes
in `curl-config`.
- configure: add missing `LDFLAGS` to `Libs.private` in `libcurl.pc`.
To sync with CMake.
- cmake: skip adding `CMAKE_C_IMPLICIT_LINK_LIBRARIES` for MINGW and
UNIX. They added these values as seen in CI:
MINGW: `-lmingw32 -lgcc -lmoldname -lmingwex -ladvapi32 -lshell32 -luser32 -lkernel32 -lmingw32 -lgcc -lmoldname -lmingwex`
Linux: `-lgcc -lgcc_s -lc -lgcc -lgcc_s`
- cmake: delete FIXME about enabling libssh2 by default.
`./configure` has special defaults for these packages (called: "off"):
brotli, zstd, libpsl, libssh2, libssh, wolfssl, librtmp
It looks for them, but only at system locations, which makes them
never detected e.g. on macOS. CMake doesn't offer such default mode
for now.
- GHA/macos: drop now redundant `-DCURL_DISABLE_LDAPS=ON`.
- cmake: use `CMAKE_INSTALL_INCLUDEDIR` and `CMAKE_INSTALL_LIBDIR`
instead of hardcoded `include`/`lib` when generating `libcurl.pc`.
Updates to the GHA workflow:
- move autotools out-of-tree and rename cmake out-of-tree directory
to `bld_cm` to tell it's cmake.
- disable static libcurl for `./configure` to match cmake.
- enable `pkg-config` debug output with `./configure`.
- dump list of Homebrew packages on macOS.
- dump `./configure` detailed logs.
- disable zstd and brotli for Linux, to match cmake.
There remain differences, mostly due to detection order and method. Also
some values are inherently different when using CMake and autotools,
such as `--cc`, `--configure`. autotools also generates duplicates for
`-lssl` and `-lcrypto`. macOS LDAP wants to link `-lber` while autotools
doesn't. Some build defaults are also different in autotools and cmake.
These differences are smoothened out for now by the checker script, or
via build options. Notice that lib order (a dupes) _can_ be significant
in some cases. E.g. the binutils linker is infamous for that on Windows.
Closes #14681
50 lines
1.6 KiB
Bash
Executable File
50 lines
1.6 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
# Copyright (C) Viktor Szakats
|
|
#
|
|
# SPDX-License-Identifier: curl
|
|
|
|
# Sort list of libs, libpaths, cflags found in libcurl.pc and curl-config files,
|
|
# then diff the autotools and cmake generated ones.
|
|
|
|
sort_lists() {
|
|
prevline=''
|
|
section=''
|
|
while IFS= read -r l; do
|
|
if [[ "${prevline}" =~ (--cc|--configure) ]]; then # curl-config
|
|
echo "<IGNORED>"
|
|
else
|
|
# libcurl.pc
|
|
if [[ "${l}" =~ ^(Requires|Libs|Cflags)(\.private)?:\ (.+)$ ]]; then
|
|
if [ "${BASH_REMATCH[1]}" = 'Requires' ]; then
|
|
# Spec does not allow duplicates here:
|
|
# https://manpages.debian.org/unstable/pkg-config/pkg-config.1.en.html#Requires:
|
|
# "You may only mention the same package one time on the Requires: line"
|
|
val="$(printf '%s' "${BASH_REMATCH[3]}" | tr ',' '\n' | sort | tr '\n' ' ')"
|
|
else
|
|
val="$(printf '%s' "${BASH_REMATCH[3]}" | tr ' ' '\n' | sort -u | tr '\n' ' ')"
|
|
fi
|
|
l="${BASH_REMATCH[1]}${BASH_REMATCH[2]}: ${val}"
|
|
# curl-config
|
|
elif [[ "${section}" =~ (--libs|--static-libs) && "${l}" =~ ^( *echo\ \")(.+)(\")$ ]]; then
|
|
val="$(printf '%s' "${BASH_REMATCH[2]}" | tr ' ' '\n' | sort -u | tr '\n' ' ')"
|
|
l="${BASH_REMATCH[1]}${val}${BASH_REMATCH[3]}"
|
|
section=''
|
|
fi
|
|
echo "${l}"
|
|
fi
|
|
# curl-config
|
|
prevline="${l}"
|
|
if [[ "${l}" =~ --[a-z-]+\) ]]; then
|
|
section="${BASH_REMATCH[0]}"
|
|
fi
|
|
done < "$1"
|
|
}
|
|
|
|
am=$(mktemp -t autotools.XXX); sort_lists "$1" > "${am}"
|
|
cm=$(mktemp -t cmake.XXX) ; sort_lists "$2" > "${cm}"
|
|
diff -u "${am}" "${cm}"
|
|
res="$?"
|
|
rm -r -f "${am}" "${cm}"
|
|
|
|
exit "${res}"
|