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

Preparing for 3.0 release #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Preparing for 3.0 release #26

wants to merge 1 commit into from

Conversation

elliotweiser
Copy link
Owner

New Features

Users can now define a timeout value for waiting for
softwareupdate operations to complete. By default, the role will
wait up to 5 minutes to check for updates and 20 minutes to install
the Command Line Tools package. This is generous, but it provides
some wiggle room for slow/Wi-Fi internet connections. Users can
enforce a faster installation time by reducing those values.
This feature relates to #18 and will prevent the installation
from getting stuck for too long. This is not a true fix, but
will at least save time for certain users/use-cases.

Breaking Changes

The force_install variable was renamed to clt_force_install.
The README explains the reasoning behind the prefix.

Summary of Changes

  • Move installation tasks to separate file
  • Detection of ansible_distribution is now an assertion.
  • Add Command Line Tools tmp file path to vars
  • Move handlers to tasks (include_tasks in handlers obscures names)
  • Asynchronously run softwareupdate and poll for completion
  • Update README with new default vars

Summary of Changes
------------------

* Rename role variables with `osx_clt_` prefix
* DRY up install task `when` condition by moving tasks to a `block`
* Detection of `ansible_distribution` is now an `assert`ion.
* Add Command Line Tools tmp file path to vars
* Update README with new variables as a table
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.

None yet

1 participant