Bug 218023 (alphabet-soup)
Summary: | Review Request: alphabet-soup - Guide your worm through the soup to spell words | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Hans de Goede <hdegoede> |
Component: | Package Review | Assignee: | Christopher Stone <chris.stone> |
Status: | CLOSED NEXTRELEASE | QA Contact: | Fedora Package Reviews List <fedora-package-review> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | ||
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2006-12-04 05:36:08 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: | 217998 | ||
Bug Blocks: | 163779 |
Description
Hans de Goede
2006-12-01 11:09:02 UTC
Build fails with: ncds.c:8:20: error: aldumb.h: No such file or directory Review blocked due to build failure. ==== REVIEW CHECKLIST ==== - rpmlint output clean - package named according to package naming guidelines - spec file name matches %{name} - meets packaging guidelines - licensed with open source compatible license - license field matches actual license - license file included in %doc - spec written in American english - spec legible - sources match upstream: 364fe30c53a86217db37f53308604f74 as11src.zip - successfully compiles and builds on FC6 x86_64 X missing BR: dumb-devel - no locales - no shared libraries - package is not relocatable - package owns all directories it creates X package uses directory not pulled in through Requires must require hicolor-icons - no duplicates in %files - file permissions set properly - contains proper %clean - macro usage consistent - contains code - no large documentation - %doc does not affect runtime - no header files or static libraries - no pkgconfig files - no need for devel subpackage - no .la files - contains proper .desktop file - does not own files or directories owned by other packages ==== MUST FIX ==== - Add dumb-devel to BR - requires hicolor-icon-theme package (In reply to comment #2) > ==== MUST FIX ==== > - Add dumb-devel to BR Good catch! > - requires hicolor-icon-theme package Will I ever learn? :( Both fixed, new version is here: Spec URL: http://people.atrpms.net/~hdegoede/alphabet-soup.spec SRPM URL: http://people.atrpms.net/~hdegoede/alphabet-soup-1.1-2.fc7.src.rpm All MUST items fixed. APPROVED. Thanks! Imported and build requested, closing. |