{"action":"created","issue":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/3850","labels_url":"https://api.github.com/repos/influxdb/influxdb/issues/3850/labels{/name}","comments_url":"https://api.github.com/repos/influxdb/influxdb/issues/3850/comments","events_url":"https://api.github.com/repos/influxdb/influxdb/issues/3850/events","html_url":"https://github.com/influxdb/influxdb/pull/3850","id":103266355,"number":3850,"title":"Drop /opt from system paths","user":{"login":"ktriponis","id":1522437,"avatar_url":"https://avatars.githubusercontent.com/u/1522437?v=3","gravatar_id":"","url":"https://api.github.com/users/ktriponis","html_url":"https://github.com/ktriponis","followers_url":"https://api.github.com/users/ktriponis/followers","following_url":"https://api.github.com/users/ktriponis/following{/other_user}","gists_url":"https://api.github.com/users/ktriponis/gists{/gist_id}","starred_url":"https://api.github.com/users/ktriponis/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ktriponis/subscriptions","organizations_url":"https://api.github.com/users/ktriponis/orgs","repos_url":"https://api.github.com/users/ktriponis/repos","events_url":"https://api.github.com/users/ktriponis/events{/privacy}","received_events_url":"https://api.github.com/users/ktriponis/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-08-26T13:13:11Z","updated_at":"2015-08-26T15:27:10Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/influxdb/influxdb/pulls/3850","html_url":"https://github.com/influxdb/influxdb/pull/3850","diff_url":"https://github.com/influxdb/influxdb/pull/3850.diff","patch_url":"https://github.com/influxdb/influxdb/pull/3850.patch"},"body":"While this is definitely not a bug I feel like using paths like `/etc/opt/influxdb` and `/var/opt/influxdb` is quite misleading as it is more accepted to simply use `/etc/[program name]` and `/var/[program name]`. I'm not entirely sure if this was a typo (and considering that logs are placed in `/var/log/influxdb` and not `/var/log/opt/influxdb` I'd guess this was a typo of some sort) or done intentionally but if there's no actual reason to add `/opt`, dropping it does look a little bit cleaner."},"comment":{"url":"https://api.github.com/repos/influxdb/influxdb/issues/comments/135062640","html_url":"https://github.com/influxdb/influxdb/pull/3850#issuecomment-135062640","issue_url":"https://api.github.com/repos/influxdb/influxdb/issues/3850","id":135062640,"user":{"login":"otoolep","id":536312,"avatar_url":"https://avatars.githubusercontent.com/u/536312?v=3","gravatar_id":"","url":"https://api.github.com/users/otoolep","html_url":"https://github.com/otoolep","followers_url":"https://api.github.com/users/otoolep/followers","following_url":"https://api.github.com/users/otoolep/following{/other_user}","gists_url":"https://api.github.com/users/otoolep/gists{/gist_id}","starred_url":"https://api.github.com/users/otoolep/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/otoolep/subscriptions","organizations_url":"https://api.github.com/users/otoolep/orgs","repos_url":"https://api.github.com/users/otoolep/repos","events_url":"https://api.github.com/users/otoolep/events{/privacy}","received_events_url":"https://api.github.com/users/otoolep/received_events","type":"User","site_admin":false},"created_at":"2015-08-26T15:27:10Z","updated_at":"2015-08-26T15:27:10Z","body":"Thanks for the change, but this is not actually a typo. See section 5.12 of\nthe Filesystem Hierarchy Standard:\n\nhttp://www.pathname.com/fhs/\n\nWe may revisit packaging in the future, but there are no plans right now.\n\n\nOn Wednesday, August 26, 2015, Kęstutis Triponis <notifications@github.com>\nwrote:\n\n> While this is definitely not a bug I feel like using paths like\n> /etc/opt/influxdb and /var/opt/influxdb is quite misleading as it is more\n> accepted to simply use /etc/[program name] and /var/[program name]. I'm\n> not entirely sure if this was a typo (and considering that logs are placed\n> in /var/log/influxdb and not /var/log/opt/influxdb I'd guess this was a\n> typo of some sort) or done intentionally but if there's no actual reason to\n> add /opt, dropping it does look a little bit cleaner.\n> ------------------------------\n> You can view, comment on, or merge this pull request online at:\n>\n>   https://github.com/influxdb/influxdb/pull/3850\n> Commit Summary\n>\n>    - Drop /opt from paths that already belong to some standard package\n>\n> File Changes\n>\n>    - *M* etc/config.sample.toml\n>    <https://github.com/influxdb/influxdb/pull/3850/files#diff-0> (8)\n>    - *M* package.sh\n>    <https://github.com/influxdb/influxdb/pull/3850/files#diff-1> (4)\n>    - *M* scripts/influxdb.service\n>    <https://github.com/influxdb/influxdb/pull/3850/files#diff-2> (2)\n>    - *M* scripts/init.sh\n>    <https://github.com/influxdb/influxdb/pull/3850/files#diff-3> (2)\n>\n> Patch Links:\n>\n>    - https://github.com/influxdb/influxdb/pull/3850.patch\n>    - https://github.com/influxdb/influxdb/pull/3850.diff\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/influxdb/influxdb/pull/3850>.\n>\n"}}