Opened 11 years ago

Closed 11 years ago

#39775 closed defect (fixed)

ansible @1.2: Missing /opt/local/etc/ansible directory after install

Reported by: macports@… Owned by: blair (Blair Zajac)
Priority: Normal Milestone:
Component: ports Version: 2.1.3
Keywords: Cc:
Port: ansible

Description (last modified by larryv (Lawrence Velázquez))

I installed sysutils/ansible (http://www.ansibleworks.com/) but there was no /opt/local/etc/ansible directory created.

If one clones their git repo and does the port install using their Portfile, one gets:

/opt/local/etc/ansible/ansible.cfg
/opt/local/etc/ansible/hosts

Arguably the hosts file isn't required, because it's just examples. But it seems to me that the cfg file should be installed.

Attachments (1)

ansible_post-destroot.diff (1.1 KB) - added by macports@… 11 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by larryv (Lawrence Velázquez)

Description: modified (diff)
Owner: changed from macports-tickets@… to blair@…
Port: ansible added; sysutils/ansible removed
Summary: Missing /opt/local/etc/ansible directory after installansible @1.2: Missing /opt/local/etc/ansible directory after install

comment:2 Changed 11 years ago by blair (Blair Zajac)

Feel free to send me a diff on the Portfile and I'll commit the change. I have few cycles to devote to MacPorts these days.

Changed 11 years ago by macports@…

Attachment: ansible_post-destroot.diff added

comment:3 Changed 11 years ago by macports@…

I've attached a diff that includes a post-destroot stanza for the Portfile. It will install man pages, examples and default config. I based it on https://raw.github.com/ansible/ansible/devel/packaging/macports/sysutils/ansible/Portfile

comment:4 Changed 11 years ago by blair (Blair Zajac)

Thanks for putting the diff together.

I think we want the config file to be installed with a suffix so that upgrades to ansible don't clobber the file.

comment:5 Changed 11 years ago by blair (Blair Zajac)

Resolution: fixed
Status: newclosed

Fixed in r108818.

BTW, next time you submit a diff, please use the -u command line option, it makes it easier to apply to a file.

Note: See TracTickets for help on using tickets.