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

Support qtvirtualkeyboard module #74

Open
fireghostea opened this issue Sep 21, 2017 · 1 comment
Open

Support qtvirtualkeyboard module #74

fireghostea opened this issue Sep 21, 2017 · 1 comment

Comments

@fireghostea
Copy link

fireghostea commented Sep 21, 2017

i add module name in qt-modules.txt and sucessfully install that and now can run example code on my rpi
but that is support only English for supporting other language i should use "qmake-rpi" with this argument "CONFIG+=lang-all" ( base on answer in this link link )how do this? should i change ./init-qtrpi-full.sh code?

@GuillaumeLazar
Copy link
Member

Thanks for your feedback. It's a good news that the qtvirtualkeyboard module works.

Currently all Qt modules from qt-modules.txt are built in the same way without any arguments.
Check the file compile-qt-modules.sh:

  1. The for loop (end of the file) iterates on all Qt modules (on the text file) and calls the function build_qtbase if the module is qtbase otherwise it calls build_module
  2. The function build_module uses the function qmake_cmd (defined in common.sh)
  3. The function qmake_cmd performs the "real" qmake:
$ROOT/raspi/qt5/bin/qmake -r |& tee $ROOT/logs/$LOG_FILE.log

Based on your link, I guess you want to do something like this only for the "qtvirtualkeyboard" module:

$ROOT/raspi/qt5/bin/qmake "CONFIG+=lang-all" -r |& tee $ROOT/logs/$LOG_FILE.log

So I guess you can update qtrpi in multiple ways to achieve this. For example you can check if the module name is "qtvirtualkeyboard" in the file compile-qt-modules.sh and do your specifc qmake command with the arguments.

If you are proud of your enhancement, you can create a Pull Request, after a review time we will merge it on the develop branch.

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

No branches or pull requests

2 participants