remove dependency on geerlingguy drupal console ansible role because …

[elizoller]

…composer installs it just fine

GitHub Issue: (Islandora/documentation#1599)

What does this Pull Request do?

Removes dependency on geerlingguy ansible role for drupal console

What's new?

Depending on composer for installing drupal console

How should this be tested?

Pull in this PR
run ISLANDORA_DISTRO="ubuntu/bionic64" vagrant up
let it build and make sure you have a drupal installation
ssh in vagrant ssh and make sure you can run a drupal command

Additional Notes

I didn't test this on centos, so it probably should be

Interested parties

@Islandora-Devops/committers

[mjordan]

I'll smoke test this.

[seth-shaw-unlv]

Testing this on CentOS 7 hits an error with this addition:

TASK [webserver-app : Add project's /vendor/bin to $PATH] **********************
Tuesday 01 September 2020  08:54:29 -0700 (0:00:04.522)       0:24:15.422 ***** 
fatal: [default]: FAILED! => {"changed": false, "msg": "Destination /home/vagrant/.profile does not exist !", "rc": 257}

[elizoller]

is there a centos equivalent?

[seth-shaw-unlv]

I think it is ~/bash.profile.

[seth-shaw-unlv]

Of course, now I have a separate VM running right now, so I can't check at the moment. Give me a few minutes.

[elizoller]

no worries

[seth-shaw-unlv]

Yup, ~/.bash_profile is what you need.

[elizoller]

ok, i'll add a conditional for the two options

[elizoller]

@seth-shaw-unlv see 9841a94

[mjordan]

I started building an ubuntu box earlier this morning before @elizoller pushed 470e3de (built successfully) so I'll start over with the latest changes.

[seth-shaw-unlv]

I've pulled in the updates and am running it again.

[seth-shaw-unlv]

Heh, okay, I initially said 'bash.profile' but then said 'Yep, ~/.bash_profile' implying that the first statement was correct while giving a different answer. Anyway, it is ~/.bash_profile, NOT ~/bash.profile. 🤦‍♂️

[elizoller]

agh sorry

[seth-shaw-unlv]

Giving it one more centos spin-up while I grab lunch. @mjordan want me to spin up an ubuntu as well or are you okay with the recent changes?

[mjordan]

Am in the middle of spinning up an ubuntu box now with the most recent changes. Will report back when it's done.

[seth-shaw-unlv]

CentOS 7 is 👍

[mjordan]

Ubuntu is also 👍

[mjordan]

I'm happy to merge if everyone is ready!

[mjordan]

Thanks for fixing this @elizoller!