Bug 411421
Summary: | Review Request: d-feet - D-Bus debugger | ||||||
---|---|---|---|---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | John (J5) Palmieri <johnp> | ||||
Component: | Package Review | Assignee: | Parag AN(पराग) <panemade> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Fedora Extras Quality Assurance <extras-qa> | ||||
Severity: | medium | Docs Contact: | |||||
Priority: | medium | ||||||
Version: | rawhide | CC: | a.badger, fedora-package-review, jkeck, notting | ||||
Target Milestone: | --- | Flags: | panemade:
fedora-review+
gwync: fedora-cvs+ |
||||
Target Release: | --- | ||||||
Hardware: | All | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | devel,F-8,F-7 | Doc Type: | Bug Fix | ||||
Doc Text: | Story Points: | --- | |||||
Clone Of: | Environment: | ||||||
Last Closed: | 2007-12-09 02:51:17 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
John (J5) Palmieri
2007-12-05 02:07:22 UTC
+ desktop-file-install --vendor= --delete-original --dir=/var/tmp/d-feet-0.1.2-1.fc9-root-mockbuild/usr/share/applications ui/d-feet.desktop Error on file "ui/d-feet.desktop": No such file or directory it should be dfeet.desktop 1)some other things scriptlet scrollkeeper is not used so its BR are not required right now. 2)Ok. So I corrected SPEC and tried to rebuild but it failed with /var/tmp/d-feet-0.1.2-1.fc9-root-mockbuild/usr/share/applications/dfeet.desktop: error: value "dfeet-icon.png" for key "Icon" in group "Desktop Entry" is an icon name with an extension, but there should be no extension as described in the Icon Theme Specification if the value is not an absolute path Error on file "ui/dfeet.desktop": Failed to validate the created desktop file did mock build =>http://koji.fedoraproject.org/koji/getfile?taskID=275767&name=build.log 3)I don't see any python eggs needed here. http://fedoraproject.org/wiki/Packaging/Python/Eggs (/me need to read above guidelines) Everything should be fixed in latest version Spec URL: http://johnp.fedorapeople.org/d-feet.spec SRPM URL: http://johnp.fedorapeople.org/d-feet-0.1.2-3.fc8.src.rpm I didn't do the %if 0%{?fedora} >= 8 BuildRequires: python-setuptools-devel %else BuildRequires: python-setuptools %endif conditional because we can change this if we want an older build. Spec URL: http://johnp.fedorapeople.org/d-feet.spec SRPM URL: http://johnp.fedorapeople.org/d-feet-0.1.3-1.fc8.src.rpm More cleanups upstream. http://koji.fedoraproject.org/koji/taskinfo?taskID=277227 with new updated SPEC, I saw => d-feet.src: E: description-line-too-long D-Bus is an RPC library used on the Desktop. D-Feet can be used to inspect D-Bus objects of running programs and invoke methods on those objects. Your description lines must not exceed 79 characters. If a line is exceeding this number, cut it to fit in two lines. I can still see BR: scrollkeeper I don't see any of the *egg files getting installed as part of following http://fedoraproject.org/wiki/Packaging/Python/Eggs The BR and description length are easy to fix. The egg stuff however is unclear since I followed all the steps and it still isn't producing eggs. I also read from the page: We need to provide eggs for those packages but if upstream does not provide eggs we should only provide them if necessary to support another package. This package is an application and does not support other packages so I think this rule can be waved. If you give me advice on making eggs work upstream I would gladly add it but right now it seems like a useless blocker. Parag AN, do you know of a package or program that needs to use d-feet eggs? Otherwise, J5's in the right here. Upstream isn't providing eggs and it's primarily a program anyway so there's no reason for us to to so. (In reply to comment #7) > Parag AN, do you know of a package or program that needs to use d-feet eggs? From my side no. and reason to discuss this eggs issue is to ask reporter if he knows any other packages that needs this package and thus should take care of eggs files installation. So, in this case is BR of python-setuptools-devel, still mandatory? > Otherwise, J5's in the right here. Upstream isn't providing eggs and it's > primarily a program anyway so there's no reason for us to to so. Anyway, I don't see this as a blocker here. (In reply to comment #8) > (In reply to comment #7) > > Parag AN, do you know of a package or program that needs to use d-feet eggs? > From my side no. and reason to discuss this eggs issue is to ask reporter if > he knows any other packages that needs this package and thus should take care of > eggs files installation. > So, in this case is BR of python-setuptools-devel, still mandatory? I would assume it is still mandatory since I use python-setuptools to do the build. Review: + package builds in mock (development i386). + rpmlint is NOT silent for SRPM. But silent for RPM. d-feet.src: E: description-line-too-long D-Bus is an RPC library used on the Desktop. D-Feet can be used to inspect D-Bus objects of running programs and invoke methods on those objects. + source files match upstream url 35f713feee9b9b8a7ec5fff6d2fd18fa d-feet-0.1.3.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + BuildRequires are proper(with extra unnecessary scrollkeeper) + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available. + Does owns the directories it creates. + gtk-update-icon-cache scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Desktop file installed correctly. + GUI App. APPROVED. SHOULD: fix rpmlint and BR problem. According to http://fedoraproject.org/wiki/Packaging/Guidelines#head-d559ee7363418a5840ce63090c608c991cd39ce6 If upstream uses <vendor_id>, leave it intact, otherwise use fedora as <vendor_id>. Created attachment 280511 [details]
traceback log
Hey I got many messages and Traceback errors when I started this d-feet program
from gnome-terminal on F8.
Attaching here log of starting d-feet from terminal and then closing it.
Those are known upstream issues. Will fix the None errors in the next release. The glade errors come from using gazpacho instead of glade so I may need to fiddle with the glade file manually. New Package CVS Request ======================= Package Name: d-feet Short Description: D-Bus debugger Owners: johnp Branches: F-7 F-8 Cvsextras Commits: no cvs done. built in devel, F-7 and F-8. Closing. Package Change Request ====================== Package Name: d-feet New Branches: el6 Owners: johnp Git done (by process-git-requests). |