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

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

 



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@xxxxxxx  2007-02-21 17:49 EST -------
(In reply to comment #11)

> I've added %config(noreplace) as rpmlint remain silent this way.

It is better to be correct than shut rpmlint. %config(noreplace) 
seems clearly wrong to me. %config could be acceptable, however.

* the following is unusefull:
Requires(postun): /sbin/ldconfig

* /usr/bin/elektra-kdb_static
should be removed from the main package

* /usr/lib/elektra/*.la shouldn't be shipped

* the scripts would better be in %doc than in /usr/share/elektra/scripts
in my opinion. Or some could be in %doc and others in 
/usr/share/elektra/scripts

* I won't object the prefixing with elektra although I don't like
it that much. However 2 man pages became wrong:
/usr/share/man/man5/elektra-elektra.5.gz
/usr/share/man/man7/elektra-elektra.7.gz

* The *.la files should not be in elektra-static-devel

* The following is a bad idea (although it is likely that I added it):
Requires:       %{_sysconfdir}/profile.d

* to have rpmbuild -bi --short-circuit working, you should add, before
mkdir __doc and mkdir __doc-devel:

rm -rf __doc __doc-devel



-- 
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@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]