Skip to content

ls: use Unicode quotes for locale/clocale styles in UTF-8 locales - #12675

Merged
sylvestre merged 1 commit into
uutils:mainfrom
sylvestre:ls-locale-quoting-utf8
Jun 28, 2026
Merged

ls: use Unicode quotes for locale/clocale styles in UTF-8 locales#12675
sylvestre merged 1 commit into
uutils:mainfrom
sylvestre:ls-locale-quoting-utf8

Conversation

@sylvestre

Copy link
Copy Markdown
Contributor

The locale and clocale quoting styles always emitted ASCII quotes and escaped embedded apostrophes/double quotes, regardless of the locale. GNU uses Unicode quotation marks U+2018/U+2019 (keyed off LC_CTYPE) for both styles in a UTF-8 locale, leaving embedded ASCII quotes untouched.

Should make test tests/ls/quoting-utf8.sh pass.

@sylvestre
sylvestre force-pushed the ls-locale-quoting-utf8 branch 3 times, most recently from 5a33477 to 6793424 Compare June 7, 2026 19:55
The locale and clocale quoting styles always emitted ASCII quotes and
escaped embedded apostrophes/double quotes, regardless of the locale.
GNU uses Unicode quotation marks U+2018/U+2019 (keyed off LC_CTYPE) for
both styles in a UTF-8 locale, leaving embedded ASCII quotes untouched.

Should make test tests/ls/quoting-utf8.sh pass.
@sylvestre
sylvestre force-pushed the ls-locale-quoting-utf8 branch from 6793424 to 934f8fc Compare June 18, 2026 06:24
@github-actions

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/tail/tail-n0f (fails in this run but passes in the 'main' branch)
Congrats! The gnu test tests/ls/quoting-utf8 is no longer failing!

@sylvestre
sylvestre merged commit 145b743 into uutils:main Jun 28, 2026
173 of 175 checks passed
@sylvestre

Copy link
Copy Markdown
Contributor Author

simple enough, auto merge

@sylvestre
sylvestre deleted the ls-locale-quoting-utf8 branch June 28, 2026 08:23
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.

1 participant