Skip to content

Add GitHub Actions matrix build & release workflows; improve COM wrapper implementations - #1

Merged
yeetbot90 merged 1 commit into
mainfrom
codex/improve-xbox-one-emulator-and-release
May 23, 2026
Merged

yeetbot90 merged 1 commit into
mainfrom
codex/improve-xbox-one-emulator-and-release

Conversation

@yeetbot90

Copy link
Copy Markdown
Owner

Motivation

  • Add reproducible CI builds for multiple vcpkg triplets and automate release packaging for tagged versions.
  • Replace ad-hoc vcpkg invocation with a maintained action and enable caching to speed subsequent builds.
  • Harden COM wrapper implementations to correctly handle null pointers and return appropriate HRESULTs and to return wrapped ILicenseInformation objects.

Description

  • Updated .github/workflows/build.yml to trigger on pull_request, introduce a matrix for triplet: [x64-windows, x64-windows-static], use lukka/run-vcpkg@v11, add actions/cache@v4 for vcpkg_installed and build directories, switch to PowerShell shells, and build with cmake --build build --config Release --parallel while uploading per-triplet artifacts named WinDurango-build-${{ matrix.triplet }}.
  • Added a new .github/workflows/release.yml that builds the same matrix on tag pushes (v*), packages per-triplet zips into dist/, and publishes GitHub releases with softprops/action-gh-release@v2 while supporting manual workflow runs that upload artifacts.
  • Updated README.md to document the new build and release workflows and explain matrix triplets and release behavior.
  • Modified projects/WinDurango.KernelX/src/CurrentApp.cpp to validate out-parameters and return E_POINTER where appropriate, initialize out pointers to nullptr, implement GetIids, GetRuntimeClassName, GetTrustLevel with sane defaults, implement EraCurrentAppWrapper::get_LicenseInformation to return a new EraLicenseInformationWrapper instance, and improve error handling and return values across several COM methods.

Testing

  • No automated tests were executed as part of this change in the repository; CI workflow definitions were added/updated so subsequent pushes or PRs will run the matrix builds and release packaging when triggered.

Codex Task

@yeetbot90
yeetbot90 merged commit a16e234 into main May 23, 2026
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant