Skip to content

tidy up install-dependencies script a bit to be more obvious/verbose about usage - #1966

Merged
craigcomstock merged 1 commit into
cfengine:masterfrom
craigcomstock:tidy-install-dependencies-script
Oct 13, 2025
Merged

tidy up install-dependencies script a bit to be more obvious/verbose about usage#1966
craigcomstock merged 1 commit into
cfengine:masterfrom
craigcomstock:tidy-install-dependencies-script

Conversation

@craigcomstock

Copy link
Copy Markdown
Contributor

PROJECT should be community or nova, not enterprise

Print out usage message when BUILD_TYPE is not provided or one of the supported values.

@craigcomstock
craigcomstock force-pushed the tidy-install-dependencies-script branch from 80b52db to 2abeaf7 Compare October 3, 2025 14:26
@craigcomstock
craigcomstock requested a review from larsewi October 3, 2025 14:27
@craigcomstock
craigcomstock force-pushed the tidy-install-dependencies-script branch from 2abeaf7 to f4317ff Compare October 3, 2025 14:37

@larsewi larsewi 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.

🚀

…about usage

PROJECT should be community or nova, not enterprise

Print out usage message when BUILD_TYPE is not provided or one of the supported values.

Added note about where the revision file in deps-packaging is generated: build-scripts/autogen

Ticket: none
Changelog: none

Craigs-MacBook-Pro.local
@craigcomstock
craigcomstock force-pushed the tidy-install-dependencies-script branch from f4317ff to 3f94a8f Compare October 13, 2025 21:07
@craigcomstock
craigcomstock merged commit 87a4374 into cfengine:master Oct 13, 2025
1 check passed
@craigcomstock
craigcomstock deleted the tidy-install-dependencies-script branch October 13, 2025 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants