Skip to content

Update the SONAME to be MAJOR.MINOR - #186

Merged
ac000 merged 1 commit into
masterfrom
sover
Jun 18, 2026
Merged

ac000 merged 1 commit into
masterfrom
sover

Conversation

@ac000

@ac000 ac000 commented Jun 18, 2026

Copy link
Copy Markdown
Owner

Currently I quite often make changes that break ABI, such as updating enums in the middle.

But rather than continuously bumping the MAJOR version, I'd rather just bump the MINOR.

Currently I quite often make changes that break ABI, such as updating
enums in the middle.

But rather than continuously bumping the MAJOR version, I'd rather just
bump the MINOR.

Signed-off-by: Andrew Clayton <ac@sigsegv.uk>
Copilot AI review requested due to automatic review settings June 18, 2026 16:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the library’s shared-object versioning scheme so the ELF SONAME tracks MAJOR.MINOR instead of only MAJOR, aligning runtime linkage with the project’s intent to bump MINOR for ABI breaks.

Changes:

  • Change SOVER from $(MAJOR) to $(MAJOR).$(MINOR) for SONAME generation.
  • Derive VERSION from SOVER so the output filename remains MAJOR.MINOR.PATCH(+...).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/Makefile
@ac000
ac000 merged commit 976956a into master Jun 18, 2026
8 checks passed
@ac000
ac000 deleted the sover branch June 18, 2026 17:03
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

Successfully merging this pull request may close these issues.

2 participants