Skip to content

Add Android cross-compilation support to CI and publish artifacts#90

Merged
nbassler merged 3 commits into
mainfrom
77-add-android-cross-compilation-to-ci-and-publish-so-artifacts
Apr 1, 2026
Merged

Add Android cross-compilation support to CI and publish artifacts#90
nbassler merged 3 commits into
mainfrom
77-add-android-cross-compilation-to-ci-and-publish-so-artifacts

Conversation

@nbassler
Copy link
Copy Markdown
Member

@nbassler nbassler commented Apr 1, 2026

closes #77

Copilot AI review requested due to automatic review settings April 1, 2026 12:17
@nbassler nbassler linked an issue Apr 1, 2026 that may be closed by this pull request
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.39%. Comparing base (c62da12) to head (2b81f08).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
+ Coverage   72.33%   75.39%   +3.06%     
==========================================
  Files          38       30       -8     
  Lines        3148     2764     -384     
  Branches      452      396      -56     
==========================================
- Hits         2277     2084     -193     
+ Misses        784      593     -191     
  Partials       87       87              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds packaging + CI release workflows so libdedx can be consumed as prebuilt artifacts (notably for Android), and updates the CMake install to support find_package(dedx CONFIG) consumers.

Changes:

  • Add Android and Windows GitHub Actions workflows that build/install and upload packaged artifacts on releases.
  • Add CMake package export/install support (dedxTargets, dedxConfig.cmake, version file) and make install paths use GNUInstallDirs.
  • Extend install smoke testing with an additional CMake-based consumer project.

Reviewed changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
CMakeLists.txt Adds install/package config generation and build toggles for examples/tests.
src/CMakeLists.txt Switches to exported targets + GNUInstallDirs install paths; installs dedxTargets.
cmake/dedxConfig.cmake.in Adds config-mode package file to expose exported targets and a dedx::dedx convenience target.
tests/install_smoke_test.cmake Adds a CMake-based consumer build/run to validate find_package usage.
tests/install_consumer_cmake/CMakeLists.txt New minimal CMake consumer project used by the smoke test.
examples/CMakeLists.txt Aligns install destination with GNUInstallDirs and normalizes formatting.
.github/workflows/build-android.yml New Android cross-compile workflow producing per-ABI packaged artifacts.
.github/workflows/build-windows.yml New Windows workflow producing a packaged install tree artifact.
.gitignore Ignores build-packaging/.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tests/install_smoke_test.cmake
Comment thread .github/workflows/build-android.yml
Comment thread .github/workflows/build-android.yml Outdated
…ate packaging metadata and improve test configurations
@nbassler nbassler mentioned this pull request Apr 1, 2026
@nbassler nbassler merged commit d7fa81a into main Apr 1, 2026
24 checks passed
@nbassler nbassler deleted the 77-add-android-cross-compilation-to-ci-and-publish-so-artifacts branch April 1, 2026 12:53
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.

Add Android cross-compilation to CI and publish .so artifacts RPM target In case it's needed, bring back packaging to CMakeLists.txt

2 participants