Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at bugzilla.redhat.com.
Bug 166550
Summary: | Review Request: synce-software-manager | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Andreas Bierfert <andreas.bierfert> |
Component: | Package Review | Assignee: | Jef Spaleta <jspaleta> |
Status: | CLOSED NEXTRELEASE | QA Contact: | David Lawrence <dkl> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | d.lesca, fedora-extras-list, gauret, jspaleta |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
URL: | http://synce.sourceforge.net/ | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-12-13 17:29: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: | |||
Bug Depends On: | |||
Bug Blocks: | 163779 |
Description
Andreas Bierfert
2005-08-23 09:44:04 UTC
*** Bug 166559 has been marked as a duplicate of this bug. *** Okay I'm willing to pick up the synce* submissions from late August to do the package guidelines review. But I don't have the winCE based hardware to test functionality against. Is there a way to fake a device with synce or anything like that? I can get started on reviewing synce-software-manager first and then move on to the other ones from there. -jef Thanks :) Hm, to my knowledge there is no non-tivial way to fake a winCE device. Does anybody know more about this? Okay I've gone over the review. 2 blockers 1) Need to add BuildRequires: libgnomeui-devel I fixed this to do my review and located here: http://jef.is-a-geek.com/downloads/synce-software-manager/synce-software-manager-0.9.0-2.src.rpm http://jef.is-a-geek.com/downloads/synce-software-manager/synce-software-manager.spec 2) Need to add a desktop file and associated scriptlet and Requires I haven't fixed this, You'll need to add a .desktop as a SOURCE1 and then use the instructions in http://www.fedoraproject.org/wiki/PackagingGuidelines for Desktop Files. Please add the desktop file and respin the srpm so I can do a final review of the changes. Full review: - GOOD: builds in mock on an fc4 host system. - GOOD: rpmlint returns clean. - GOOD: The package is named according to the PackageNamingGuidelines. - GOOD: The spec file name matches the base package %{name} - GOOD: package meets the PackagingGuidelines. - GOOD: The package is licensed with an open-source compatible license (MIT) - GOOD: The License field in the package spec file matches the actual license. - GOOD: The source package includes the text of the license(s) in its own file, COPYING and is included in %doc. - GOOD: The spec file is written American English. - GOOD: The spec file for the package is be legible. - GOOD: The sources used to build the package match the upstream source. - Fixed: The package must successfully compiles and build into binary rpms on fc4 i386 and fc-devel i386. Needed to add BuildRequires: libgnomeui-devel - GOOD: package does not contain any BuildRequires that are listed in the exceptions section of PackagingGuidelines. - Fixed: All other Build dependencies must be listed in BuildRequires. Need to add BuildRequires: libgnomeui-devel - GOOD: Uses the %find_lang macro. - GOOD: No shared library files located in the dynamic linker's default paths - GOOD: package owns all directories that it creates. - GOOD: package does not contain any duplicate files in the %files listing. - GOOD: Permissions on files must be set properly. - GOOD: package has a %clean section, which contains rm -rf %{buildroot} (or $RPM_BUILD_ROOT). - GOOD: package consistently use macros, as described in the macros section of PackagingGuidelines. - GOOD: package contains code, or permissable content. - GOOD: No large documentation files. - GOOD: %doc files do not affect the runtime of the application. - GOOD: No header files or static libraries. - GOOD: No files used by pkgconfig (.pc files). - GOOD: No library files with a suffix (e.g. libfoo.so.1.1) - BAD: Packages containing GUI applications must include a %{name}.desktop file, and that file must be properly installed with desktop-file-install in the %install section. This is described in detail in the desktop files section of PackagingGuidelines. If you feel that your packaged GUI application does not need a .desktop file, you must put a comment in the spec file with your explanation. Here you go :) http://fedora.lowlatency.de/review/synce-software-manager-0.9.0-3.src.rpm http://fedora.lowlatency.de/review/synce-software-manager.spec I'll help testing, I have a wince device. But I have little time at the moment, sorry. Review for release 3: * RPM name is OK * Source synce-software-manager-0.9.0.tar.gz is the same as upstream * Builds fine in mock * rpmlint of synce-software-manager looks OK * File list of synce-software-manager looks OK * Works fine I would APPROVE this version, I'm changing the blocker bug. Jef, if you disagree, feel free to set it back. imported and build, THANKS you both for your time |