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

Remove entries from locales.json incompatible with punic #21003

Merged
merged 1 commit into from
May 18, 2020

Conversation

kesselb
Copy link
Contributor

@kesselb kesselb commented May 16, 2020

As reported at #20999 the list contains en_US_POSIX as locale but punic is unable to parse such a locale. If you select that locale everyone is confused

As reported at #20999 the list contains en_US_POSIX as locale but punic is unable to parse such a locale. If you select that locale everyone is confused.

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb added bug 3. to review Waiting for reviews labels May 16, 2020
@kesselb kesselb requested review from rullzer and MorrisJobke May 16, 2020 21:22
@kesselb kesselb linked an issue May 16, 2020 that may be closed by this pull request
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

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

🙈

@MorrisJobke MorrisJobke merged commit 0a21f0f into master May 18, 2020
@MorrisJobke MorrisJobke deleted the enh/noid/validate-locale branch May 18, 2020 09:48
@MorrisJobke MorrisJobke added this to the Nextcloud 20 milestone May 18, 2020
@MorrisJobke
Copy link
Member

/backport to stable19

@MorrisJobke
Copy link
Member

/backport to stable18

@MorrisJobke
Copy link
Member

/backport to stable17

@MorrisJobke
Copy link
Member

/backport to stable16

@backportbot-nextcloud
Copy link

The backport to stable17 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable16 failed. Please do this backport manually.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

en_US_POSIX locale
3 participants