personal_ansible_desktop_configs icon indicating copy to clipboard operation
personal_ansible_desktop_configs copied to clipboard

Replace short module names with FQCN

Open alexs77 opened this issue 3 years ago • 2 comments

Instead of using eg copy, use ansible.builtin.copy to avoid possible clashes in names.

I went through all of the Ansible YAML files and replaced the short names. This is the result of running the following shell script:

find . -type f -name "*.yml" -exec perl -pi -e 's, (pacman|timezone|locale_gen|dconf|flatpak|snap|flatpak_remote):, community.general.$1:,' '{}' '+'

find . -type f -name "*.yml" -exec perl -pi -e 's, (file|apt|service|pip|lineinfile|copy|apt_repository|apt|cron|command|template|group|user|git|import_tasks|apt_key|get_url|unarchive|stat|blockinfile|debconf|include_tasks|include_vars|package|replace):, ansible.builtin.$1:,' '{}' '+'

alexs77 avatar Nov 22 '21 20:11 alexs77

This fixes #5 .

alexs77 avatar Nov 22 '21 20:11 alexs77

One potential issue I see here, is that in my experience, some distributions do well with FQCN while it causes Ansible runs to fail on others. I haven't had a chance to dive in extremely deep, but it seems to be due to some distros (for example, older LTS releases) that haven't yet moved to newer Ansible versions.

Which distro(s) did you try the change on? Thanks!

LearnLinuxTV avatar Nov 25 '21 16:11 LearnLinuxTV

Thank you for taking the time to write this in. However, I don't accept pull requests or issues on this particular repository - reason being, it's provided as reference for a video and isn't actually useful on its own. I have to make sure it's the same as in the video. But maybe this might be a consideration for the future though!

LearnLinuxTV avatar Oct 13 '22 15:10 LearnLinuxTV