Bug 426567 - Review Request: glpi-mass-ocs-import - GLPI Plugin for OCS Massive import
Summary: Review Request: glpi-mass-ocs-import - GLPI Plugin for OCS Massive import
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2007-12-22 08:30 UTC by Remi Collet
Modified: 2008-02-19 03:20 UTC (History)
3 users (show)

Fixed In Version: 1.1-1.fc7
Clone Of:
Environment:
Last Closed: 2008-02-19 03:14:03 UTC
Type: ---
Embargoed:
fedora: fedora-review+
kevin: fedora-cvs+


Attachments (Terms of Use)

Description Remi Collet 2007-12-22 08:30:35 UTC
Spec URL: http://remi.fedorapeople.org/glpi-mass-ocs-import.spec
SRPM URL: http://remi.fedorapeople.org/glpi-mass-ocs-import-1.0-1.fc8.src.rpm
Mock Log: http://remi.fedorapeople.org/glpi-mass-ocs-import-build.log
Description: 
Plugin which allow OCS continuous synchronization and massive importation.

The extension Config panel is provided to handle the synchronization options.

--- 
rpmlint output :
glpi-mass-ocs-import.noarch: W: dangerous-command-in-%post install
glpi-mass-ocs-import.noarch: W: dangerous-command-in-%postun rm

=> creation/deletion of the (ghost file) plugin lock

GLPI web site provides a lot of useful plugins.
This is the first proposed to Review, some other (i use or/and i maintain upstream) will follow.

Comment 2 Johan Cwiklinski 2008-02-16 07:21:58 UTC
Review:
+ package builds in mock.
+ rpmlint is silent for SRPM.
+ rpmlint warnings for RPM should be safely ignored
+ 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.

APPROVED.

Comment 3 Remi Collet 2008-02-16 08:06:43 UTC
New Package CVS Request
=======================
Package Name: glpi-mass-ocs-import
Short Description: GLPI Plugin for OCS Massive import
Owners: remi
Branches: F-7 F-8 EL-4 EL-5
InitialCC: 
Cvsextras Commits: yes



Comment 4 Kevin Fenzi 2008-02-17 04:20:54 UTC
cvs done.

Comment 5 Fedora Update System 2008-02-17 07:51:04 UTC
glpi-mass-ocs-import-1.1-1.fc8 has been submitted as an update for Fedora 8

Comment 6 Fedora Update System 2008-02-17 07:53:30 UTC
glpi-mass-ocs-import-1.1-1.fc7 has been submitted as an update for Fedora 7

Comment 7 Fedora Update System 2008-02-19 03:14:00 UTC
glpi-mass-ocs-import-1.1-1.fc7 has been pushed to the Fedora 7 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 8 Fedora Update System 2008-02-19 03:20:42 UTC
glpi-mass-ocs-import-1.1-1.fc8 has been pushed to the Fedora 8 stable repository.  If problems still persist, please make note of it in this bug report.


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