Skip to content

Update pipeline to produce zip with binaries + pdbs for addition to GitHub Releases page#310

Merged
bbonaby merged 17 commits into
mainfrom
user/bbonaby/add-wxc-pdb-to-npm-package
May 16, 2026
Merged

Update pipeline to produce zip with binaries + pdbs for addition to GitHub Releases page#310
bbonaby merged 17 commits into
mainfrom
user/bbonaby/add-wxc-pdb-to-npm-package

Conversation

@bbonaby

@bbonaby bbonaby commented May 14, 2026

Copy link
Copy Markdown
Collaborator

📖 Description

PR makes our build pipeline produce a mxc-release-binaries.zip file that can be added to the github releases page on every release. The pipeline will now also produce pdbs and add it to the zip. This is a stop gap until we have symbol publishing, Although I think we'll keep doing this for folks who want easy access to the symbols.

🔗 References

Resolves #309

🔍 Validation

Will confirm artifacts are produced from PR pipeline on success.

✅ Checklist

📋 Issue Type

  • Bug fix
  • Feature
  • Task
Microsoft Reviewers: Open in CodeFlow

Copilot AI review requested due to automatic review settings May 14, 2026 15:12

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Azure Pipelines release flow to emit a single mxc-release-binaries.zip containing per-architecture MXC executables plus symbol files, intended for upload to GitHub Releases as a stopgap until symbol publishing is available.

Changes:

  • Enables full release debuginfo in the Rust workspace so CI builds produce symbol artifacts (PDB/DWP).
  • Extends per-target build artifacts to include symbol files alongside binaries.
  • Adds a new packaging job to bundle all per-arch artifacts into mxc-release-binaries.zip, while stripping symbols from the npm SDK package.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/Cargo.toml Enables full debuginfo in release profile to generate symbols suitable for crash-dump analysis.
.azure-pipelines/templates/Rust.Build.Job.yml Copies .pdb / .dwp files into the published per-target pipeline artifacts.
.azure-pipelines/templates/Package.NpmSdk.Job.yml Sets default target artifacts and removes symbol files from the SDK’s packaged bin/ before npm pack.
.azure-pipelines/templates/Mxc.Binary.Packaging.Job.yml New job to download per-arch artifacts and archive them into mxc-release-binaries.zip for release upload.
.azure-pipelines/1ES.Build.yml Wires in the new release-binaries packaging job and relies on template defaults for target lists.

Comment thread .azure-pipelines/templates/Package.NpmSdk.Job.yml Outdated
@bbonaby bbonaby requested a review from a team May 14, 2026 21:57
@bbonaby bbonaby merged commit 81b5085 into main May 16, 2026
18 checks passed
@bbonaby bbonaby deleted the user/bbonaby/add-wxc-pdb-to-npm-package branch May 16, 2026 17:59
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.

3 participants