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

Update FreeBSD default version for OpenLDAP #356

Merged
merged 1 commit into from
Aug 17, 2022
Merged

Conversation

smortex
Copy link
Member

@smortex smortex commented Jul 13, 2022

FreeBSD ports are now build against OpenLDAP 2.6 by default. Update the
package names accordingly.

https://cgit.freebsd.org/ports/commit/?id=eebc8b21260631e2b3ec7f21b92d9d0879661d0e

FreeBSD ports are now build against OpenLDAP 2.6 by default.  Update the
package names accordingly.
@smortex
Copy link
Member Author

smortex commented Jul 13, 2022

Not sure how to tag this: if you keep your system up-to-date, the module will break it and this patch prevent that (so bug, should lead to bumping the patch version), but if you don't update your packages the update will break things (so backwards-incompatible, should lead to bumping the major version).

Also depending on the repo you track the moment the default change is in the past or a few month in the future 😄

@smortex smortex added the bug Something isn't working label Jul 19, 2022
@jhoblitt
Copy link
Member

I would say this is at at least a minor version bump as it is changing the installed version of a package.

@smortex
Copy link
Member Author

smortex commented Aug 17, 2022

So if it is "at least a minor bump", it will be a major one as detailed above 😄

@smortex smortex added backwards-incompatible and removed bug Something isn't working labels Aug 17, 2022
@smortex smortex requested a review from jhoblitt August 17, 2022 01:18
@smortex smortex merged commit 353fe0e into master Aug 17, 2022
@smortex smortex deleted the freebsd-default branch August 17, 2022 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants