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

chore(linux): Fix bug with incomplete parameter in run-tests.sh, add dependency and update documentation #9602

Merged
merged 7 commits into from
Sep 21, 2023

Conversation

ermshiperete
Copy link
Contributor

@ermshiperete ermshiperete commented Sep 20, 2023

  • Fix crash if error is not set and we can't access the system service
  • Check existence of services directory
  • Update readme for ibus-keyman tests
  • Make ibus-keyman depend on keyman-system-service (needed for integration tests)
  • Fix meson version in keyman-system-service

@keymanapp-test-bot skip

@keymanapp-test-bot
Copy link

keymanapp-test-bot bot commented Sep 20, 2023

User Test Results

Test specification and instructions

User tests are not required

Test Artifacts

Copy link
Contributor

@darcywong00 darcywong00 left a comment

Choose a reason for hiding this comment

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

lgtm

@mcdurdin
Copy link
Member

Fix bug and update documentation

Could you come up with a more descriptive title please? 😁

When running the integration tests, we provide a test service that
replaces `keyman-system-service`. DBus needs the updated `.service`
files in the `services` directory so that it will use the test
service. This change outputs an error and returns an non-zero
exit code if it can't find the `services` directory, which usually
means that `keyman-system-service` didn't get build.
@ermshiperete ermshiperete changed the title chore(linux): Fix bug and update documentation chore(linux): Fix bugs, add dependency and update documentation Sep 21, 2023
For integration tests ibus-keyman requires keyman-system-service to
be built, so we add this as a dependency.
This change updates gha-ubuntu-packaging to a version that creates
a non-empty changelog entry. This prevents a warning during
package build.
This no longer suppresses the output of cleanup.
@ermshiperete ermshiperete merged commit 7de6472 into master Sep 21, 2023
@ermshiperete ermshiperete deleted the chore/linux/fixbug branch September 21, 2023 17:30
@keyman-server
Copy link
Collaborator

Changes in this pull request will be available for download in Keyman version 17.0.178-alpha

@mcdurdin
Copy link
Member

Hmm, 'fix bugs' is still not clear! Would be better to state which bugs this PR is fixing

@ermshiperete ermshiperete changed the title chore(linux): Fix bugs, add dependency and update documentation chore(linux): Fix bug with incomplete parameter in run-tests.sh, add dependency and update documentation Sep 26, 2023
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.

4 participants