{"action":"created","issue":{"url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/65","repository_url":"https://api.github.com/repos/bitwiseworks/qtbase-os2","labels_url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/65/labels{/name}","comments_url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/65/comments","events_url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/65/events","html_url":"https://github.com/bitwiseworks/qtbase-os2/issues/65","id":393178294,"node_id":"MDU6SXNzdWUzOTMxNzgyOTQ=","number":65,"title":"Improve QProcess::startDetached","user":{"login":"dmik","id":900726,"node_id":"MDQ6VXNlcjkwMDcyNg==","avatar_url":"https://avatars0.githubusercontent.com/u/900726?v=4","gravatar_id":"","url":"https://api.github.com/users/dmik","html_url":"https://github.com/dmik","followers_url":"https://api.github.com/users/dmik/followers","following_url":"https://api.github.com/users/dmik/following{/other_user}","gists_url":"https://api.github.com/users/dmik/gists{/gist_id}","starred_url":"https://api.github.com/users/dmik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmik/subscriptions","organizations_url":"https://api.github.com/users/dmik/orgs","repos_url":"https://api.github.com/users/dmik/repos","events_url":"https://api.github.com/users/dmik/events{/privacy}","received_events_url":"https://api.github.com/users/dmik/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"dmik","id":900726,"node_id":"MDQ6VXNlcjkwMDcyNg==","avatar_url":"https://avatars0.githubusercontent.com/u/900726?v=4","gravatar_id":"","url":"https://api.github.com/users/dmik","html_url":"https://github.com/dmik","followers_url":"https://api.github.com/users/dmik/followers","following_url":"https://api.github.com/users/dmik/following{/other_user}","gists_url":"https://api.github.com/users/dmik/gists{/gist_id}","starred_url":"https://api.github.com/users/dmik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmik/subscriptions","organizations_url":"https://api.github.com/users/dmik/orgs","repos_url":"https://api.github.com/users/dmik/repos","events_url":"https://api.github.com/users/dmik/events{/privacy}","received_events_url":"https://api.github.com/users/dmik/received_events","type":"User","site_admin":false},"assignees":[{"login":"dmik","id":900726,"node_id":"MDQ6VXNlcjkwMDcyNg==","avatar_url":"https://avatars0.githubusercontent.com/u/900726?v=4","gravatar_id":"","url":"https://api.github.com/users/dmik","html_url":"https://github.com/dmik","followers_url":"https://api.github.com/users/dmik/followers","following_url":"https://api.github.com/users/dmik/following{/other_user}","gists_url":"https://api.github.com/users/dmik/gists{/gist_id}","starred_url":"https://api.github.com/users/dmik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmik/subscriptions","organizations_url":"https://api.github.com/users/dmik/orgs","repos_url":"https://api.github.com/users/dmik/repos","events_url":"https://api.github.com/users/dmik/events{/privacy}","received_events_url":"https://api.github.com/users/dmik/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2018-12-20T19:19:05Z","updated_at":"2018-12-21T20:08:58Z","closed_at":null,"author_association":"CONTRIBUTOR","body":"I'm creating a separate issue out of #64 as it's rather big.\r\n\r\nCurrently, `QProcess::startDetached` it's half-broken. It can start, but the communication between processes is not functioning to the level required by Qt (which only supports redirecting standard I/O to/from files for detached processes). Another problem is that we don't provide a PID of the started process to Qt (an OS/2 limitation).\r\n\r\nBoth problems are solvable and it partly relates to #63 (in terms that it needs some `spawn2` work in LIBCx)."},"comment":{"url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/comments/449486801","html_url":"https://github.com/bitwiseworks/qtbase-os2/issues/65#issuecomment-449486801","issue_url":"https://api.github.com/repos/bitwiseworks/qtbase-os2/issues/65","id":449486801,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0OTQ4NjgwMQ==","user":{"login":"dmik","id":900726,"node_id":"MDQ6VXNlcjkwMDcyNg==","avatar_url":"https://avatars0.githubusercontent.com/u/900726?v=4","gravatar_id":"","url":"https://api.github.com/users/dmik","html_url":"https://github.com/dmik","followers_url":"https://api.github.com/users/dmik/followers","following_url":"https://api.github.com/users/dmik/following{/other_user}","gists_url":"https://api.github.com/users/dmik/gists{/gist_id}","starred_url":"https://api.github.com/users/dmik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dmik/subscriptions","organizations_url":"https://api.github.com/users/dmik/orgs","repos_url":"https://api.github.com/users/dmik/repos","events_url":"https://api.github.com/users/dmik/events{/privacy}","received_events_url":"https://api.github.com/users/dmik/received_events","type":"User","site_admin":false},"created_at":"2018-12-21T20:08:58Z","updated_at":"2018-12-21T20:08:58Z","author_association":"CONTRIBUTOR","body":"Regarding the PID of the started process. Our cool `spawn2` provides that as well but only when P_2_THREADSAFE is used together with P_SESSION and P_UNRELATED and there is simply no other way to achieve that. Given that P_2_THREADSAFE will cause a wrapper process to be up and running as long as the detached one runs, I want to make this functionality also dependent on `QProcess::setThreadSafe`. I.e. if this flag is set, then `startDetached` will also use it and provide the PID of the started process in the result. Otherwise, it will provide 0 as it used to do in Qt4. Given that PIDs of detached processes are rarely needed in fact, this should be more than OK."}}
{"actor":{"display_login":"dmik"}}