{"action":"created","issue":{"url":"https://api.github.com/repos/dnschneid/crouton/issues/1754","labels_url":"https://api.github.com/repos/dnschneid/crouton/issues/1754/labels{/name}","comments_url":"https://api.github.com/repos/dnschneid/crouton/issues/1754/comments","events_url":"https://api.github.com/repos/dnschneid/crouton/issues/1754/events","html_url":"https://github.com/dnschneid/crouton/pull/1754","id":77109515,"number":1754,"title":"Added a volume script with interface similar to brightness.","user":{"login":"samueldr","id":132835,"avatar_url":"https://avatars.githubusercontent.com/u/132835?v=3","gravatar_id":"","url":"https://api.github.com/users/samueldr","html_url":"https://github.com/samueldr","followers_url":"https://api.github.com/users/samueldr/followers","following_url":"https://api.github.com/users/samueldr/following{/other_user}","gists_url":"https://api.github.com/users/samueldr/gists{/gist_id}","starred_url":"https://api.github.com/users/samueldr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samueldr/subscriptions","organizations_url":"https://api.github.com/users/samueldr/orgs","repos_url":"https://api.github.com/users/samueldr/repos","events_url":"https://api.github.com/users/samueldr/events{/privacy}","received_events_url":"https://api.github.com/users/samueldr/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dnschneid/crouton/labels/audio","name":"audio","color":"eb6420"},{"url":"https://api.github.com/repos/dnschneid/crouton/labels/enhancement","name":"enhancement","color":"84b6eb"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":19,"created_at":"2015-05-16T19:29:12Z","updated_at":"2015-05-27T19:07:58Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dnschneid/crouton/pulls/1754","html_url":"https://github.com/dnschneid/crouton/pull/1754","diff_url":"https://github.com/dnschneid/crouton/pull/1754.diff","patch_url":"https://github.com/dnschneid/crouton/pull/1754.patch"},"body":"This script interfaces with same volume slider that Chrome OS uses.\r\nIt has been tested on a C720p, but not with HDMI output.\r\n\r\nThere is an issue currently with this script:\r\n  * The status of the volume chrome-side is not updated. This means that lowering the volume in crouton, then switching to Chrome OS, then lowering it again will reset it at first to where it was the last time it was changed in Chrome OS.\r\n\r\nThis script is far from minimal, it uses some ugly command pipes to get status informations, but it works and should be solid.\r\n\r\nIt might be a good idea to build a native cras client that does this by itself instead of relying on the test utility and the dbus interface."},"comment":{"url":"https://api.github.com/repos/dnschneid/crouton/issues/comments/106038263","html_url":"https://github.com/dnschneid/crouton/pull/1754#issuecomment-106038263","issue_url":"https://api.github.com/repos/dnschneid/crouton/issues/1754","id":106038263,"user":{"login":"samueldr","id":132835,"avatar_url":"https://avatars.githubusercontent.com/u/132835?v=3","gravatar_id":"","url":"https://api.github.com/users/samueldr","html_url":"https://github.com/samueldr","followers_url":"https://api.github.com/users/samueldr/followers","following_url":"https://api.github.com/users/samueldr/following{/other_user}","gists_url":"https://api.github.com/users/samueldr/gists{/gist_id}","starred_url":"https://api.github.com/users/samueldr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/samueldr/subscriptions","organizations_url":"https://api.github.com/users/samueldr/orgs","repos_url":"https://api.github.com/users/samueldr/repos","events_url":"https://api.github.com/users/samueldr/events{/privacy}","received_events_url":"https://api.github.com/users/samueldr/received_events","type":"User","site_admin":false},"created_at":"2015-05-27T19:07:58Z","updated_at":"2015-05-27T19:07:58Z","body":"I'll look into it.\r\n\r\nI would implement an \"amixer error handler\", which would somehow read the error message. Then, would spit out an error message that would contain the original error message with a notice that most errors are due to out-of-date chroots, with a (shortened?) link to the update guide.\r\n\r\n    The volume script hit an unexpected error:\r\n        > [original amixer error message]\r\n    Most of the time, this means the chroot is out of date.\r\n    Update your chroot with this guide: [link].\r\n    If the issue persists after an update, consider filing a bug report.\r\n\r\nThough, this might be verbose. I don't expect to implement this before the week-end, if you have suggestion for a better error message, please do, I'm really not fond of this one.\r\n\r\n(Maybe instead add an (hidden?) debug flag to spit out more infos like version of things used by this script to help debug issues)."}}