bug 219448

Jörg Sommer joerg@alea.gnuu.de
Wed, 11 May 2005 18:37:28 +0200


G. Milde schrieb am Wed 11. May, 14:09 (+0200):
> On 11.05.05, Paul Boekholt wrote:
> > On Tue, 10 May 2005 13:51:40 +0200, "G. Milde" <g.milde@web.de> said:
> > > + foreach (["/etc/jed.conf", "/usr/etc/jed.conf", "/usr/local/etc/j=
ed.conf"])
> > For me, this would still load jed.conf and then 00debian.sl
> ...
> > A locally installed JED should probably not load a debian-specific fi=
le just
> > because it is on Debian and a Debian JED happens to be installed as w=
ell.
> 
> This raises the more generic question: should a locally installed progr=
am
> read configuration files in /etc/ (which might belong to non-local
> programs or be just leftovers)?

I think, no.

> #ifdef UNIX
> else
> {
>    if (is_substr(JED_ROOT, "/usr/local" == 1))

Isn't JED_ROOT[[:10]] == "/usr/local" better.

>      Jed_Conf_Files = "/usr/local/etc/jed.conf";
>    else
>      Jed_Conf_Files = "/etc/jed.conf,/usr/etc/jed.conf";
>
>  [...]
>
> BTW: Is there need for /usr/etc/jed.conf or could this one leaved out
> making the code even easier by dropping the foreach loop.

Yes, it should be removed, because it is not FHS compliant.

Jörg.

-- 
And 1.1.81 is officially BugFree(tm), so if you receive any bug-reports
on it, you know they are just evil lies.
         -- Linus Torvalds