{"action":"created","issue":{"url":"https://api.github.com/repos/syncthing/syncthing/issues/1326","labels_url":"https://api.github.com/repos/syncthing/syncthing/issues/1326/labels{/name}","comments_url":"https://api.github.com/repos/syncthing/syncthing/issues/1326/comments","events_url":"https://api.github.com/repos/syncthing/syncthing/issues/1326/events","html_url":"https://github.com/syncthing/syncthing/pull/1326","id":56224175,"number":1326,"title":"Some systemd tweaks","user":{"login":"rumpelsepp","id":1961699,"avatar_url":"https://avatars.githubusercontent.com/u/1961699?v=3","gravatar_id":"","url":"https://api.github.com/users/rumpelsepp","html_url":"https://github.com/rumpelsepp","followers_url":"https://api.github.com/users/rumpelsepp/followers","following_url":"https://api.github.com/users/rumpelsepp/following{/other_user}","gists_url":"https://api.github.com/users/rumpelsepp/gists{/gist_id}","starred_url":"https://api.github.com/users/rumpelsepp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rumpelsepp/subscriptions","organizations_url":"https://api.github.com/users/rumpelsepp/orgs","repos_url":"https://api.github.com/users/rumpelsepp/repos","events_url":"https://api.github.com/users/rumpelsepp/events{/privacy}","received_events_url":"https://api.github.com/users/rumpelsepp/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-02-02T11:19:07Z","updated_at":"2015-02-12T08:18:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/syncthing/syncthing/pulls/1326","html_url":"https://github.com/syncthing/syncthing/pull/1326","diff_url":"https://github.com/syncthing/syncthing/pull/1326.diff","patch_url":"https://github.com/syncthing/syncthing/pull/1326.patch"},"body":"- Removed environment file to keep the service file minimal.\r\n  \"systemctl edit syncthing.service\" does the job if somebody wants\r\n  to customize the service.\r\n- Changed \"cmdline.target\" to \"default.target\" as \"cmdline.target\"\r\n  does not exist in systemd.special:\r\n  http://www.freedesktop.org/software/systemd/man/systemd.special.html\r\n- Added a missing \"After=network.target\"."},"comment":{"url":"https://api.github.com/repos/syncthing/syncthing/issues/comments/74033332","html_url":"https://github.com/syncthing/syncthing/pull/1326#issuecomment-74033332","issue_url":"https://api.github.com/repos/syncthing/syncthing/issues/1326","id":74033332,"user":{"login":"rumpelsepp","id":1961699,"avatar_url":"https://avatars.githubusercontent.com/u/1961699?v=3","gravatar_id":"","url":"https://api.github.com/users/rumpelsepp","html_url":"https://github.com/rumpelsepp","followers_url":"https://api.github.com/users/rumpelsepp/followers","following_url":"https://api.github.com/users/rumpelsepp/following{/other_user}","gists_url":"https://api.github.com/users/rumpelsepp/gists{/gist_id}","starred_url":"https://api.github.com/users/rumpelsepp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rumpelsepp/subscriptions","organizations_url":"https://api.github.com/users/rumpelsepp/orgs","repos_url":"https://api.github.com/users/rumpelsepp/repos","events_url":"https://api.github.com/users/rumpelsepp/events{/privacy}","received_events_url":"https://api.github.com/users/rumpelsepp/received_events","type":"User","site_admin":false},"created_at":"2015-02-12T08:18:48Z","updated_at":"2015-02-12T08:18:48Z","body":"> The Description= should contain the name of the program and a very short description, what it does.\r\n\r\nAgreed.\r\n\r\n> In Description= the lower case %i should be an upper case %I (we don't need to escape here, right?)\r\n\r\nAgreed.\r\n\r\n>  Add a Documentation= line.\r\n\r\nAgreed. But we should take the sycnthing wiki instead.\r\n\r\n> 1. Should it really say After=network.target? This only says that if network.target is started, then syncthing.service should be startet after network.target. It does not tell systemd to start network.target at all when syncthing.service is started.\r\n2. I think it should rather be Wants=network.target. This tells systemd that when starting syncthing.service, network.service should be started as well.\r\n\r\nThis one should be fine as is, see here: http://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/#conceptsinsystemd\r\n\r\n> Why is it WantedBy=multi-user.target in one file and WantedBy=default.target in the other?\r\n\r\nI had used this in the first place but syncthing did not start... No idea what was wrong, imo this should be `multi-user.target` but maybe I am missing sth. here."}}