{"action":"created","issue":{"url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/8","repository_url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit","labels_url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/8/comments","events_url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/8/events","html_url":"https://github.com/ebu/ebu-tt-live-toolkit/issues/8","id":163273246,"number":8,"title":"graphviz dependency not documented","user":{"login":"frans-ebu","id":4577613,"avatar_url":"https://avatars.githubusercontent.com/u/4577613?v=3","gravatar_id":"","url":"https://api.github.com/users/frans-ebu","html_url":"https://github.com/frans-ebu","followers_url":"https://api.github.com/users/frans-ebu/followers","following_url":"https://api.github.com/users/frans-ebu/following{/other_user}","gists_url":"https://api.github.com/users/frans-ebu/gists{/gist_id}","starred_url":"https://api.github.com/users/frans-ebu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/frans-ebu/subscriptions","organizations_url":"https://api.github.com/users/frans-ebu/orgs","repos_url":"https://api.github.com/users/frans-ebu/repos","events_url":"https://api.github.com/users/frans-ebu/events{/privacy}","received_events_url":"https://api.github.com/users/frans-ebu/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/labels/OK%20-%20to%20merge","name":"OK - to merge","color":"c2e0c6"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-06-30T21:40:19Z","updated_at":"2016-07-01T09:45:20Z","closed_at":null,"body":"I think graphviz needs to be installed in order to see the figures in the documentation correctly, but this is not clear from the README.md.\r\n\r\n(Building sphinx gives a warning it cannot find the dot executable if the user has not installed it).\r\n\r\nI see two options:\r\n- [ ] install graphviz automatically (not easy to do platform independently?)\r\n- [x] document in README.md that the user needs to install graphviz and add the location of the dot executable to the PATH\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/comments/229905349","html_url":"https://github.com/ebu/ebu-tt-live-toolkit/issues/8#issuecomment-229905349","issue_url":"https://api.github.com/repos/ebu/ebu-tt-live-toolkit/issues/8","id":229905349,"user":{"login":"frans-ebu","id":4577613,"avatar_url":"https://avatars.githubusercontent.com/u/4577613?v=3","gravatar_id":"","url":"https://api.github.com/users/frans-ebu","html_url":"https://github.com/frans-ebu","followers_url":"https://api.github.com/users/frans-ebu/followers","following_url":"https://api.github.com/users/frans-ebu/following{/other_user}","gists_url":"https://api.github.com/users/frans-ebu/gists{/gist_id}","starred_url":"https://api.github.com/users/frans-ebu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/frans-ebu/subscriptions","organizations_url":"https://api.github.com/users/frans-ebu/orgs","repos_url":"https://api.github.com/users/frans-ebu/repos","events_url":"https://api.github.com/users/frans-ebu/events{/privacy}","received_events_url":"https://api.github.com/users/frans-ebu/received_events","type":"User","site_admin":false},"created_at":"2016-07-01T09:45:20Z","updated_at":"2016-07-01T09:45:20Z","body":"@nigelmegitt you are right.\r\n\r\n**Labels**\r\nBasically I am wondering how to signal I believe this issue can be closed, without closing it as others (e.g. you) have not reviewed it yet. I thought the merge request (when we have more commits ready to be merged) may be that review point. Happy to use any better wording/workflow.\r\n\r\n**The commit**\r\nhttps://github.com/ebu/ebu-tt-live-toolkit/commit/d41d7584d866497082a7bf05f5e8ab22f56c2afd\r\n\r\n**Make solution**\r\nOk, I see we could make a different make and make.bat, as we do for the rest.\r\n"}}
{"actor":{"display_login":"frans-ebu"}}