Note: This is a beta release of Red Hat Bugzilla 5.0. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Also email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback here.
Bug 812526 - Review Request: kolab-webadmin - Kolab Groupware Web Administration Panel [NEEDINFO]
Summary: Review Request: kolab-webadmin - Kolab Groupware Web Administration Panel
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2012-04-14 11:41 UTC by Jeroen van Meeuwen
Modified: 2014-09-24 20:18 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2014-09-24 20:18:55 UTC
rdieter: needinfo? (vanmeeuwen+fedora)

Attachments (Terms of Use)

Description Jeroen van Meeuwen 2012-04-14 11:41:23 UTC
Spec URL:
Description: Flexible user and group Web administration panel for Kolab Groupware.

Comment 3 Rex Dieter 2013-02-12 19:44:09 UTC

Comment 4 Rex Dieter 2013-02-12 20:26:14 UTC
first off, 

1.  I see this package creating 3 users with static UIDs 412, 413, 414.  Is there any justification for static uid over allowing the system to create these dynamically per examples mentioned on ?

Comment 5 Rex Dieter 2013-02-12 20:42:54 UTC
naming: ok

sources: NOT ok.

2. package MUST use verifiable sources.  
srpm source md5sum:
19ff232d654d5497c9dabe4d7f159946  kolab-webadmin-3.0.3.tar.gz

Source0 URL md5sum:
9246c9844036141e21096361147eceac  kolab-webadmin-3.0.3.tar.gz

license: ok

macros: ok

scriptlets: ok
but... I assume this is from legacy stuff to cleanup after older packaging?

if [ $1 -gt 1 ]; then
    if [ ! -L "/usr/share/kolab-webadmin/hosted/js" ...
        rm -rf /usr/share/kolab-webadmin/hosted/js >/dev/null 2>&1 || :

3.  whatever it's purpose, you SHOULD document with a comment what it's for exactly.  and maybe even consider removing or conditionalizing it for rhel only (ie, since none of these packages have been published for fedora, they are likely not needed).

otherwise, relatively simple web app here, please clarify points 1-3 for me.

Comment 6 Rex Dieter 2013-03-12 19:14:58 UTC
ping?  been about a month ...

Comment 7 Rex Dieter 2014-09-24 20:18:55 UTC
Marking as stalled review, feel free to reopen if you're ever interested in picking things up again.

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