{"action":"created","issue":{"url":"https://api.github.com/repos/karpathy/ulogme/issues/32","repository_url":"https://api.github.com/repos/karpathy/ulogme","labels_url":"https://api.github.com/repos/karpathy/ulogme/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/karpathy/ulogme/issues/32/comments","events_url":"https://api.github.com/repos/karpathy/ulogme/issues/32/events","html_url":"https://github.com/karpathy/ulogme/pull/32","id":124638296,"number":32,"title":"Make it work under xmonad and Docker","user":{"login":"csmarosi","id":2635668,"avatar_url":"https://avatars.githubusercontent.com/u/2635668?v=3","gravatar_id":"","url":"https://api.github.com/users/csmarosi","html_url":"https://github.com/csmarosi","followers_url":"https://api.github.com/users/csmarosi/followers","following_url":"https://api.github.com/users/csmarosi/following{/other_user}","gists_url":"https://api.github.com/users/csmarosi/gists{/gist_id}","starred_url":"https://api.github.com/users/csmarosi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/csmarosi/subscriptions","organizations_url":"https://api.github.com/users/csmarosi/orgs","repos_url":"https://api.github.com/users/csmarosi/repos","events_url":"https://api.github.com/users/csmarosi/events{/privacy}","received_events_url":"https://api.github.com/users/csmarosi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-01-03T09:03:26Z","updated_at":"2016-05-09T17:36:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/karpathy/ulogme/pulls/32","html_url":"https://github.com/karpathy/ulogme/pull/32","diff_url":"https://github.com/karpathy/ulogme/pull/32.diff","patch_url":"https://github.com/karpathy/ulogme/pull/32.patch"},"body":"First, thanks for the great project!\r\n\r\nTo use it with my window manager (xmonad) and under Docker (so I can remove the network access, check also [(In)Security](https://github.com/csmarosi/dockerFiles/tree/master/ulogme_gui#insecurity)) I needed some changes (in sort: using only X11-tools).\r\n\r\nI have some strong feelings *against* this pull request, as I wanted something more elegant.\r\nIt comes with the 'works for me (TM)' warranty; in particular I don't have any lock screen/screensaver.\r\n\r\nWhether you merge it or no, I hope someone can profit form this PR."},"comment":{"url":"https://api.github.com/repos/karpathy/ulogme/issues/comments/217933556","html_url":"https://github.com/karpathy/ulogme/pull/32#issuecomment-217933556","issue_url":"https://api.github.com/repos/karpathy/ulogme/issues/32","id":217933556,"user":{"login":"csmarosi","id":2635668,"avatar_url":"https://avatars.githubusercontent.com/u/2635668?v=3","gravatar_id":"","url":"https://api.github.com/users/csmarosi","html_url":"https://github.com/csmarosi","followers_url":"https://api.github.com/users/csmarosi/followers","following_url":"https://api.github.com/users/csmarosi/following{/other_user}","gists_url":"https://api.github.com/users/csmarosi/gists{/gist_id}","starred_url":"https://api.github.com/users/csmarosi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/csmarosi/subscriptions","organizations_url":"https://api.github.com/users/csmarosi/orgs","repos_url":"https://api.github.com/users/csmarosi/repos","events_url":"https://api.github.com/users/csmarosi/events{/privacy}","received_events_url":"https://api.github.com/users/csmarosi/received_events","type":"User","site_admin":false},"created_at":"2016-05-09T17:36:44Z","updated_at":"2016-05-09T17:36:44Z","body":"I did not actually test whether I need the two sleep, but the reasoning:\r\n* I spawn many background tasks with `timeout $windowLength xinput test $id > $fileName &`\r\n* these tasks terminated by the `timeout`\r\n\r\nThere could be some race condition, so I sleep a little bit more (.5s) to wait for the files. I just wanted to make this explicit with the 2 consecutive sleeps.\r\nI have little experience, how bash works/handles these. In PR #36  there is the jobs shell built-in, which seems like the tool I should have been used.\r\n\r\nBTW, yes with this 1/17th of the keystrokes are lost :)\r\nFeel free to steal/merge it with #36 if I don't have the time for that."}}