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

Automation test fix #32

Merged
merged 3 commits into from
Oct 9, 2024
Merged

Conversation

phoepsilonix
Copy link
Contributor

@phoepsilonix phoepsilonix commented Oct 4, 2024

Workflow and Script Modifications

Workflows:
  Added --privileged option to the container.
  Set QEMU_LD_PREFIX and LD_LIBRARY_PATH for aarch64.
  Manually configured QEMU binfmt for aarch64 as systemd is not running.
  Commented out bazel clean --expunge due to timeout issues.

update_mozc_deps:
  Modified the part that calls the run_bazel function for arm64
  Slightly modified the pkill command in cleanup to prevent workflow stoppage.


To avoid timeouts in bazel clean or bazel build

  add the --init option to the container startup options.

@phoepsilonix
Copy link
Contributor Author

This workflow seems to take slightly longer than the approach in #31, where related packages were prepared using Flatpak for both x86_64 and aarch64 on Jammy before executing Bazel.

@phoepsilonix phoepsilonix force-pushed the automation_test_final branch 2 times, most recently from e00bd86 to 72ae371 Compare October 4, 2024 22:23
@phoepsilonix phoepsilonix force-pushed the automation_test_final branch 5 times, most recently from 1a5004a to 0de9b6f Compare October 5, 2024 19:12
Workflow and Script Modifications

Workflows:
  Added --privileged option to the container.
  Set QEMU_LD_PREFIX and LD_LIBRARY_PATH for aarch64.
  Manually configured QEMU binfmt for aarch64 as systemd is not running.
  Commented out bazel clean --expunge due to timeout issues.

update_mozc_deps:
  Modified the part that calls the run_bazel function for arm64
  Slightly modified the pkill command in cleanup to prevent workflow stoppage.
@phoepsilonix phoepsilonix force-pushed the automation_test_final branch from 0de9b6f to b2a450f Compare October 5, 2024 19:13
add the --init option to the container startup options.

container:
  options: --init
@wengxt wengxt merged commit ac2280c into fcitx:master Oct 9, 2024
1 check passed
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.

2 participants