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



--- Comment #5 from gil cattaneo <[email protected]> ---
(In reply to Zbigniew Jędrzejewski-Szmek from comment #4)
> Why apacheds-ldap-api not apache-ldap-api in the name?

i prefer use apacheds instead of apache because this package is only required
(and it also use as gId org.apache.directory.) by ApacheDS at the moment

> Why are tests disabled? Ah, no junit-test-addons... Any chance to have those
> packaged?

org.apache.directory.junit:junit-addons:0.1 is a crap of bundled code of
different projects
e.g. mycila-junit. i don't want use or import this orrible "thing"

> In the %description, no space before a colon (" : ").

Fixed

> Looks good, see minor rpmlint nitpick below.
> apacheds-ldap-api.noarch: W: spurious-executable-perm
> /usr/share/doc/apacheds-ldap-api/README.txt
> ^^^ could be fixed

Fixed

Thanks!

Spec URL: https://gil.fedorapeople.org/apacheds-ldap-api.spec
SRPM URL:
https://gil.fedorapeople.org/apacheds-ldap-api-1.0.0-0.2.M31.fc22.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
[email protected]
https://admin.fedoraproject.org/mailman/listinfo/package-review

Reply via email to