Halton: Note bugster bug #6915777. This bug complains that GDM is calling chmod/setfacl on /dev/audio and /dev/audioctl even on systems that do not have such a device.
I think that if you are fixing this code anyway, it would be good if you could also add some g_stat() calls to verify that the file exists, and only call chmod/setfacl if the file exists. So, can you go ahead and also fix this related bug as well? The bug is currently assigned to me, but you can reassign it to yourself. Thanks, Brian > I reworked this patch to use /usr/bin/chown rather > than /usr/gnu/bin/chown. > > On Tue, 2010-02-09 at 17:17 +0800, Halton Huo wrote: >> Hi, >> >> The attached patch is to change audio device for console and SunRay >> sessions. This allows use audio device on GDM greeter window. >> >> For Console Login: >> Run /usr/bin/setfacl or /usr/bin/chmod for /dev/audio. >> >> For SunRay Login: >> Run chown for AUDIODEV which was return value >> of /opt/SUNWut/bin/utaudio. >> http://defect.opensolaris.org/bz/show_bug.cgi?id=13570 >> >> This is an branding patch, should works only for Solaris. I'm going to >> commit this patch into spec-files/trunk. >> >> Thanks for Bob and Brian's help to work out this patch. >> >> -Halton >
