Skip to content

Commit

Permalink
Merge pull request #46 from ccdc-opensource/no-ansible-user
Browse files Browse the repository at this point in the history
Don't use `ansible_user` (NO_JIRA)
  • Loading branch information
florianpiesche authored Jun 4, 2024
2 parents bc7d849 + 34b41b4 commit 22ebdab
Showing 1 changed file with 12 additions and 8 deletions.
20 changes: 12 additions & 8 deletions tasks/Darwin.yml
Original file line number Diff line number Diff line change
@@ -1,21 +1,24 @@
---
- name: Get remote username
ansible.builtin.set_fact:
remote_user: "{{ ansible_user_id }}"

- name: Ensure user can deploy manpages
ansible.builtin.file:
path: /usr/local/share/man/man8
state: directory
recurse: yes
owner: "{{ ansible_user }}"
owner: "{{ remote_user }}"
become: true

- name: Install Homebrew + tools
ansible.builtin.include_role:
name: geerlingguy.mac.homebrew
vars:
# This is overwritten by our group_vars (cpp.build-machines)
homebrew_installed_packages:
- svn
# Compression utilities
- xz # efficient compression

- name: Ensure necessary homebrew packages are installed
community.general.homebrew:
name:
- xz
# Download utilities
- wget # for downloads, curl is part of the system
- jq
Expand All @@ -33,6 +36,7 @@
when: not cpp_buildmachine

- name: Link conan

Check notice on line 38 in tasks/Darwin.yml

View workflow job for this annotation

GitHub Actions / ansible-lint

[ansible-lint] tasks/Darwin.yml#L38

no-changed-when Commands should not change things if nothing needs doing
Raw output
tasks/Darwin.yml:38: no-changed-when Commands should not change things if nothing needs doing
ansible.builtin.command: brew link conan@1
ansible.builtin.command:
cmd: brew link conan@1
environment:
PATH: "/usr/local/bin:/usr/bin"

0 comments on commit 22ebdab

Please sign in to comment.