Bug 428364 (fedora-ds-admin-cons) - Review Request: fedora-ds-admin-console: Fedora Directory Administration Server Management Console
Summary: Review Request: fedora-ds-admin-console: Fedora Directory Administration Serv...
Keywords:
Status: CLOSED RAWHIDE
Alias: fedora-ds-admin-cons
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Dennis Gilmore
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2008-01-11 01:49 UTC by Rich Megginson
Modified: 2008-04-14 16:30 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2008-04-03 18:41:47 UTC
Type: ---
Embargoed:
dennis: fedora-review+
dennis: fedora-cvs+


Attachments (Terms of Use)

Description Rich Megginson 2008-01-11 01:49:27 UTC
Spec URL: http://rmeggins.fedorapeople.org/fedora-admin-console.spec
SRPM URL: http://rmeggins.fedorapeople.org/fedora-admin-console-1.1.1-1.src.rpm
Description: A Java based remote management console used for Managing Fedora
Directory Server.

Builds in mock, both f-8 i386 and x86_64.

rpmlint says nothing

Comment 1 Dennis Gilmore 2008-01-12 20:53:08 UTC
ill take this

Comment 2 Dennis Gilmore 2008-01-16 03:22:36 UTC
source does not match upstream 
[dennis@bratac ~]$ sha1sum fedora-admin-console-1.1.1.tar.bz2
36fd62edfcba7838ff7e066046147a8f5d032376  fedora-admin-console-1.1.1.tar.bz2
[dennis@bratac ~]$ sha1sum fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2
a7c8d5fdf4406a107227cddc18e04931a8e6de56  fedora/SOURCES/fedora-admin-a7c8d5fdf4406a107227cddc18e04931a8e6de56  fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2

Comment 3 Yanko Kaneti 2008-01-16 03:59:16 UTC
IMHO fedora-admin-console is a little too grand a name.
fedora-ds-admin-console ?
Same goes for the other too generally named packages at
directory.fedoraproject.org/sources/

Comment 4 Rich Megginson 2008-01-16 15:46:27 UTC
(In reply to comment #3)
> IMHO fedora-admin-console is a little too grand a name.
> fedora-ds-admin-console ?
> Same goes for the other too generally named packages at
> directory.fedoraproject.org/sources/

Ok, so rename
fedora-admin-console -> fedora-ds-admin-console
but
fedora-ds-console -> fedora-ds-base-console?
idm-console-framework -> fedora-ds-idm-console-framework????
fedora-idm-console -> ???

One thing is that idm-console-framework and fedora-idm-console will be used by
other projects in the near future.  So I don't want to name them
fedora-ds-something.

Comment 5 Rich Megginson 2008-01-16 15:51:50 UTC
(In reply to comment #2)
> source does not match upstream 
> [dennis@bratac ~]$ sha1sum fedora-admin-console-1.1.1.tar.bz2
> 36fd62edfcba7838ff7e066046147a8f5d032376  fedora-admin-console-1.1.1.tar.bz2
> [dennis@bratac ~]$ sha1sum fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2
> a7c8d5fdf4406a107227cddc18e04931a8e6de56 
fedora/SOURCES/fedora-admin-a7c8d5fdf4406a107227cddc18e04931a8e6de56 
fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2

Sorry.  The source in the srpm was from a cvs checkout - the source tarball is
from a cvs export.  I've updated the srpm with the correct source tarball -
http://rmeggins.fedorapeople.org/fedora-admin-console-1.1.1-1.src.rpm

Comment 6 Yanko Kaneti 2008-01-16 16:38:16 UTC
(In reply to comment #4)
> (In reply to comment #3)
> > IMHO fedora-admin-console is a little too grand a name.
> > fedora-ds-admin-console ?
> > Same goes for the other too generally named packages at
> > directory.fedoraproject.org/sources/
> 
> Ok, so rename
> fedora-admin-console -> fedora-ds-admin-console
> but
> fedora-ds-console -> fedora-ds-base-console?

I agree it comes off a little ugly, but I prefer ugly over potentially
confusing. Was/is "fds" as a common part taken?

> idm-console-framework -> fedora-ds-idm-console-framework????
> fedora-idm-console -> ???
> 
> One thing is that idm-console-framework and fedora-idm-console will be used by
> other projects in the near future.  So I don't want to name them
> fedora-ds-something.

the "idm" makes these distinct enough, so I don't think they need a rename.
I was mostly concerned about the combinations of
fedora,admin,util,console,directory that all seem too generic.



Comment 7 Dennis Gilmore 2008-01-16 16:53:55 UTC
[dennis@bratac ~]$ sha1sum fedora-admin-console-1.1.1.tar.bz2
36fd62edfcba7838ff7e066046147a8f5d032376  fedora-admin-console-1.1.1.tar.bz2
[dennis@bratac ~]$ sha1sum fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2
36fd62edfcba7838ff7e066046147a8f5d032376 
fedora/SOURCES/fedora-admin-console-1.1.1.tar.bz2

rpmlint is quiet. 

approved.

only condition is to rename as fedora-ds-admin-console  on import 

make the CVS request use the new name.

Comment 8 Rich Megginson 2008-01-16 18:39:08 UTC
SRPM: http://rmeggins.fedorapeople.org/fedora-ds-admin-console-1.1.1-2.src.rpm
Spec: http://rmeggins.fedorapeople.org/fedora-ds-admin-console.spec
Source:
http://directory.fedoraproject.org/sources/fedora-ds-admin-console-1.1.1.tar.bz2

Note two main changes: fedora-ds-admin-console Obsoletes fedora-admin-console,
in order to work with upgrades to extant versions.  fedora-ds will need to
change to Require fedora-ds-admin-console instead of fedora-admin-console

Comment 9 Rich Megginson 2008-01-16 18:40:47 UTC
New Package CVS Request
=======================
Package Name: fedora-ds-admin-console
Short Description: Fedora Directory Administration Server Management Console
Owners: rmeggins nkinder nhosoi
Branches: F-8
InitialCC: 
Cvsextras Commits:

Comment 10 Dennis Gilmore 2008-01-16 19:50:17 UTC
CVS Done

Comment 11 Orion Poplawski 2008-04-14 16:06:07 UTC
Why has this never been built?  It would be nice to get into F-9.

Comment 12 Rich Megginson 2008-04-14 16:30:07 UTC
I'm working on it.


Note You need to log in before you can comment on or make changes to this bug.