feat(packaging): add Fedora COPR RPM with Packit publication - #360
Draft
Nan Liu (liunan-ms) wants to merge 1 commit into
Draft
Nan Liu (liunan-ms) wants to merge 1 commit into
Nan Liu (liunan-ms) wants to merge 1 commit into
Conversation
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The SRPM can include uncommitted spec content, the new internal test file violates the repository lint convention, and an existing user-guide link was removed.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
Open (3)
What changed in this PR
Adds Fedora RPM packaging, deterministic source/SRPM generation, and Packit COPR publication integrated with release preparation.
Changes:
- Adds RPM spec, Mage targets, and Fedora 43 validation.
- Configures Packit release-triggered COPR builds.
- Synchronizes release metadata and documents installation and packaging workflows.
| File | Description |
|---|---|
| README.md | Adds COPR installation guidance. |
| packaging/azldev.spec | Defines the Fedora RPM package. |
| magefiles/magerpm/magerpm.go | Adds archive and SRPM targets. |
| magefiles/magerpm/magerpm_test.go | Tests RPM helpers. |
| magefiles/magerelease/magerelease.go | Syncs release metadata. |
| magefiles/magerelease/magerelease_internal_test.go | Tests release helpers. |
| magefiles/magefile.go | Registers RPM targets. |
| docs/user/README.md | Updates user-guide navigation. |
| docs/user/how-to/install-azldev.md | Documents COPR installation. |
| docs/developer/README.md | Links packaging guidance. |
| docs/developer/how-to/releasing.md | Documents Packit releases. |
| docs/developer/how-to/package-for-copr.md | Documents COPR packaging. |
| docs/developer/how-to/get-started.md | Updates Go prerequisite. |
| .packit.yaml | Configures COPR publication. |
| .github/workflows/rpm.yml | Validates Fedora RPM builds. |
| .github/workflows/prepare-release.yml | Syncs and verifies release metadata. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| // Copyright (c) Microsoft Corporation. | ||
| // Licensed under the MIT License. | ||
|
|
||
| package magerelease |
Comment on lines
+106
to
+108
| if err := sh.Run("rpmbuild", "-bs", specPath, | ||
| "--define", "_sourcedir "+outputDir, | ||
| "--define", "_srcrpmdir "+outputDir); err != nil { |
| @@ -7,7 +7,6 @@ | |||
| - [Build a Component](./how-to/build-component.md) — build RPMs from component definitions | |||
| - [Build an Image](./how-to/build-image.md) — build and boot Azure Linux images | |||
| - [Set Up AI Coding Agents](./how-to/set-up-ai-agents.md) — emit agent skill and instruction files | |||
Add deterministic vendored RPM source generation and Fedora 43 package validation. Publish each GitHub Release to COPR via Packit, and keep the RPM spec version and changelog in sync during release preparation, with 'mage release' verifying the spec matches the changelog before tagging. Include COPR publication documentation and user installation instructions.
Nan Liu (liunan-ms)
force-pushed
the
liunan/package-azldev-copr
branch
from
September 21, 2026 23:27
5591ad7 to
a0092ff
Compare
Contributor
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The new internal test violates the repository lint convention, and documentation inconsistencies remain unresolved.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 1
| ## Prerequisites | ||
|
|
||
| - **go 1.25+** - Required for building and running the application | ||
| - **Go 1.26+** - Required for building the application |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds Fedora RPM packaging and automated COPR publication for
azldev.mage archive) and SRPM (mage srpm), always built from committedHEAD..github/workflows/rpm.yml)..packit.yaml,copr_buildon thereleasetrigger).Version/Release/%changelogfromCHANGELOG.md; both the prepare-release workflow andmage releaseverify the spec matches the changelog before tagging.Validation
mage unitandmage check allpass.mage archiveproduces a vendored archive; verifiedvendor/contents..packit.yamlvalidated with the officialpackit validate.