Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Makefile: Use ?= for initial variable definitions(part 2) #1004

Merged
merged 1 commit into from
Jul 2, 2024

Conversation

anoopcs9
Copy link
Collaborator

@anoopcs9 anoopcs9 commented Jul 2, 2024

Iteration #2 of #725.

CEPH_IMG and CEPH_TAG are commonly used to include custom ceph images as base while building go-ceph container images for testing and development purposes.

@anoopcs9 anoopcs9 changed the title Makefile: Do not override custom variables with defaults Makefile: Use ?= for initial variable definitions Jul 2, 2024
@anoopcs9 anoopcs9 changed the title Makefile: Use ?= for initial variable definitions Makefile: Use ?= for initial variable definitions(part 2) Jul 2, 2024
@phlogistonjohn
Copy link
Collaborator

I don't use the environment this heavily for make, but OK.
In the meantime, wrt to the failing nautilus jobs do you prefer to have this PR first with an admin override of the requirements, or wait until we fix/resolve that and then come back to this topic?

@phlogistonjohn phlogistonjohn added the no-API This PR does not include any changes to the public API of a go-ceph package label Jul 2, 2024
Copy link

mergify bot commented Jul 2, 2024

This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏

Copy link
Collaborator

@phlogistonjohn phlogistonjohn left a comment

Choose a reason for hiding this comment

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

lgtm

:= is not capable of accepting values(for variables) from execution
environment. Therefore replace := with ?= for completeness.

Signed-off-by: Anoop C S <anoopcs@cryptolab.net>
@mergify mergify bot merged commit 83cd6ac into ceph:master Jul 2, 2024
15 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-API This PR does not include any changes to the public API of a go-ceph package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants