[gnutls-devel] GnuTLS | Drop lib{gnutls, dane}-latest-x86_64.abi from the repo (#954)

Development of GNU's TLS library gnutls-devel at lists.gnutls.org
Sat Mar 14 08:15:05 CET 2020



Daiki Ueno created an issue: https://gitlab.com/gnutls/gnutls/-/issues/954



When adding a new API function, one biggest pain point is updating `lib{gnutls,dane}-latest-x86_64.abi` files, because the output of `abidw` is not stable and information that relies on the current build environment is emitted, e.g., `comp-dir-path='/home/nmav/cvs/gnutls-mine/lib'`.

Interestingly, `abidiff` only takes into account of the existence of `elf-symbol` (ignores type changes). So one can cheat `abi-check` by manually adding an `elf-symbol` entry for the new function: [example 2](623058337490b847d27b736c67b6e710efb980a7), [example 1](6037706541616cfd2d4b49f6f5939ce6dddd1a53). That means, there is not much sense to maintain `lib{gnutls,dane}-latest-x86_64.abi`, as we already have `devel/symbols.last`.

Therefore I propose:
- drop `lib{gnutls,dane}-latest-x86_64.abi` and check against it (keep `lib{gnutls,dane}-$(VERSION)-x86_64.abi`
- maintain the output of `abidiff` from the previous `lib{gnutls,dane}-$(VERSION)-x86_64.abi` in the repository, say `lib{gnutls,dane}-x86_64.abidiff`
- add an instruction to update `lib{gnutls,dane}-$(VERSION)-x86_64.abi` and clear `lib{gnutls,dane}-x86_64.abidiff` at release time

This not only makes `make file-updates` reliable, but also tightens the check as it covers type changes.

Thoughts?

-- 
Reply to this email directly or view it on GitLab: https://gitlab.com/gnutls/gnutls/-/issues/954
You're receiving this email because of your account on gitlab.com.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.gnupg.org/pipermail/gnutls-devel/attachments/20200314/7f1f6982/attachment-0001.html>


More information about the Gnutls-devel mailing list