This service will be undergoing maintenance at 00:00 UTC, 2016-08-01. It is expected to last about 1 hours
Bug 477948 - Review Request: simple-ccsm - Simplified plugin and configuration tool - Compiz Fusion Project
Review Request: simple-ccsm - Simplified plugin and configuration tool - Comp...
Status: CLOSED WONTFIX
Product: Fedora
Classification: Fedora
Component: Package Review (Show other bugs)
rawhide
All Linux
medium Severity medium
: ---
: ---
Assigned To: Nobody's working on this, feel free to take it
Fedora Extras Quality Assurance
:
Depends On:
Blocks: FE-DEADREVIEW
  Show dependency treegraph
 
Reported: 2008-12-26 03:50 EST by Izhar Firdaus
Modified: 2010-07-31 19:52 EDT (History)
6 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Environment:
Last Closed: 2010-07-31 19:52:55 EDT
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:


Attachments (Terms of Use)

  None (edit)
Description Izhar Firdaus 2008-12-26 03:50:47 EST
Spec URL: http://izhar.fedorapeople.org/simple-ccsm/simple-ccsm.spec
SRPM URL: http://izhar.fedorapeople.org/simple-ccsm/simple-ccsm-0.7.8-1.fc10.src.rpm
Description: 

The Compiz Project brings 3D desktop visual effects that improve
usability of the X Window System and provide increased productivity
though plugins and themes contributed by the community giving a
rich desktop experience.

This package contains a simplified gui configuration tool to
configure Compiz plugins and the composite window manager.
Comment 1 Fabian Affolter 2009-04-20 03:45:08 EDT
Just some quick comments on your spec file.  I did only a 'look-at-it' review.

- License is GPLv2+ (or (at your option) any later version) not GPLv2.  Mentioned in the LICENSE file and the source header.  PKG-INFO says GPL. 
- Is there no URL for Source0? If not, can you please describe the generation of the source tarball as mentioned in the guidelines.
- Isn't 'BR: gettext' a duplicate?  'intltool' requires 'gettext' already.
- INSTALL says '...depends on Python and PyGTK'.  
- Isn't 'Requires: hicolor-icon-theme' is missing?
- '--vendor="fedora"' is no longer needed for new packages
  https://fedoraproject.org/wiki/Packaging:Guidelines#Desktop_files

Version 0.8.3 was released some weeks ago.
http://gitweb.compiz-fusion.org/?p=compiz/compizconfig/ccsm;a=commit;h=2deb7e7194ddae144c98ea2cbef7e40f2d468b33
Comment 2 Christoph Wickert 2009-04-29 17:44:55 EDT
(In reply to comment #1)
> - Isn't 'BR: gettext' a duplicate?  'intltool' requires 'gettext' already.

It's not really needed but it's a close following of the packging guidelines that say: "If the package includes translations, add BuildRequires: gettext."

> - INSTALL says '...depends on Python and PyGTK'.  
> - Isn't 'Requires: hicolor-icon-theme' is missing?

No, because pygtk requires gtk2 requires hicolor-icon-theme.
Comment 3 Jason Tibbitts 2009-11-12 19:59:16 EST
Wow, there's been no response to review commentary in nearly eleven months; I guess I'll close this soon if there's no further response.
Comment 4 Mohd Izhar Firdaus Ismail 2009-11-12 22:53:59 EST
eek

sorry i totally forgotten i submitted this >.<

seems like i need to update it to 0.8.2 first ..
Comment 5 Mohd Izhar Firdaus Ismail 2009-11-14 01:30:39 EST
added 0.8.2 (because compiz in fedora is 0.8.2)

http://izhar.fedorapeople.org/simple-ccsm/simple-ccsm.spec
http://izhar.fedorapeople.org/simple-ccsm/simple-ccsm-0.8.2-1.fc12.src.rpm

- update to 0.8.2
- added requires pygtk
- changed license to GPLv2+
- gdk.threads_init caused simple-ccsm 0.8.2 to hang, remove it for now
Comment 6 Julian Aloofi 2009-11-18 10:32:07 EST
OK, I'm taking this. rpmlint output on the spec file is clean so far, a full review is coming soon.
Comment 7 Julian Aloofi 2009-11-18 12:44:17 EST
rpmlint output is clean.
2 packages and 1 specfiles checked; 0 errors, 0 warnings.


MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the 
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license. OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. OK
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. N/A
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. N/A
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
MUST: Header files must be in a -devel package. N/A
MUST: Static libraries must be in a -static package. N/A
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. N/A
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. OK
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. N/A
SHOULD: The package builds in mock.
- successfully builds in mock for fedora-12-i386, fedora-rawhide-i386 and fedora-11-i386


NOTE: The package must Require: compiz-fusion, as it won't run without compiz and compiz-fusion
Comment 8 Christoph Wickert 2009-11-18 17:09:38 EST
SHOULD: If scriptlets are used, those scriptlets must be sane. FIX:
Please update the icon-cache scriptlets to the latest version from
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache
Comment 9 Julian Aloofi 2009-11-27 19:15:07 EST
Ping?
Comment 10 Julian Aloofi 2010-01-25 07:14:54 EST
You're still alive?
Comment 11 Julian Aloofi 2010-03-28 04:58:08 EDT
Mohd are you still working on this package?

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