Re: pkg/34860 (New package database/ipa_sdb (database module for sysutils/ipa))

[Andrey Simonenko <simon%comsys.ntu-kpi.kiev.ua@localhost>]

The following reply was made to PR pkg/34860; it has been noted by GNATS.

From: Andrey Simonenko <simon%comsys.ntu-kpi.kiev.ua@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: OBATA Akio <obache%NetBSD.org@localhost>
Subject: Re: pkg/34860 (New package database/ipa_sdb (database module for 
sysutils/ipa))
Date: Tue, 23 Jan 2007 12:45:27 +0200

 On Mon, Jan 22, 2007 at 01:20:04PM +0000, OBATA Akio wrote:
 > The following reply was made to PR pkg/34860; it has been noted by GNATS.
 > 
 > From: OBATA Akio <obache%NetBSD.org@localhost>
 > To: gnats-bugs%NetBSD.org@localhost
 > Cc: 
 > Subject: Re: pkg/34860 (New package database/ipa_sdb (database module for 
 > sysutils/ipa))
 > Date: Mon, 22 Jan 2007 22:16:31 +0900
 > 
 >  On Mon, 22 Jan 2007 12:00:05 +0000 (UTC), Andrey Simonenko wrote:
 >  >  I found some problems with modified version of this package:
 >  >  
 >  >  1. options.mk has some mistakes with names of options (a patch
 >  >     is given below).
 >  
 >  Fixed mismatch of option names, definition v.s. usage.
 >  
 >  But why are you suggest to use option name different from sysutils/ipa?
 
 I have already thought about checking set options for sysutils/ipa
 from databases/ipa_sdb, but understood that this idea is wrong.
 
 If some feature is required from sysutils/ipa, then this does not mean
 that this feature also should be enabled in databases/ipa_sdb (or any
 other module), because modules' methods (functions) can be used only for
 parts of ipa.conf or ipastat.conf configuration file.  By default
 everything is enabled, if one decides to disable something (to save some
 memory and speed up applications), then he/she will need to decide what
 and where can be disabled, in any case before using module's methods
 ipa and ipastat check if a module has these methods, so they will not
 call methods which were disabled.