Skip to content

Conversation

@hoffie
Copy link
Member

@hoffie hoffie commented Mar 18, 2021

This removes .qm files from git, adds them to .gitignore and changes Jamulus.pro to generate the files dynamically during build in a way which hopefully even works on ancient Qt versions.

This makes use of custom compiler support in qmake which seems to be
available (at least) as of Qt 5.5.
https://doc.qt.io/archives/qt-5.5/qmake-advanced-usage.html#adding-compilers

Tested on Linux, needs validation on Mac/Windows.

Fixes #1111

This removes .qm files from git, adds them to .gitignore and changes
Jamulus.pro to generate the files dynamically during build in a way
which hopefully even works on ancient Qt versions.

This makes use of custom compiler support in qmake which seems to be
available (at least) as of Qt 5.5.
https://doc.qt.io/archives/qt-5.5/qmake-advanced-usage.html#adding-compilers

Fixes jamulussoftware#1111
@hoffie hoffie added this to the Release 3.7.1 milestone Mar 18, 2021
@hoffie hoffie requested review from pljones and softins March 18, 2021 23:12
@hoffie hoffie self-assigned this Mar 18, 2021
@ann0see
Copy link
Member

ann0see commented Mar 19, 2021

Windows fails. Probably you'd need to use another path and can't just run lrelease same for android.

Copy link
Member

@softins softins left a comment

Choose a reason for hiding this comment

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

This needs a bit more investigation before being included. I went round the same loop a month or so ago, without success. The build gave warnings that the .qm resources could not be found, as they were mentioned in the resource file. And if I built anyway, I think the executable did not include the other languages.

Currently, building from source does not have lrelease as a dependency, and with this approach it will do. That’s ok, but a change in requirements.

@hoffie hoffie marked this pull request as draft March 19, 2021 08:26
@pljones
Copy link
Collaborator

pljones commented Mar 19, 2021

Currently, building from source does not have lrelease as a dependency, and with this approach it will do. That’s ok, but a change in requirements.

Like I said last time, I think most Qt developers -- and that should be anyone building from source -- would have lrelease.

@hoffie
Copy link
Member Author

hoffie commented Mar 21, 2021

I'm closing this for now as I'm not sure whether it's the right approach to tackle this anymore.

@hoffie hoffie closed this Mar 21, 2021
@softins softins mentioned this pull request Jul 22, 2021
5 tasks
@hoffie hoffie deleted the 1111-generate-qm-during-build branch March 19, 2022 20:18
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.

Remove QM files from source tree; generate during build process

4 participants