search

andrewrothstein / ansible-anaconda

Ansible role for installing Anaconda
MIT License
47 stars 42 forks source link

Removes unnecessary dependencies on other Ansible roles #14

Closed colin-nolan closed 7 years ago

colin-nolan commented 7 years ago

Although I agree that it's always good to be DRY, the dependency on other Ansible roles is just too much in this case.

andrewrothstein.unarchive-deps installs more packages than are minimally required: only bzip2 and tar are actually needed. And, as far as I can tell, andrewrothstein.bash is not required at all.

andrewrothstein commented 7 years ago

respectfully decline. the bash dependence came from my failed attempts to get this working on Alpine Linux when IIRC it didn't work without bash. my perspective is that these dependencies whilst perhaps excessive are small in magnitude and generally useful complements to Anaconda. feel free to fork the role and publish a less bloated version reflecting this commit. regardless, thanks for all of your contributions!