Bug 225773
| Summary: | Merge Review: f-spot | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nobody's working on this, feel free to take it <nobody> | ||||
| Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> | ||||
| Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> | ||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | medium | ||||||
| Version: | rawhide | CC: | caillon, chkr, ma, panemade | ||||
| Target Milestone: | --- | Flags: | panemade:
fedora-review+
|
||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | 2010-12-15 07:21:31 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
Nobody's working on this, feel free to take it
2007-01-31 18:40:26 UTC
rpmlint reported invalid desktop files Use desktop-file-validate on f-spot.desktop and f-spot-view.desktop More on how to install .desktop files in SPEC is at http://fedoraproject.org/wiki/Packaging/Guidelines#head-254ddf07aae20a23ced8cecc219d8f73926e9755 Kindly update SPEC. any updates? Isn't F-Spot available in Fedora now? Martin, yes this is a merge review (for previous Core packages that never had a formal review). (In reply to comment #1) > rpmlint reported invalid desktop files > Use desktop-file-validate on f-spot.desktop and f-spot-view.desktop > More on how to install .desktop files in SPEC is at > http://fedoraproject.org/wiki/Packaging/Guidelines#head-254ddf07aae20a23ced8cecc219d8f73926e9755 These problems are solved long time ago. ;-) Created attachment 455392 [details]
spec cleanup
spec cleanup patch. Please don't commit this as it is. I have used Packaging guidelines to cleanup this package.
Also saw some rpmlint errors
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/f-spot.exe.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/FSpot.Cms.dll.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/Hyena.dll.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/FSpot.Core.dll.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/FSpot.Gui.dll.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/FSpot.Utils.dll.config
f-spot.x86_64: E: script-without-shebang /usr/lib64/f-spot/Hyena.Gui.dll.config
Also, you need to add comment in spec for which part of code is in licenses given in license tag.
Can maintainer please fix all above issues?
chkr ping Sorry for the delay. Please see git for the changes to the spec file: http://pkgs.fedoraproject.org/gitweb/?p=f-spot.git;a=blobdiff;f=f-spot.spec;h=7659678c4b43817180038aed2d9a58e47d71b6ee;hp=1c8a502c1d7722744b65c25d8a23d3aab2f67b96;hb=3c719fff3bae48ebeb2f233207a56b6311e47b6a;hpb=8224187cd0ec6b137061cc505671c149ce3d68e7 Looks good now. Is this update built for rawhide? (In reply to comment #9) > Looks good now. Is this update built for rawhide? Yes, done: http://koji.fedoraproject.org/koji/buildinfo?buildID=208945 Thanks. APPROVED. |