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

Fix OpenBSD support #999

Merged
merged 1 commit into from
Jun 28, 2024
Merged

Conversation

brad0
Copy link
Contributor

@brad0 brad0 commented Jun 28, 2024

OpenBSD does not have LC_NUMERIC locale support.

OpenBSD does not have LC_NUMERIC locale support.
@ddennedy
Copy link
Member

Please change this to draft, add all commits you need to fix the build, and then submit it for review (instead of numerous PRs).

@brad0
Copy link
Contributor Author

brad0 commented Jun 28, 2024

Please change this to draft, add all commits you need to fix the build, and then submit it for review (instead of numerous PRs).

It's the last diff I have. I submitted multiple PRs as most projects want self contained separate diffs instead of one big diff.

@ddennedy
Copy link
Member

Yeah, well, "fix the OpenBSD" is self-contained IMO, and a PR can hold multiple commits. OK, thanks for the update!

@ddennedy ddennedy added this to the v7.26.0 milestone Jun 28, 2024
@ddennedy ddennedy merged commit 9bf3022 into mltframework:master Jun 28, 2024
6 checks passed
@brad0
Copy link
Contributor Author

brad0 commented Jun 28, 2024

Yeah, well, "fix the OpenBSD" is self-contained IMO, and a PR can hold multiple commits. OK, thanks for the update!

The commit message was poor, I admit that and I was going to update it to something better.

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.

None yet

2 participants