Allow use of dragonfly locale code so non-C locales work.#56
Open
Bill-Sommerfeld wants to merge 3 commits intoillumos:il-14_2_0from
Open
Allow use of dragonfly locale code so non-C locales work.#56Bill-Sommerfeld wants to merge 3 commits intoillumos:il-14_2_0from
Bill-Sommerfeld wants to merge 3 commits intoillumos:il-14_2_0from
Conversation
From Jonathan Perkin's patch in pkgsrc-extra
encoded characters. C++'s locale interface can only cope with a single char or a single wchar_t.
3bbee0f to
8aeedbc
Compare
Bill-Sommerfeld
added a commit
to Bill-Sommerfeld/oi-userland
that referenced
this pull request
Feb 10, 2026
The generic locale implementation chosen naturally by gcc's configure throws an exception if a locale other than "C" is selected. We can do better. Use the DragonflyBSD locale support with a few minor changes. From my illumos-gcc pull request, illumos/gcc#56
Bill-Sommerfeld
added a commit
to Bill-Sommerfeld/oi-userland
that referenced
this pull request
Feb 10, 2026
The generic locale implementation chosen naturally by gcc's configure throws an exception if a locale other than "C" is selected. We can do better. Use the DragonflyBSD locale support with a few minor changes. From my illumos-gcc pull request, illumos/gcc#56
AndWac
pushed a commit
to OpenIndiana/oi-userland
that referenced
this pull request
Feb 10, 2026
The generic locale implementation chosen naturally by gcc's configure throws an exception if a locale other than "C" is selected. We can do better. Use the DragonflyBSD locale support with a few minor changes. From my illumos-gcc pull request, illumos/gcc#56
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This tweaks the libstdc++
configurescript to let you pass--enable-clocale=dragonflyto select the dragonfly locale code; this will only work on illumos if you also have the illumos-gate changes in https://code.illumos.org/c/illumos-gate/+/4242The
ctype_members.ccpatch is from Jonathan Perkin's patch in pkgsrc-extra and is needed to avoid undefined symbols.