-
Notifications
You must be signed in to change notification settings - Fork 23.8k
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
unarchive: Add example to clear the usage of extra_opts
#31873
unarchive: Add example to clear the usage of extra_opts
#31873
Conversation
extra_opts
extra_opts
Spot on. For style, I would've preferred
because it makes the transform arg a cohesive unit list item, but this is a much better explanation. |
This pr tries to clear the usage of command line options with blank space. ansible will escape passed command line options, so you can't use |
Yes, exactly, but that's not what I wrote. Here, at least, tar takes --transform ... as two args, or --transform=... as one arg. My point is that the latter is preferable for style, but former for use as an example. |
@Akasurde LGTM. |
rebuild_merge |
@kassiansun Thanks for the contribution. @daxlab @gundalow @mkrizek Thanks for review. |
SUMMARY
Add an example to clear the usage of
extra_opts
Close #26217, as I don't have original repository now.
Fixes #26173, seems that ansible changed its behavior of
extra_opts
, it's essential to make it clearer for users.ISSUE TYPE
COMPONENT NAME
files/unarchive
ANSIBLE VERSION
ADDITIONAL INFORMATION
None