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

Unbreak loading translations #4022

Merged
merged 1 commit into from
Dec 28, 2021
Merged

Conversation

antonio-rojas
Copy link
Contributor

Commit 18ddb9d changed SHAREDIR to point to CMAKE_INSTALL_DATADIR, which is a relative path by default. This prevents the C++ code from finding the translations at runtime.

@mgallien
Copy link
Collaborator

mgallien commented Dec 9, 2021

@antonio-rojas sorry for the delayed review due to the team being busy
coudl you have a look at the failed DCO check ?

@mgallien mgallien linked an issue Dec 9, 2021 that may be closed by this pull request
@antonio-rojas
Copy link
Contributor Author

@antonio-rojas sorry for the delayed review due to the team being busy coudl you have a look at the failed DCO check ?

Done

@mgallien mgallien requested a review from claucambra December 15, 2021 10:56
@claucambra
Copy link
Collaborator

/rebase

@allexzander
Copy link
Contributor

/rebase

Commit 18ddb9d changed SHAREDIR to point to CMAKE_INSTALL_DATADIR, which is a relative path by default. This prevents the C++ code from finding the translations at runtime.

Signed-off-by: Antonio Rojas <[email protected]>
@nextcloud-desktop-bot
Copy link

AppImage file: Nextcloud-PR-4022-9d4d11df61275d578a5ebb1478e0afcd1df96273-x86_64.AppImage

To test this change/fix you can simply download above AppImage file and test it.

Please make sure to quit your existing Nextcloud app and backup your data.

@allexzander allexzander merged commit fdba105 into nextcloud:master Dec 28, 2021
@mgallien mgallien added this to the 3.5.0 milestone Jan 8, 2022
@mgallien
Copy link
Collaborator

mgallien commented Jan 8, 2022

/backport to stable-3.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Client no longer localised
5 participants