Skip to content

fix: ansible dnf deploy cargo/mdbook install issues#269

Merged
kolkhis merged 1 commit intoProfessionalLinuxUsersGroup:mainfrom
cmckee786:ansible-fix
Jun 6, 2025
Merged

fix: ansible dnf deploy cargo/mdbook install issues#269
kolkhis merged 1 commit intoProfessionalLinuxUsersGroup:mainfrom
cmckee786:ansible-fix

Conversation

@cmckee786
Copy link
Copy Markdown
Contributor

Ansible cargo community module is not immune to rust cargo install issues. Tested in a Rocky 9.5 container

Copy link
Copy Markdown
Contributor

@kolkhis kolkhis left a comment

Choose a reason for hiding this comment

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

Interesting, didn't know the argv key for ansible.builtin.command. I always just use cmd:, cool stuff!

@cmckee786
Copy link
Copy Markdown
Contributor Author

cmckee786 commented Jun 6, 2025

Saw it in one of my books I think. Fairly certain ansible.builtin.command is just the long form of cmd:, I think the long form is mainly for readability/'typing' of code.

@kolkhis
Copy link
Copy Markdown
Contributor

kolkhis commented Jun 6, 2025

Well, it's long form for command: but still best practice to specify the ansible.builtin.
I meant more like:

ansible.builtin.command:
  cmd: /usr/bin/cargo install --locked mdbook@0.4.48

Though, I know in Perl when running external commands you should always use separate arguments (like your argv) so it properly does a fork() and exec(), I wonder if ansible's python is the same way.

Anyway, I digress 🤷‍♂️

@kolkhis kolkhis merged commit 794b4f1 into ProfessionalLinuxUsersGroup:main Jun 6, 2025
@cmckee786 cmckee786 deleted the ansible-fix branch June 8, 2025 18:34
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