Re: oaf patch for bug 1703

[Mathieu Lacage <mathieu eazel com>]

Maciej Stachowiak <mjs@eazel.com> writes:

> Mathieu Lacage <mathieu@eazel.com> writes:
> 
> > Maciej Stachowiak <mjs@eazel.com> writes:
> > 
> > 
> > > 
> > > 2) What environment variables get checked should be set by the config
> > >    file, not hard coded.
> > 
> > It is not trivial since the GNOME_PATH env var and the OAF_INFO_PATH 
> > env var do not have the same syntax at all: I would need:
> > 
> > <oafconfig>
> > 
> > <searchpath>
> > 	<envvar type="standard">OAF_INFO_PATH</envvar type="standard">
> > 	<envvar type="standard">OAFINFODIR</envvar type="standard">
> > 	<envvar type="gnome">GNOME_PATH</envvar type="standard">
> > 	<directory>/opt/gnome/share/oaf</directory>
> > </searchpath>
> > 
> > </oafconfig>
> > 
> > However, the above seems definitely overkill to me.
> 
> That's more or less what I had in mind (except I wouldn't call the two
> types "standard" and "gnome"; I'd call them perhaps "oaf path" and
> "prefix path").

using the config file to specify which env vars to use seems 
overkill to me but if that is what eliot believes to be the
right way(tm), let it be so.

I could also perform a sed -e 's/gnome//' on all oaf source files ;-)


Mathieu
-- 
Mathieu Lacage <mathieu@eazel.com>