check-abi: explicitly provide public headers

jonasnick · jonasnick · commit dd695563e68d · 2024-05-06T16:28:01.000Z
Without this commit, the check-abi shell script outputs false positives because
it consider some headers public that are actually not public.
diff --git a/tools/check-abi.sh b/tools/check-abi.sh
@@ -49,7 +49,7 @@ checkout_and_build() {
         -DSECP256K1_BUILD_CTIME_TESTS=OFF \
         -DSECP256K1_BUILD_EXAMPLES=OFF
     cmake --build . -j "$(nproc)"
-    abi-dumper src/libsecp256k1.so -o ABI.dump -lver "$2"
+    abi-dumper src/libsecp256k1.so -o ABI.dump -lver "$2" -public-headers ../include/
     cd "$_orig_dir"
 }
 

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ checkout_and_build() {`
`49`	`49`	`-DSECP256K1_BUILD_CTIME_TESTS=OFF \`
`50`	`50`	`-DSECP256K1_BUILD_EXAMPLES=OFF`
`51`	`51`	`cmake --build . -j "$(nproc)"`
`52`		`- abi-dumper src/libsecp256k1.so -o ABI.dump -lver "$2"`
	`52`	`+ abi-dumper src/libsecp256k1.so -o ABI.dump -lver "$2" -public-headers ../include/`
`53`	`53`	`cd "$_orig_dir"`
`54`	`54`	`}`
`55`	`55`