toplog / toplog-forwarder-installer

The script that installs the forwarder and configuration.
MIT License
2 stars 0 forks source link

installer improvements #14

Closed PLaRoche closed 9 years ago

PLaRoche commented 9 years ago

A few things:

cameronevans commented 9 years ago

proposed: configs to /etc/logstash-forwarder/conf.d/ binary to /usr/local/logstash-forwarder/ logs to /var/log/logstash-forwarder/

The only issue I could see would be if the user already had logstash-forwarder installed. Perhaps we could make these paths configurable?

PLaRoche commented 9 years ago

Some guidance, which comes from security stuff Apple is pushing out:

"The specifics of System Integrity Protection are that no user, application, or process will be able to write files or modify files in the root System folder or the /bin, /sbin, and /usr directories, which are hidden by default in OS X’s Finder. The /usr/local folder remains accessible, however; it’s a long-running convention in Unix and variants as a place to stash material and software that individual users rely on.

El Capitan will also remove files from those directories that don’t belong to Apple. Upgrading to El Capitan will therefore disable some software you want, but also pull out old cruft that isn’t needed, and perhaps kill some lurking horrors. Only Apple installer software and software updater can modify the contents of those folders."

So putting the binaries in /usr/local/ seems to follow what they want people to do

As for people who already have it installed, detect that its in an old location and let them know that the installer will move it to the above listed places. Then the installer just need to copy the old stuff to new location.

RyanNielson commented 9 years ago

Ya, I'd say we default to those paths @cameronevans mentioned above but give them the option to change them in case they're already in use.