[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    Re: kdebase/kioslave/ldap
From:       David Faure <david () mandrakesoft ! com>
Date:       2001-02-28 22:39:36
[Download RAW message or body]

On Wednesday 28 February 2001 22:05, Stephan Kulow wrote:
> David Faure wrote:
> > 
> > On Wednesday 28 February 2001 14:11, CVS by coolo wrote:
> > > kdebase/kioslave/ldap configure.in.in,1.6,1.7
> > > Author: coolo
> > > Wed Feb 28 14:11:49 UTC 2001
> > > In directory cvs.kde.org:/var/tmp/cvs-serv16184
> > >
> > > Modified Files:
> > >       configure.in.in
> > > Log Message:
> > > without is more clear
> > >
> > > -[  --with-ldap=PATH        Set path for LDAP files [default=check]],
> > > +[  --without-ldap=PATH     Set path for LDAP files [default=check]],
> > 
> > "Without" and a path ??
> > Doesn't make much sense to me.
> > 
> Hmm, good point. This is a two flies with one hand thing again. If you specify
> --with the PATH has a meaning, if you specify it without PATH, it checks itself,
> if you specify --without-ldap it won't compile LDAP (because PATH is "no", but that's
> another story :)
> 
> So the most logical would be to have two options again. Or leave it as it was in 2.1
> and add docu - but --help is docu ;)

An additional dummy --without-ldap option, just to get docu for the existing
feature, would be the nicest solution IMHO.
A second option is what I was looking for when doing --help.

-- 
David FAURE, david@mandrakesoft.com, faure@kde.org
http://perso.mandrakesoft.com/~david/, http://www.konqueror.org/
KDE, Making The Future of Computing Available Today

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic