Bug 501385 (389-admin-console)
Summary: | Review Request: 389-admin-console - 389 Admin Server Management Console | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Rich Megginson <rmeggins> |
Component: | Package Review | Assignee: | Jochen Schmitt <jochen> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | fedora-package-review, jochen, mastahnke, notting, shaines |
Target Milestone: | --- | Flags: | jochen:
fedora-review+
gwync: 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: | 2009-07-17 17:28:03 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
Rich Megginson
2009-05-18 20:59:43 UTC
Any takers? What can I do to help this along? ping What can I do to speed up the package review process? I will not be able to release the 389 directory server suite without this package. ping Good: + Basename of SPEC files matches with package name + URL tag shows to proper project home page + Package contains proper License tag + License tag states GPLv2 as a valid OSS license + Package contains verbatin copy of the license text + Could download upstream source with spectool -g + Providate/Obsoletes statement seems ok for renaming + Package contains no subpackages + Package source tar ball matches with upstream (md5sum: 68a27a56334d4ab44b3b44a81f84d8a8) + Proper BuildRoot definition + Local build works fine + Rpmlint is silent for Source rpm + Rpmlint is silent for binary rpm + Koji build works fine + Local install works fine. + %files stanza contains no duplicates + All files are owned by the package + %files contains no package which are belaongs to other packages + package contains proper %Changelog stanza Bad: - License tag states GPLv2 but the copyright state GPLv2+ as the license for the project. - Why you put the jar files into %{_datadir}/dirsrv/manual/html/java? - Because the manual seems to be very large, a separate doc subpackage make sense (In reply to comment #5) > Bad: > - License tag states GPLv2 but the copyright state GPLv2+ > as the license for the project. Source files have been changed to use plain GPLv2. New version 1.1.4 Spec: http://rmeggins.fedorapeople.org/pkgreview/389-admin-console.spec SRPM: http://rmeggins.fedorapeople.org/pkgreview/389-admin-console-1.1.4-1.src.rpm Source: http://port389.org/sources/389-admin-console-1.1.4.tar.bz2 md5sum 389-admin-console-1.1.4.tar.bz2 55c661be949c27b9ff2f754abbc2607f 389-admin-console-1.1.4.tar.bz2 sha1sum 389-admin-console-1.1.4.tar.bz229a4ba674b4f532c67ed1303162e5b232bcf9821 389-admin-console-1.1.4.tar.bz2 > - Why you put the jar files into %{_datadir}/dirsrv/manual/html/java? The way the console works is that it dynamically downloads (and caches) the jar file corresponding to the server version you are managing. This allows you to use the console to manage older and compatible newer versions of directory server and admin server. The location %{_datadir}/dirsrv/manual/html/java is where the admin server looks for the jar files when a request is made to download them to the console machine. The files under %{_datadir}/dirsrv/manual/html/java are not used directly - they are downloaded to $HOME/.fedora-idm-console (or now $HOME/.389-console) where they are loaded into the console. > - Because the manual seems to be very large, a separate doc > subpackage make sense Done. See above. Good: + Could download upstream tar ball with spectool -g + Package tar ball matches with upstream (md5sum: 55c661be949c27b9ff2f754abbc2607f) + License on the license tag matches with copyright notes of the source files + Local build works fine + Koji build works fine + Package contains several subpackage + Rpmlint is silent for source package + Rpmlint is silent for binary packages + Documentation is in a separate doc subpackage *** APPROVED *** danke New Package CVS Request ======================= Package Name: 389-admin-console Short Description: 389 Admin Server Management Console Owners: rmeggins nkinder nhosoi Branches: F-10 F-11 CVS done. imported into all branches - builds in koji New Package CVS Request ======================= Package Name: 389-admin-console Short Description: 389 Admin Server Management Console Owners: stahnma Branches: EL4, EL5 cvs done. Package Change Request ====================== Package Name: 389-admin-console New Branches: epel7 Owners: mreynolds nhosoi nkinder rmeggins Git done (by process-git-requests). |