Review of "Headphone Event" version 2

Details Page Preview

***Deprecated*** 3.12 has independent volume control. ------------------------------------------- PAUSE playing songs when headphone get unplugged. You can set icon color you like in dconf-editor GS3.8 ✔ GS3.10 support delayed due to my laptop always crash while install it. Can anyone test the code in dev branch on the GIthub Homepage below?

Extension Homepage
https://github.com/mengzhuo/gnome-shell-headphone-indicator

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

All Versions

Version Status
11 Active
10 Active
9 Active
8 Active
7 Rejected
6 Active
5 Rejected
4 Active
3 Inactive
2 Rejected
1 Rejected

Previous Reviews on this Version

nzjrs posted a review
did you test again using pulseaudio / gvc? if you are going to depend on a command line tool (amixer) check it exists and do not start this extension if it doesnt. if you are just going to call the command line tool a few times per second, what was wrong with reading /proc/asound/card0/xxx every few seconds? It would reduce the dependency on a command line tool
Meng Zhuo posted a review
@nzjrs I had test my laptop(Lenovo X200) with the latest version of PulseAudio(2.0) and Kernel (3.4), unfortunately, Jack Detection won't work after I run get_ports or get_port and neither the pacmd too. The next version had already add check exists function, but I haven't upload it yet. The previous version of this extension had been rejected because reading that file, and I quote:"I'm not actually confident to let a shell extension mess with /proc..."
nzjrs posted a review
@gcampmax: he was only reading in /proc, what was your objection? please upload your new version
Meng Zhuo posted a review
That write pemisson of that file is root, no one can change it unless authorized. Sure.
gcampax posted a review
Rejecting, newer version