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@xxxxxxxxx 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. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review