[Bug 443248] Review Request: rubygem-activeldap - LDAP APIs for Ruby

bugzilla at redhat.com bugzilla at redhat.com
Mon Apr 28 17:13:24 UTC 2008


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: rubygem-activeldap - LDAP APIs for Ruby


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





------- Additional Comments From tcallawa at redhat.com  2008-04-28 13:13 EST -------
You can simplify that %define installroot like this:

%define installroot %{buildroot}%{geminstdir}

Also, please move those macro defines to the top. It helps the spec stay clean
for others, who may not be expecting new macro defines in the middle of the spec.

-- 
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.




More information about the package-review mailing list