chore: Add docker images for building the different platforms #11397
Conversation
User Test ResultsTest specification and instructions User tests are not required |
github-actions
bot
added
linux/
common/
common/resources/
ermshiperete
force-pushed
the
chore/linux/cicontainer
branch
2 times, most recently
from
90fe8aa to
1e10c81
Compare
This updates the minimum required node version to Node 20. The latest version of Node 18 still contains the npm bug (npm/cli#7072) whereas Node 20 got updated to a npm version that contains a fix. Node 20 is known to work with our current code, so this change updates to that as an intermediate step before we investigate if we can update to Node 22 as discussed for Keyman 18.
This change allows to build a docker image that can build Keyman for Linux with test coverage reports, and installs the necessary dependencies for running integration tests.
ermshiperete
force-pushed
the
chore/linux/cicontainer
branch
from
1e10c81 to
8a4cdce
Compare
Also add test action to build.sh which builds all images and then tests them by running configure,build,test on each. Also add new dependencies to web image which Playwright requires.
These changes bring us one step further, but the build is still failing.
Setting permissions is now done in the base image.
| $HOME/tmp/gradlew --quiet && \ | ||
| rm -rf $HOME/tmp | ||
|
|
||
| ENTRYPOINT [ "/usr/bin/bashwrapper" ] |
There was a problem hiding this comment.
Does this file need to be fully deleted?
There was a problem hiding this comment.
yes, it's replaced by this PR. Splitting it in multiple Dockerfiles has the advantage that we notice if we have the documented dependencies wrong for a platform. Previously it was possible that e.g. the dependencies for Android were wrong in the documentation, but the build still worked locally because for Linux you already installed the missing dependencies. Now when you tried to set up a build agent that only builds Android things would fail.
Also some cleanup.
| std::unique_ptr<uint8_t> data(new uint8_t[0]); | ||
|
|
||
| // Execute | ||
| auto status = km_core_keyboard_load_from_blob(kmxfile.stem().c_str(), data.get(), 0, &this->keyboard); | ||
| auto status = km_core_keyboard_load_from_blob(kmxfile.stem().c_str(), nullptr, 0, &this->keyboard); |
There was a problem hiding this comment.
Why is this change in this PR?
There was a problem hiding this comment.
because the tests failed to compile in the container. Don't know why only in the container and not elsewhere, but that's why it's in this PR.
|
Changes in this pull request will be available for download in Keyman version 18.0.168-alpha |
This change adds the ability to build several docker images which can be used to build the different platforms (core, linux, web, android) and run tests. This can be used on a developer's machine but also on CI.
This change also updates the minimum node version to Node 20 as a temporary step until we try if we can use Node 22. This is necessary because the latest release of Node 18 still comes with the buggy npm that crashes the network. A better solution might have been to use
nvmin the docker images (which would allow to easily update to the latest npm), but that would have required more changes and I decided to do it this way for now in order to finish this PR.@keymanapp-test-bot skip