Bug 456919

Summary: when a sound is selected with the file chooser in g-s-p, make sure to default to /usr/share/sounds if that dir exists
Product: [Fedora] Fedora Reporter: Lennart Poettering <lpoetter>
Component: control-centerAssignee: Bastien Nocera <bnocera>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: rawhideCC: mclasen, rstrode
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-10-10 09:20:45 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 465130    
Attachments:
Description Flags
Set a default directory none

Description Lennart Poettering 2008-07-28 16:37:43 UTC
Summary says it all.

Comment 1 Bastien Nocera 2008-07-28 16:51:56 UTC
When you manage to say all that in the subject, it means it's too long.

But the problem is that we use XDG_DATA_DIRS. Which one of the directories
should we select? Or should we force using $(datadir)/sounds?

Comment 2 Matthias Clasen 2008-08-01 03:43:09 UTC
Since you add only /usr/share/sounds to the shortcuts list, not all
$XDG_DATA_DIRS/sound, it would seem logical to open the file chooser in that
directory too.

Comment 3 Lennart Poettering 2008-08-06 13:41:47 UTC
The first part of XDG_DATA_DIRS perhaps? 

I guess it would even be OK to hard code that to $(datadir)/sounds since if XDG_DATA_DIRS is not set (wich it usually is not) it would default to that dir anyway (plus /usr/share/local)

Comment 4 Bastien Nocera 2008-08-08 14:46:08 UTC
The default for g_get_system_data_dirs() is:
- when XDG_DATA_DIRS isn't set:
-> /usr/local/share/:/usr/share/
- when set, well, all bets are off.

I'd hard-code it to $(datadir)/sounds myself...

Comment 5 Matthias Clasen 2008-08-08 14:54:40 UTC
Yeah /usr/share/sounds is the only sane default. 
Since that is the only place where a newly installed system will have sounds...

Comment 6 Bastien Nocera 2008-10-09 16:14:49 UTC
Created attachment 319874 [details]
Set a default directory

We use whatever we managed to insert first as a shortcut folder for the default directory. Not tested.

Comment 7 Bastien Nocera 2008-10-10 09:20:45 UTC
Fixed in 2.24.0.1-6