Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ABI checks have not been run since 3.1.8 #262

Closed
rdmark opened this issue Apr 2, 2023 · 4 comments
Closed

ABI checks have not been run since 3.1.8 #262

rdmark opened this issue Apr 2, 2023 · 4 comments

Comments

@rdmark
Copy link
Member

rdmark commented Apr 2, 2023

The ABI check step of the release QA process has not been run since the 3.1.8 release https://github.com/Netatalk/netatalk/wiki/Developer-Notes#abi-checking

@rdmark
Copy link
Member Author

rdmark commented Apr 4, 2023

@slowfranklin Since you were the last person committing an ABI file, can you comment on whether this QA mechanism is still valid today? I would assume so, and it seems to be a good idea to make sure we don't inadvertently change the behavior of the libatalk interface.

@slowfranklin
Copy link
Member

Yes, it's a good idea to do semantic versioning of public libraries and this was initially added to allow external consumers of libatalk safe usage, but no, as afaict there are no external consumers of libatalk I just dropped maintaing the ABI stuff at some point and we can drop this stuff. As an additional change we can then make libatalk a private library.

@rdmark
Copy link
Member Author

rdmark commented Apr 4, 2023

Now I understand the original intent. Agreed that there is little use in keeping this around. For starters I removed the ABI checking steps from the developer notes wiki page.

A few things to clean up:

  • Remove the DEVELOPER flag from makefiles
  • Remove VERSION_INFO from makefiles
  • Remove the --enable-developer configure option
  • Remove all stored abi files
  • Remove any remaining documentation

Anything else?

@rdmark
Copy link
Member Author

rdmark commented Apr 4, 2023

@slowfranklin Please see if this covers all the bases #269

@rdmark rdmark closed this as completed Apr 6, 2023
rdmark added a commit that referenced this issue Jul 18, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
rdmark added a commit that referenced this issue Jul 19, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

The ABI version has been set to "18" in the Meson build system
since release version 3.2.0.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
rdmark added a commit that referenced this issue Jul 19, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

The ABI version has been set to "18" in the Meson build system
since release version 3.2.0.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
rdmark added a commit that referenced this issue Jul 19, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

The ABI version has been set to "18" in the Meson build system
since release version 3.2.0.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
rdmark added a commit that referenced this issue Jul 20, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

The ABI version has been set to "18" in the Meson build system
since release version 3.2.0.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
rdmark added a commit that referenced this issue Jul 21, 2024
The libatalk binary API versioning scheme was removed
after discussion in GitHub #262 and effective with v3.1.16

However, since then, external consumers of libatalk have been found.
Additionally, ABI versioning allows distinguishing
3.x libatalk from 2.x libatalk in the wild.

The ABI version has been set to "18" in the Meson build system
since release version 3.2.0.

Signed-off-by: Daniel Markstedt <daniel@mindani.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants