Bug 245527
Summary: | Review Request: gdesklets-goodweather - weather and forcast display for gdesklets | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Tyler Owen <tyler.l.owen> | ||||
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> | ||||
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | low | ||||||
Version: | rawhide | CC: | fedora-package-review, mtasaka, notting | ||||
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
kevin: fedora-cvs+ |
||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2007-07-11 01:27:24 UTC | Type: | --- | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Tyler Owen
2007-06-25 03:48:10 UTC
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 cvs done. |