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.

Sign up for GitHub

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

Jump to bottom

Use $(shell ...) to filter the redis docker volumes #15115

Merged
merged 1 commit into from
Apr 17, 2024
Merged

Use $(shell ...) to filter the redis docker volumes #15115

merged 1 commit into from
Apr 17, 2024

Conversation

jbradberry
Copy link
Member

SUMMARY

Makefiles use $() for variable templating, so trying to use it directly as a shell subcommand doesn't work.

related #15083

ISSUE TYPE
  • Bug, Docs Fix or other nominal change
COMPONENT NAME
  • API

@jbradberry jbradberry self-assigned this Apr 17, 2024
@jbradberry
Use $(shell ...) to filter the redis docker volumes
3ab43b7 
Makefiles use $() for variable templating, so trying to use it
directly as a shell subcommand doesn't work.
@jbradberry jbradberry enabled auto-merge (rebase) April 17, 2024 20:36
@jbradberry jbradberry merged commit c0f7180 into ansible:devel Apr 17, 2024
19 of 20 checks passed
@jbradberry jbradberry deleted the fix-clean-redis-volumes branch April 17, 2024 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fosterseth fosterseth fosterseth approved these changes

@AlanCoding AlanCoding Awaiting requested review from AlanCoding

Assignees

@jbradberry jbradberry

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jbradberry @fosterseth