Bug 322781 - Review Request: glpi - Free IT asset management software
Summary: Review Request: glpi - Free IT asset management software
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Johan Cwiklinski
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-10-08 06:11 UTC by Remi Collet
Modified: 2007-12-22 08:25 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2007-12-22 08:25:25 UTC
Type: ---
Embargoed:
fedora: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Remi Collet 2007-10-08 06:11:48 UTC
Spec URL: http://remi.fedorapeople.org/glpi.spec
SRPM URL: http://remi.fedorapeople.org/glpi-0.70-0.1.rc1.fc8.src.rpm
Mock Log: http://remi.fedorapeople.org/glpi-build.log
Description: 
GLPI is the Information Resource-Manager with an additional Administration-
Interface. You can use it to build up a database with an inventory for your 
company (computer, software, printers...). It has enhanced functions to make
the daily life for the administrators easier, like a job-tracking-system with
mail-notification and methods to build a database with basic information 
about your network-topology.

--
This is the Release Candidate of new 0.70 version.
Database schema is freezed.
I submit it to review. RC will be pushed to "testing", final version to "stable"

Comment 1 Remi Collet 2007-10-08 06:15:51 UTC
Notes :
1/ I work with upstream on this project
2/ I use this version in production (large inventory : more than 30.000 PC now,
target is 90.000)
3/ Some "plugins" (among the >50 available) will also be proposed to review

Remi.

Comment 2 Johan Cwiklinski 2007-10-08 16:49:55 UTC
Hello,

A few remarks :
- you should use macros or commands everywhere (mkdir or %{__mkdir})
- Source0 is not correct

The package build successfully. 

rpmlint is not clean, assuming for a webapp, 'non-standard-uid' and
'non-standard-dir-perm' can safely be ignored.
Also, rpmlint complains about a 'zero-lenght' file.

Regards,
Johan

Comment 3 Remi Collet 2007-10-08 17:26:41 UTC
Thanks for the comments.

- editor_plugin.js (zero-lenght) is required in tiny_mce_gzip.js (intended for
extension)

- Source0 URL fixed
- macros fixed

Spec URL: http://remi.fedorapeople.org/glpi.spec
SRPM URL: http://remi.fedorapeople.org/glpi-0.70-0.2.rc1.fc8.src.rpm


Comment 4 Johan Cwiklinski 2007-10-08 18:00:08 UTC
Review:
+ package builds in mock.
+ rpmlint is silent for SRPM and not RPMs but it is correct.
+ source file are ok.
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ BuildRequires are proper.
+ package installed properly.
+ Macro use appears rather consistent.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ package runs correctly.

APPROVED.

Comment 5 Remi Collet 2007-10-08 18:28:12 UTC
New Package CVS Request
=======================
Package Name: glpi
Short Description: Free IT asset management software
Owners: remi
Branches: devel, F-7, FC-6, EL-5, EL-4
InitialCC: 
Cvsextras Commits: yes



Comment 6 Kevin Fenzi 2007-10-09 04:37:50 UTC
cvs done.

Comment 7 Fedora Update System 2007-10-11 01:43:57 UTC
glpi-0.70-0.2.rc1.fc7 has been pushed to the Fedora 7 testing repository.  If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update glpi'


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