[Bug 235293] Review Request: adminutil - Utility library for Fedora Directory Server administration

[bugzilla at redhat.com]

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: adminutil - Utility library for Fedora Directory Server administration
Alias: adminutil

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





------- Additional Comments From rmeggins at redhat.com  2007-05-24 12:04 EST -------
(In reply to comment #27)
> (In reply to comment #26)
> > (In reply to comment #25)
> > > Well,
> > > 
> > > * Patching Makefile.am requires 
> > >   - to call "autoreconf" in %build stage
> > >   - to add "Requires: libtool"
> > >   - And please write "Patch0: ..." and "%patch0 ....."
> > 
> > I will commit the changes upstream (I am the upstream maintainer), 
> > so no patch
> > required.  There is a shell script called autogen.sh which 
> > makes sure the system
> > has the correct version of the autotools before running autoreconf.
> 
> Okay. Then if you release new tarball (1.1.2), it is not a problem.

Is it necessary to re-version from 1.1.1 to 1.1.2?  1.1.1 has not yet been
officially released.  I'd like to get all of these changes for fedora extras
into 1.1.1.

> > > * For Requires problem of mozldap-devel against cyrus-sasl-devel,
> > >   please see my comment (bug 240516 comment 2)
> So please add "BuildRequires: cyrus-sasl-devel" to this package
> (adminutil).

But I thought since adminutil does not use any cyrus-sasl include files nor link
directly to any cyrus-sasl libs, it doesn't need BuildRequires: cyrus-sasl-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.