Skip to content

solution win xp installation#1899

Merged
Be-ing merged 4 commits intomixxxdj:masterfrom
legendary-acp:legendary-acp
Nov 20, 2018
Merged

solution win xp installation#1899
Be-ing merged 4 commits intomixxxdj:masterfrom
legendary-acp:legendary-acp

Conversation

@legendary-acp
Copy link
Copy Markdown
Contributor

This pull request is for solving solution of installation in win xp.
Bug link: https://bugs.launchpad.net/mixxx/+bug/1772218
To solve this I uncommented the line mentioned in one of message in chat of above bug.

Comment thread build/wix/mixxx.wxs Outdated
Anyway, we display a warning message when installing on XP and below
Anyway, we display a warning message when installing on XP and below -->
<Condition Message="!(loc.OS2Old)">
<![CDATA[Installed OR (VersionNT >= 501)]]>
Copy link
Copy Markdown
Contributor

@JosepMaJAZ JosepMaJAZ Nov 13, 2018

Choose a reason for hiding this comment

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

change this too to 601 (windows 7) instead of 501.

Also, I just realized that we have to change the "OS2Old" message and translations in wix/Localization. We need to replace "Windows XP" for "Windows 7"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ok I'll update it

@JosepMaJAZ
Copy link
Copy Markdown
Contributor

LGTM . Let's wait for what @rryan or the others say.

@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Nov 20, 2018

Thanks for working on this. In the future, do not update translation files directly; update the translation template and ask for the template file to be pushed to Transifex, then update the translations on Transifex and pull the changes. I pushed the updated .po file translation template from this branch to Transifex. Now the translations have to be updated on Transifex for each language, otherwise, the next time someone pulls translations from Transifex for the Windows installer, the outdated translations that still say Windows XP will come back.

@Be-ing Be-ing merged commit 85e3459 into mixxxdj:master Nov 20, 2018
@Be-ing
Copy link
Copy Markdown
Contributor

Be-ing commented Nov 20, 2018

Whoops, I realized after merging that this PR was targeted at master. I rebased it onto 2.2.

@Be-ing Be-ing mentioned this pull request Dec 16, 2018
3 tasks
esbrandt added a commit to esbrandt/mixxx that referenced this pull request Dec 16, 2018
…illing strings from translations memory. They were lost in a previous commit 343246a

``tx pull -r mixxxdj-windows-installer.mixxxpot -a --minimum-perc=1``

The template and the translations became out-of-sync, since a manual push to TF following mixxxdj#1899
I do not exactly know what the issue was, probably it does not work well that strings in the language files were manually changed, but not the timestamp.

Anyway, now the template and the translations are n-sync again. Back to normal. In the future , we should NOT accept manual edits to translation files.
Either use the TF online editor or one of the many available text translation softwares, see https://mixxx.org/wiki/doku.php/internationalization#offline_translation_software
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.

3 participants