[Bug 209906] Review Request: elektra - A key/value pair database to store software configurations

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: elektra -  A key/value pair database to store software configurations


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=209906





------- Additional Comments From pertusus at free.fr  2006-10-09 11:11 EST -------
(In reply to comment #3)

> I can avoid re-emphasizing my comments from:
> https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=187430#c15
> 
> In particular:
> - header name are too general to all allow them to installed into $(includedir)

I agree, but I'd prefer to follow upstream, and only if they don't 
cooperate start changing things.

> -  Still many warnings, which justify doubts on the package  to be installed
> into /lib rsp. /bin , /sbin

There are less warnings, most of them now corresponds with
error codes not checked, I hope that they are harmless, an audit
of the code is required to know for sure.

But elektra is a bit experimental, and in my opinion apps using it 
are being adventurous, so I am not too worried by those warnings.
If elektra is in /lib, /bin and so on it is because some apps
may use it during boot without /usr mounted, but this won't 
happen magically, apps have to explicitely use elektra, and 
at that point developper know what they are doing.

> In addition to that:
> - kbd is too general as a program name, c.f.
> http://docs.sun.com/app/docs/doc/816-0210/6m6nb7mcf?a=view

It is kdb, not kbd. But I agree that it is too generic anyway, 
but here too, I'd prefer following upstream.

> - Please explain in detail why you want to ship static libs. As you probably are
> aware about, there is a strong tendency to eliminate them.

I think static libs could be usefull, in case one want to be sure
that elektra may still be used within an app even if dynamical 
linking fails. Remember that today most of the apps reimplement
what elektra provides, so the situation today is similar with 
static linking with elektra. It doesn't meeans that I think it is 
wise to link statically against elektra in the general case, and 
the default would be dynamic linking anyway.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.