After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 355334 - gkeyfile should allow manual specification of a search path
gkeyfile should allow manual specification of a search path
Status: RESOLVED FIXED
Product: glib
Classification: Platform
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: gtkdev
gtkdev
Depends on:
Blocks: 355425
 
 
Reported: 2006-09-10 19:47 UTC by Allison Karlitskaya (desrt)
Modified: 2007-04-19 20:53 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
patch (8.66 KB, patch)
2007-03-20 01:35 UTC, William Jon McCann 		committed Details | Review
Simple test case (1.37 KB, text/plain)
2007-03-20 01:38 UTC, William Jon McCann 		  Details

Description Allison Karlitskaya (desrt) 2006-09-10 19:47:13 UTC 
it would be nice if one could have gkeyfile look in a list of directories for a specific keyfile.  this behaviour would be much like how it searches the xdg_data_dirs but would allow the programmer to give their own list of directories.

this seems like it would be an easy API to add.

you might even add two:

one that takes 'char **' and one that takes a : delimited 'char *'
Comment 1 Matthias Clasen 2006-09-14 13:24:11 UTC 
sounds not entirely unreasonable, either 

void
g_key_file_set_data_dirs (GKeyFile    *keyfile, 
                          const char **dirs)

or

g_key_file_load_from_dirs (GKeyFile   *keyfile,
                           const char *dirs)
Comment 2 William Jon McCann 2007-03-20 01:35:59 UTC 
Created attachment 84928 [details] [review]
patch

This adds:
gboolean g_key_file_load_from_dirs          (GKeyFile             *key_file,
					     const gchar	  *file,
					     const gchar	 **search_dirs,
					     gchar		 **full_path,
					     GKeyFileFlags         flags,
					     GError              **error);

And makes g_key_file_load_from_data_dirs use it internally.

What do you think?
Comment 3 William Jon McCann 2007-03-20 01:38:19 UTC 
Created attachment 84929 [details]
Simple test case
Comment 4 Matthias Clasen 2007-04-11 12:57:27 UTC 
Looks fine to commit.
Comment 5 William Jon McCann 2007-04-19 20:53:31 UTC 
Committed to trunk.  Sorry for the delay.

2007-04-19  William Jon McCann  <mccann@jhu.edu>

	* glib/gkeyfile.[ch]: (find_file_in_data_dirs),
	(g_key_file_load_from_dirs), (g_key_file_load_from_data_dirs):
	Add g_key_file_load_from_dirs for looking through a search
	path for a key-file. (#355334)