Adds tasks for installing dependencies on Mac #14
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This detects the 'Darwin' OS family, and uses homebrew to install the dependencies if so.
This detects the apt-get command, used for installing Ansible, and if it is missing, displays a message to install it manually first, before continuing.
Open
This is needed in the get_ansible_remote_environments function.
This checks if the coreutils provided version of ls (installed via brew `coreutils` package in `ansible/install-dependencies.yml`) exists (i.e. are we on a Mac), and uses that if available.
|
Wonderful! |
|
Thanks for accepting. More to follow soon, hopefully. I don't suppose you'd consider making this repo part of Hacktoberfest, by adding the |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This detects the 'Darwin' OS family, and uses homebrew to install the dependencies if so.
Description
Splits out the steps for installing the following dependencies between Mac & linux…
Then uses the homebrew & homebrew_cask modules to install the dependencies when on a Mac.
More changes are needed to get full Mac support, so perhaps you would prefer a new 'mac-support' development branch be targeted by this PR?
Motivation and Context
I'm using tads-boilerplate on a Mac, so using
become: yeson the standardisedpackagemodule fails with a warning about running as root, and Vagrant & VirtualBox need are casks, rather than standard homebrew packages, so I had to use homebrew_cask to install them.We would also need to update the
scripts/commands/install-dependencies.shto install Ansible in a Mac way, but for now I've added a message to manually install this, if this is acceptable?How Has This Been Tested?
I have run these on MacOS 12.6 and Ansible confirmed they were installed. I've not yet tested it will install them, however if there is an appetite for this PR I will remove those applications and re-test.
Types of changes
Checklist: