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

feat(I18n): MSI installer French #46024

Closed
wants to merge 2 commits into from
Closed

feat(I18n): MSI installer French #46024

wants to merge 2 commits into from

Conversation

AugustinMauroy
Copy link
Member

translation of MSI installer into French.

@nodejs-github-bot nodejs-github-bot added install Issues and PRs related to the installers. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform. labels Dec 30, 2022
tools/msvs/msi/i18n/en-us.wxl Outdated Show resolved Hide resolved
tools/msvs/msi/i18n/fr-BE.wxl Outdated Show resolved Hide resolved
@bnoordhuis
Copy link
Member

See #45090 and #45138 - i18n is disabled in the installer. Merging this PR as-is won't do any good, the file isn't going to get used.

@AugustinMauroy
Copy link
Member Author

I had read the microsoft docs normally it's working. So if you can explain why it's ins't use

@bnoordhuis
Copy link
Member

I mentioned that in #45090 (comment) but tl;dr it's never been enabled / never worked.

@AugustinMauroy
Copy link
Member Author

I understand what you are saying. But the question is why because I read a big part of the microsoft docs and normally it is supposed to work

@bnoordhuis
Copy link
Member

The version of WiX (installer tool) we used at the time only supported English and German. Before you ask, I don't know what it supports now. :-)

At any rate, more work is needed to actually make this PR work. You can reverse-engineer it from #45138.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
install Issues and PRs related to the installers. needs-ci PRs that need a full CI run. tools Issues and PRs related to the tools directory. windows Issues and PRs related to the Windows platform.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants