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

A few small updates from my experience on CentOS and Debian machines #8

Merged
merged 2 commits into from
Dec 15, 2022

Conversation

alexindrez
Copy link
Contributor

  • Replaces deprecated include with include_tasks
  • Fix type-o VNM with NVM
  • Adds config variable for nvm_link_global so folks can choose not to symlink the files
  • Adds remove tasks to clean up previously installed nvm installs from ansible-nvm role or other manual process as SELinux will fail to replace the files with symlinks

Fix type-o in task name "Create the target VNM dir"
Added variable nvm_link_global so we can disabled the symlink commands
-  these tasks fail in some scenarios and could be disabled depending on the end user's environment configuration
…ymlinks by deleting it first and when nvm_link_global is "yes"
@grzegorznowak
Copy link
Owner

that's cool, thanks for the input @alexindrez

@grzegorznowak grzegorznowak merged commit 844e6a0 into grzegorznowak:master Dec 15, 2022
@grzegorznowak
Copy link
Owner

I don't really use this role that often anymore, but it's great to see some usage and support. Keep it coming.

@alexindrez
Copy link
Contributor Author

My pleasure. Thanks for sharing this Ansible role.

The other role I used was https://github.com/stephdewit/ansible-nvm

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