[Bug 328981] Review Request: ccsm - Plugin and configuration tool - Compiz Fusion Project

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ccsm - Plugin and configuration tool - Compiz Fusion Project


https://bugzilla.redhat.com/show_bug.cgi?id=328981





------- Additional Comments From mohd.izhar.firdaus at gmail.com  2007-10-21 02:56 EST -------
(In reply to comment #8)
> 2 things need fixing
> 
> ? %exclude %{_datadir}/applications/ccsm.desktop
>   - Check if this is really needed.
>     "desktop-file-install --delete-original" should remove
>     this file.

yes .. as it desktop-file-install creates a new .desktop file with the name
fedora-ccsm.desktop (vendorID "fedora") and it does not replace ccsm.desktop

> 
> * Icon path
>   - I tried 0.6.0-1 of ccsm, then I noticed that none of
>     the icons appear on ccsm dialog.
> 
>     The following line of Constants.py needs fixing.
> -------------------------------------------------------------
>     52  # Paths
>     53  #
> !!  54  DataDir = "/usr/local/share"
>     55  IconDir = DataDir+"/icons/hicolor/scalable"
>     56  PixmapDir = DataDir+"/ccsm/images"
>     57  
> -------------------------------------------------------------

seems like the remove shebang patch somehow causes this .. anyway, patched
setup.py to make /usr as the default prefix .. the icons appears working at my
side from the below SRPM ..

http://izhar.fedorapeople.org/ccsm/ccsm-0.6.0-2.fc8.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.