Skip to content

[Feature] Cleaning up CPack - #138

Merged
MJC598 merged 9 commits into
mainfrom
hotfix/deploy-docs-workflow
Jun 24, 2026
Merged

[Feature] Cleaning up CPack#138
MJC598 merged 9 commits into
mainfrom
hotfix/deploy-docs-workflow

Conversation

@MJC598

@MJC598 MJC598 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

What does this PR do?

This PR focuses on the CPack and installation behaviors for releases.

  • Updates the workflow release yaml
  • Updates the CPack variables to be more streamlined
  • Utilizes default RPM and Debian file naming conventions to prevent name collision
  • Cleaned up exports so they go to default (expected) locations on a system

What Wrike task is this associated with?

Checklist before merging

  • If adding a core feature, I've added related tests.
  • This is part of a product update, and I've added an explanation of what is different to the changelog.

@MJC598 MJC598 self-assigned this Jun 24, 2026
@MJC598
MJC598 requested a review from ddbaptiste as a code owner June 24, 2026 19:26
@MJC598 MJC598 added the enhancement New feature or request label Jun 24, 2026

@ddbaptiste ddbaptiste left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just some capitalization things in our description. 😅

Comment thread cmake/respondCPackOptions.cmake.in Outdated
Co-authored-by: Dimitri Baptiste <55843498+ddbaptiste@users.noreply.github.com>
Signed-off-by: Matthew Carroll <28577806+MJC598@users.noreply.github.com>
@MJC598
MJC598 requested a review from ddbaptiste June 24, 2026 20:45

@ddbaptiste ddbaptiste left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks, approved!

@MJC598
MJC598 merged commit 71ccdff into main Jun 24, 2026
6 checks passed
@MJC598
MJC598 deleted the hotfix/deploy-docs-workflow branch June 24, 2026 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants