Skip to content

Commit 5f33aef

Browse files
committed
Merge branch 'develop'
2 parents 169e01c + e6f6ef3 commit 5f33aef

File tree

3 files changed

+18
-16
lines changed

3 files changed

+18
-16
lines changed

bootstrap.sh

Lines changed: 12 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -746,6 +746,12 @@ if [ -z "$SUDO" ]; then
746746
fi
747747

748748
echo "Looking for dependencies..."
749+
if [[ "$OS" == "mac" ]]; then
750+
if ! which brew; then
751+
NEED+=("brew")
752+
fi
753+
#NEED+=("brew-cask")
754+
fi
749755
if [ $BUILD -eq 0 ]; then
750756
NEED_VIRTUALENV=0 #Do we need a virtualenv with ansible for the controller? Never needed if we are not building ourselves
751757
else
@@ -759,13 +765,11 @@ else
759765
if which ansible-playbook; then
760766
NEED_VIRTUALENV=0
761767
elif [ $SUDO -eq 1 ]; then #we can only install ansible globally if we have root
762-
if [ "$OS" != "mac" ]; then #pip is preferred on mac
763-
if [ "$DISTRIB_ID" = "centos" ] || [ "$DISTRIB_ID" = "rhel" ]; then
764-
NEED+=("epel") #ansible is in EPEL
765-
fi
766-
NEED+=("ansible")
767-
NEED_VIRTUALENV=0
768+
if [ "$DISTRIB_ID" = "centos" ] || [ "$DISTRIB_ID" = "rhel" ]; then
769+
NEED+=("epel") #ansible is in EPEL
768770
fi
771+
NEED+=("ansible")
772+
NEED_VIRTUALENV=0
769773
fi
770774
if [ $NEED_VIRTUALENV -eq 1 ]; then
771775
if ! which pip; then
@@ -787,12 +791,6 @@ if [ -z "$EDITOR" ]; then
787791
EDITOR=vi
788792
fi
789793
fi
790-
if [[ "$OS" == "mac" ]]; then
791-
if ! which brew; then
792-
NEED+=("brew")
793-
fi
794-
#NEED+=("brew-cask")
795-
fi
796794
if [ ! -z "$NEED" ]; then
797795
echo " Missing dependencies: ${NEED[@]}"
798796
fi
@@ -1102,8 +1100,8 @@ for package in ${NEED[@]}; do
11021100
cmd="sudo yum $NONINTERACTIVEFLAGS install ansible"
11031101
elif [ "$OS" = "arch" ]; then
11041102
cmd="sudo pacman $NONINTERACTIVEFLAGS -Sy ansible"
1105-
else
1106-
continue
1103+
elif [ "$OS" = "mac" ]; then
1104+
cmd="brew install ansible"
11071105
fi
11081106
echo "Ansible is required for LaMachine but not installed yet. ${bold}Install now?${normal}"
11091107
if [ ! -z "$cmd" ]; then

roles/java-core/tasks/main.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,15 @@
55
include_role:
66
name: lamachine-package-install
77
with_items:
8-
- { debian: default-jdk, redhat: java-11-openjdk, arch: jdk-openjdk }
8+
- { debian: default-11-jdk, redhat: java-11-openjdk, arch: jdk11-openjdk }
99
- { debian: maven, redhat: maven, arch: maven } #not sure if this works on CentOS
1010
loop_control:
1111
loop_var: package
1212
when: root and (ansible_distribution|lower != "macosx")
1313

1414
- name: Installing Java (Mac OS X)
1515
homebrew_cask:
16-
name: java
16+
name: java11
1717
state: present
1818
when: ansible_distribution|lower == "macosx"
1919

roles/lamachine-core/tasks/macosx.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,10 @@
2626
fail:
2727
msg: "Your homebrew installation failed to install the base packages. This most likely means Homebrew on your system is in a broken state (possibly after an OS X update?). You will have to fix this first yourself prior to installing LaMachine."
2828

29+
- name: Homebrew, enabling cask versions
30+
homebrew_tap:
31+
name: homebrew/cask-versions
32+
2933
- name: (Re)install virtualenv
3034
become: yes
3135
become_user: root

0 commit comments

Comments
 (0)