Spec URL: http://scurn.net/fedora/gdesklets-goodweather.spec SRPM URL: http://scurn.net/fedora/gdesklets-goodweather-0.3-1.fc7.src.rpm Description: Themeable weather and condition display for gdesklets. Displays the current conditions along with a 5 day forcast Note: There is no license attached with the source tarball, but the README states the code was released under the GPL. I am currently seeking a sponsor. I have also submitted proxychains for review (https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244018) I have submitted a review for the following packages as well: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244086 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244171 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244411 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244623 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244704 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244936
My server that hosts the spec and srpm appears to be down and I am out of town now so I have mirrored the files on another server that I have access to. http://yes.urapita.net/fedora/gdesklets-goodweather.spec http://yes.urapita.net/fedora/gdesklets-goodweather-0.3-1.fc7.src.rpm
Created attachment 158722 [details] mock build log of gdesklets-goodweather 0.3-1 on rawhide i386 Some initial comments * Mockbuild - mockbuild failed on rawhide i386. Note that "python" package is removed from minimum build environment (please check the log attached) * Directories - The following directories are not owned by any packages. ----------------------------------------------------- /usr/share/gdesklets/ /usr/share/gdesklets/Displays/ /usr/share/gdesklets/Sensors/ ----------------------------------------------------- *For now* this package should own the directories above. (but IMO these directories should be owned by gdesklets. Would you contact with gdesklets maintainer?) * Timestamps - To keep timestamps, please use "install -p"
I have fixed the issues listed above. I have also contacted the maintainer of gdesklets and asked them to create/own the above directories. In the interim, this package owns the directories. * Sat Jul 07 2007 Tyler Owen <tyler.l.owen> - 0.3-2 - Added Python to Requires - Added -p option to install to preserve timestamps - Fixed missing directory ownership issues Spec URL: http://scurn.net/fedora/gdesklets-goodweather.spec SRPM URL: http://scurn.net/fedora/gdesklets-goodweather-0.3-2.fc7.src.rpm
http://scurn.net/ still seems no good...
Well, I downloaded your srpm from other system. Checking now...
Well, * NOTE: For now gdesklets itself does not work on rawhide. On Fedora 7, it and this package seem to be working. Okay. ---------------------------------------------------------- This package (gdesklets-goodweather) is APPROVED by me. ----------------------------------------------------------
New Package CVS Request ======================= Package Name: gdesklets-goodweather Short Description: weather and forcast display for gdesklets Owners: tyler.l.owen Branches: F-7 InitialCC:
Package Change Request ====================== Package Name: gdesklets-goodweather New Branches: FC-6
cvs done.
Built on devel, F-7 and FC-6 Requested on Bodhi to push. Closing. Thanks for the review and sponsorship!!
Package Change Request ====================== Package Name: gdesklets-goodweather Updated Fedora Owners: tyler.l.owen,luya_tfz