Skip to content

chore(ci): update DMG and ZIP file names in GUI CI workflow#978

Open
lgecse wants to merge 1 commit intomainfrom
fix-gui-asset-names
Open

chore(ci): update DMG and ZIP file names in GUI CI workflow#978
lgecse wants to merge 1 commit intomainfrom
fix-gui-asset-names

Conversation

@lgecse
Copy link
Member

@lgecse lgecse commented Feb 23, 2026

Fixes #966

Updates dir gui asset names to follow the dirctl pattern in releases.

Platform Old asset name New asset name
macOS (Apple Silicon) AGNTCY-Directory.dmg dir-gui-darwin-arm64.dmg
Windows (x64) agntcy-directory-windows.zip dir-gui-windows-amd64.zip

Signed-off-by: Laszlo Gecse <lgecse@cisco.com>
@lgecse lgecse requested a review from a team as a code owner February 23, 2026 17:58
@github-actions
Copy link
Contributor

The latest Buf updates on your PR. Results from workflow Buf CI / verify-proto (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed⏩ skipped⏩ skipped✅ passedFeb 23, 2026, 5:58 PM

@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-49 lines label Feb 23, 2026
Copy link
Member

@arpad-csepi arpad-csepi left a comment

Choose a reason for hiding this comment

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

As I see from the latest GUI release supports both x86_64 and arm64, so the universal architecture for the macOS artifacts might be better.

-srcfolder "build/macos/Build/Products/Release/dmg_staging" \
-ov -format UDZO \
"AGNTCY-Directory.dmg"
"dir-gui-darwin-arm64.dmg"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
"dir-gui-darwin-arm64.dmg"
"dir-gui-darwin-universal.dmg"

Comment on lines +157 to +158
name: dir-gui-darwin-arm64
path: gui/dir-gui-darwin-arm64.dmg
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name: dir-gui-darwin-arm64
path: gui/dir-gui-darwin-arm64.dmg
name: dir-gui-darwin-universal
path: gui/dir-gui-darwin-universal.dmg

Comment on lines +166 to +167
gh release create "${{ github.ref_name }}" dir-gui-darwin-arm64.dmg --generate-notes || \
gh release upload "${{ github.ref_name }}" dir-gui-darwin-arm64.dmg --clobber
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
gh release create "${{ github.ref_name }}" dir-gui-darwin-arm64.dmg --generate-notes || \
gh release upload "${{ github.ref_name }}" dir-gui-darwin-arm64.dmg --clobber
gh release create "${{ github.ref_name }}" dir-gui-darwin-universal.dmg --generate-notes || \
gh release upload "${{ github.ref_name }}" dir-gui-darwin-universal.dmg --clobber

@lgecse
Copy link
Member Author

lgecse commented Feb 24, 2026

Thanks for the review, I agree. Will adjust today.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-49 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardize GUI release asset names

2 participants