Bug 362051

Summary:	Review Request: edsadmin - An LDAP server administration tool
Product:	[Fedora] Fedora	Reporter:	Ignacio Vazquez-Abrams <ivazqueznet>
Component:	Package Review	Assignee:	Adel Gadllah <adel.gadllah>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting
Target Milestone:	---	Flags:	adel.gadllah: fedora-review+ kevin: fedora-cvs+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2007-11-04 19:15:47 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Ignacio Vazquez-Abrams 2007-11-01 16:28:19 UTC

Spec URL: http://ivazquez.fedorapeople.org/packages/edsadmin/edsadmin.spec
SRPM URL: http://ivazquez.fedorapeople.org/packages/edsadmin/edsadmin-1.0-1.src.rpm
Description: Erudite Directory Services are intended to be the best and fastest way for an administrator to take advantage of the directory services options on Free operating systems.

The EDS Admin tool is part of that goal. It allows admins to quickly and easily connect to and manage their directory servers.

Comment 1 Adel Gadllah 2007-11-02 12:43:27 UTC

Package looks good too me, currently doing a rebuild in mock before doing real
review.
The only thing that I noticed so far that there are redundant BuildRequires and
Requires. (remove pygtk2 its already pulled in by pygtk2-libglade)

Comment 2 Adel Gadllah 2007-11-02 12:52:15 UTC

ok builds fine in mock, and rpmlint is quite. 
License is GPLv2+ not GPLv2

Comment 3 Ignacio Vazquez-Abrams 2007-11-03 01:22:21 UTC

Updated.

http://ivazquez.fedorapeople.org/packages/edsadmin/edsadmin.spec
http://ivazquez.fedorapeople.org/packages/edsadmin/edsadmin-1.0-2.src.rpm

Comment 4 Adel Gadllah 2007-11-03 12:52:16 UTC

=> Approved

Comment 5 Ignacio Vazquez-Abrams 2007-11-03 13:05:57 UTC

New Package CVS Request
=======================
Package Name: edsadmin
Short Description: An LDAP server administration tool
Owners: ivazquez
Branches: F-7 F-8
Cvsextras Commits: yes

Comment 6 Kevin Fenzi 2007-11-03 17:25:33 UTC

cvs done.

Comment 7 Ignacio Vazquez-Abrams 2007-11-04 19:15:47 UTC

Pushed to F-7 and F-8, built for F-9.