Bug 456919 - 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
when a sound is selected with the file chooser in g-s-p, make sure to default...
Status: CLOSED RAWHIDE
Product: Fedora
Classification: Fedora
Component: control-center (Show other bugs)
rawhide
All Linux
low Severity low
: ---
: ---
Assigned To: Bastien Nocera
Fedora Extras Quality Assurance
:
Depends On:
Blocks: F10DesktopTarget
  Show dependency treegraph
 
Reported: 2008-07-28 12:37 EDT by Lennart Poettering
Modified: 2008-10-10 05:20 EDT (History)
2 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2008-10-10 05:20:45 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)
Set a default directory (926 bytes, patch)
2008-10-09 12:14 EDT, Bastien Nocera
no flags Details | Diff

  None (edit)
Description Lennart Poettering 2008-07-28 12:37:43 EDT
Summary says it all.
Comment 1 Bastien Nocera 2008-07-28 12:51:56 EDT
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-07-31 23:43:09 EDT
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 09:41:47 EDT
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 10:46:08 EDT
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 10:54:40 EDT
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 12:14:49 EDT
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 05:20:45 EDT
Fixed in 2.24.0.1-6

Note You need to log in before you can comment on or make changes to this bug.