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: xqilla101 - XQilla is an XQuery and XPath 2.0 library, built on top of Xerces-C https://bugzilla.redhat.com/show_bug.cgi?id=376541 ------- Additional Comments From [EMAIL PROTECTED] 2007-12-14 07:53 EST ------- For 1.0.2-2: * Name - As said above, it maybe better that this rpm is named as xqilla10, as Fedora requests that each time you change the name, the corresponsing review request must be submitted. * xerces-c dependency - Is this what you really intend? ------------------------------------------------------------- Requires: libstdc++ >= 4.1.1 xerces-c = 2.7.0 (require 2.7.0 exactly) BuildRequires: xerces-c-devel >= 2.7.0 (require no less than 2.7.0) ------------------------------------------------------------- * Source - I recommend to use http://downloads.sourceforge.net/xqilla/XQilla-%{version}.tar.gz because with this you won't probably have to change the SourceURL when version is upgraded. * scriptlets description ------------------------------------------------------------- %clean rm -rf %{buildroot} %post -p /sbin/ldconfig %postun [A] /sbin/ldconfig [A] # xqilla [B] %files ------------------------------------------------------------- [A] When only calling /sbin/ldconfig is needed on %postun, please write in one line like -------------------------------------------------------------- %postun -p /sbin/ldconfig -------------------------------------------------------------- [B] Don't write this comment here. This comment is in %postun scriptlet. ! %_defaultdocdir - Not a issue, however I just note that all files under %_defaultdocdir are automatically marked as %doc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review