Spec URL: http://lemenkov.googlepages.com/transset.spec SRPM URL: http://lemenkov.googlepages.com/transset-20040120-1.src.rpm Description: Simple program to toggle Translucency property. This is a little handy tool. I want to mention that I forced to use "cvs co" instead of unpacking source-archives 'cause project actually never released as tarball. Maybe it would be better to download all 4 files and put it into tarball by myself?
Hey Peter. You shouldn't set the review flag here. That will be set by a reviewer once they are reviewing your package, see: http://fedoraproject.org/wiki/PackageReviewProcess CVS sources are ok...See: http://fedoraproject.org/wiki/Packaging/SourceURL
Updated spec according to http://fedoraproject.org/wiki/Packaging/SourceURL http://lemenkov.googlepages.com/transset.spec http://lemenkov.googlepages.com/transset-20040120-2.src.rpm
Typo in cvs command fixed. http://lemenkov.googlepages.com/transset.spec http://lemenkov.googlepages.com/transset-20040120-3.src.rpm
* Sat May 12 2007 Peter Lemenkov <lemenkov> 0-4.20040120cvs - Fixed the only warning - Changed naming according to http://fedoraproject.org/wiki/Packaging/NamingGuidelines http://lemenkov.googlepages.com/transset.spec http://lemenkov.googlepages.com/transset-0-4.20040120cvs.src.rpm
Just a quick glance (well, no so quick in fact) - minor problem: there are lots of duplicate BR: xorg-x11-proto-devel (by libX11-devel), pkgconfig (by libX11-devel), libX11-devel (by libXrender-devel), libXfixes-devel (by libXcomposite-devel) - the build log shows that compiler flags are not taken into account: + /usr/bin/make cc `pkg-config --cflags xcomposite xfixes xdamage xrender` -c transSet.c cc `pkg-config --cflags xcomposite xfixes xdamage xrender` -c dsimple.c cc -o transset transSet.o dsimple.o `pkg-config --libs xcomposite xfixes xdamage xrender` -lm + exit 0 - is the application a GUI ? If so, it should have a desktop file - I've tried to use it, but since there is no documentation, no man pages, nothing... I could not. Or maybe I could, but I did not know that until I read the sources. Whatever argument I pass to the application, it just echoes it and then sits until Ctrl-C is pressed: [wolfy@wolfy64 tmp]$ ./transset 1 got arg 1 d is 1 [wolfy@wolfy64 tmp]$ ./transset 12 got arg 12 d is 12 As upstream does not provide docs, I suggest creating a Readme.fedora file and include in it the comment from the beginning of transSet.c. Or add the first line of that comment to %Description. Just waiting for a mouse click without any useful output was not helpful and I doubt that users will have the patience to read the source code as I did.
It's been very nearly a year since the last comment from the submitter. This really should be closed now, and I'll do so soon if there's no further response.
Oh, I completely forgot about this Review Request. I lost interest in visual effects such as translucency/waroing/morfing and other compiz-related stuff. I'm closint this ticket