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 [EMAIL PROTECTED] 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
