Skip to content

Use a wildcard for Make variable SUBPACKAGES to represent all family packages - #1868

Merged
ulucinar merged 1 commit into
crossplane-contrib:mainfrom
ulucinar:easy-subpackages
Sep 23, 2025
Merged

Use a wildcard for Make variable SUBPACKAGES to represent all family packages#1868
ulucinar merged 1 commit into
crossplane-contrib:mainfrom
ulucinar:easy-subpackages

Conversation

@ulucinar

Copy link
Copy Markdown
Collaborator

Description of your changes

When the make variable "SUBPACKAGES" is set to the literal "*", all family members but the "monolith" are implied. Now we can run a make SUBPACKAGES='*' ... to run the target on all family members, excluding the "monolith", which is not a family.

We could as well use a sentinel value other than '*' (like all) not to deal with globbing but I preferred to use * because:

  • It's not in the domain of valid family member names or folder names and hence safe
  • * would be a good, conventional sentinel value for representing all family members.

monolith is automatically excluded from the list of family members and the config package is included.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Manually tested a target that depends on the make variable SUBPACKAGES using make SUBPACKAGES='*' ... .

…amily members

but "monolith" are implied.

- Now we can run a `make SUBPACKAGES='*' ...` to run the target on all
  family members, excluding the "monolith", which is not a family.

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>

@turkenf turkenf left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you @ulucinar, LGTM

@ulucinar
ulucinar merged commit 5f03c10 into crossplane-contrib:main Sep 23, 2025
8 checks passed
@ulucinar
ulucinar deleted the easy-subpackages branch September 23, 2025 09:26
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.

2 participants