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.

Sign up for GitHub

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

Jump to bottom

Remove extra Ansible setup tasks as they're not needed with a Pip install #119

Closed
geerlingguy opened this issue May 31, 2021 · 0 comments
Closed

Remove extra Ansible setup tasks as they're not needed with a Pip install #119

geerlingguy opened this issue May 31, 2021 · 0 comments

Comments

@geerlingguy
Copy link
Owner

Seems silly to do all this work when it's not really necessary for Ansible to function properly (AFAICT):

TASK [Ensure Ansible downloads directory exists.] ******************************
changed: [127.0.0.1]

TASK [Ensure /etc/ansible directory exists.] ***********************************
changed: [127.0.0.1]

TASK [Symlink /usr/local/etc/ansible to /etc/ansible.] *************************
fatal: [127.0.0.1]: FAILED! => changed=false 
  msg: 'Error while linking: [Errno 2] No such file or directory'
  path: /usr/local/etc/ansible

The problem is /usr/local is not used on M1 Macs since Homebrew installs to /opt/homebrew by default there. But it doesn't matter anyways because the only reason for that symlink is if you set up Ansible differently, using a Homebred-provided Python version, and if you do that, then it's up to you to make the necessary changes.
gabrielpedepera pushed a commit to gabrielpedepera/mac-dev-playbook that referenced this issue Jan 6, 2022
Merge branch 'master' of github.com:geerlingguy/mac-dev-playbook
519b966 
* 'master' of github.com:geerlingguy/mac-dev-playbook: (63 commits)
  Close windows when empty. Please.
  Fixes geerlingguy#127: Replace Sequel Pro with Sequel Ace.
  update link to ChromeDriver (geerlingguy#128)
  Don't create a new window at startup on Mac in Sublime Text.
  Fixes geerlingguy#119: Remove extraneous Ansible setup tasks.
  Tweak one direction in install instructions [skip ci].
  Another tweak to the initial install instructions.
  Make first-time setup instructions slightly more succint. And correct.
  Fix CI for new collections-based world.
  Fixes geerlingguy#116: Switch from roles and dock tasks to geerlingguy.mac collection.
  Automate Sublime Text configuration (geerlingguy#117)
  Add /opt/homebrew/bin to example PATH in install instructions.
  After merging geerlingguy#107, can drop -i inventory.
  Update tasks/terminal.yml
  remove remove double double code code
  cleanup reuirements on request
  cleanup persoal crud
  cleanup persoal crud
  cleanup persoal crud
  Fix formatting from previous commit.
  ...
fabiogomezdiaz pushed a commit to fabiogomezdiaz/mac-dev-playbook that referenced this issue Mar 7, 2022
@geerlingguy
Fixes geerlingguy#119: Remove extraneous Ansible setup tasks.
2d5a0b0
fabiogomezdiaz pushed a commit to fabiogomezdiaz/mac-dev-playbook that referenced this issue Jan 23, 2024
@geerlingguy @fabiogomezdiaz
Fixes geerlingguy#119: Remove extraneous Ansible setup tasks.
800a091
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@geerlingguy