{"action":"created","issue":{"url":"https://api.github.com/repos/shockone/black-screen/issues/510","repository_url":"https://api.github.com/repos/shockone/black-screen","labels_url":"https://api.github.com/repos/shockone/black-screen/issues/510/labels{/name}","comments_url":"https://api.github.com/repos/shockone/black-screen/issues/510/comments","events_url":"https://api.github.com/repos/shockone/black-screen/issues/510/events","html_url":"https://github.com/shockone/black-screen/issues/510","id":160337251,"number":510,"title":"gls doesn't parse ls_colors + no shell name","user":{"login":"nhejazi","id":8527276,"avatar_url":"https://avatars.githubusercontent.com/u/8527276?v=3","gravatar_id":"","url":"https://api.github.com/users/nhejazi","html_url":"https://github.com/nhejazi","followers_url":"https://api.github.com/users/nhejazi/followers","following_url":"https://api.github.com/users/nhejazi/following{/other_user}","gists_url":"https://api.github.com/users/nhejazi/gists{/gist_id}","starred_url":"https://api.github.com/users/nhejazi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nhejazi/subscriptions","organizations_url":"https://api.github.com/users/nhejazi/orgs","repos_url":"https://api.github.com/users/nhejazi/repos","events_url":"https://api.github.com/users/nhejazi/events{/privacy}","received_events_url":"https://api.github.com/users/nhejazi/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2016-06-15T04:44:08Z","updated_at":"2016-06-15T08:54:57Z","closed_at":null,"body":"Though this project is in early development stages, I've found that it doesn't seem to yet support parsing a manually set (in ~/.bash_profile or ~/.bashrc) LS_COLORS. Use of `ls` as an alias to `gls` returns the correct output of ls, along with the error \"gls: unparsable value for LS_COLORS environment variable\". To fix this, normally I would have LS_COLORS only set manually if the shell ID/name (from `ps -p $$` in bash) was not \"black-screen\", except that, where bash returns \"CMD -bash\" from `ps -p $$`, black-screen only returns the PID number. Any fixes for this?"},"comment":{"url":"https://api.github.com/repos/shockone/black-screen/issues/comments/226127798","html_url":"https://github.com/shockone/black-screen/issues/510#issuecomment-226127798","issue_url":"https://api.github.com/repos/shockone/black-screen/issues/510","id":226127798,"user":{"login":"shockone","id":188928,"avatar_url":"https://avatars.githubusercontent.com/u/188928?v=3","gravatar_id":"","url":"https://api.github.com/users/shockone","html_url":"https://github.com/shockone","followers_url":"https://api.github.com/users/shockone/followers","following_url":"https://api.github.com/users/shockone/following{/other_user}","gists_url":"https://api.github.com/users/shockone/gists{/gist_id}","starred_url":"https://api.github.com/users/shockone/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shockone/subscriptions","organizations_url":"https://api.github.com/users/shockone/orgs","repos_url":"https://api.github.com/users/shockone/repos","events_url":"https://api.github.com/users/shockone/events{/privacy}","received_events_url":"https://api.github.com/users/shockone/received_events","type":"User","site_admin":false},"created_at":"2016-06-15T08:54:56Z","updated_at":"2016-06-15T08:54:56Z","body":"@nhejazi, Black Screen is both, as stated in README: https://github.com/shockone/black-screen#what-is-it\r\n\r\nThis is happening because shell implementation is still pretty primitive. Here's the command in question: https://github.com/shockone/black-screen/blob/master/src/Command.ts#L41-L50\r\n\r\nIt has two issues that prevent `LS_COLORS` from being parsed correctly:\r\n\r\n* It splits `argument` by `=` and there are many `=` signs in this variable.\r\n* It doesn't remove string literal quotes and doesn't strip the semicolon.\r\n\r\nRewriting it properly depends on the undergoing work of implementing a shell parser, so I'll block this issue for now. "}}
{"actor":{"display_login":"shockone"}}