andrewrothstein / ansible-miniconda

MIT License
28 stars 26 forks source link

Check for conda curl before attempting uninstall #9

Closed clintval closed 6 years ago

clintval commented 6 years ago

Closes #7 and #8.

This PR implements a check for the existence for conda curl before attempting removal. Attempting to remove conda curl if it does not exist creates an error and causes this role to not be idempotent:

$ conda remove curl
Solving environment: failed

PackagesNotFoundError: The following packages are missing from the target environment:
  - curl

$ echo $?
1
clintval commented 6 years ago

Ah! Just noticed you've been working on this! Sorry for the noise. Feel free to close all out.

andrewrothstein commented 6 years ago

covered by v5.2.0