Re: [gamin] configuration options

On Fri, Sep 02, 2005 at 11:17:48AM +1000, TomPh wrote:
> > >   Can you send a patch with just that subset. The one you sent 
> > > does like 3 different things, even changing the dnotify back-end
> > > configuration...
> See attached 3 patches:
> 1. reverts default debug support to 'on', provides corresponding
> configure help messages, reverts to using a '-g' compiler flag only
> when debug support is on, and adds debugging-related information to the
> 'summary report' printed at the end of the configure process.

  Hum, this makes the debugging API default to on, while I really
intend to see this activated only for people who run from CVS, reverted
that and fixed to CVs detection code to actually work (I really don't
want the debug API to be shipped in distro as this code is not safe I'm
afraid, and I don't want anybody to rely on that extra API).

> 2. fixes wrong outcomes from the --disable-kernel option


> 3. moves the inotify block ahead of the dnotify block in the configure
> script. That allows dnotify to be re-enabled if the builder enables
> inotify but disables dnotify - because, as I understand it, sometimes
> we need both.

  okay, that's clearer, applied too.
  This is commited to CVS,

    thanks a lot !


Daniel Veillard      | Red Hat Desktop team
veillard redhat com  | libxml GNOME XML XSLT toolkit | Rpmfind RPM search engine

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]