[Bug 189717] Review Request: gcompris - Educational suite for kids 2-10 years old

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gcompris - Educational suite for kids 2-10 years old


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=189717


wart@xxxxxxxxxx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|163778                      |163779
              nThis|                            |




------- Additional Comments From wart@xxxxxxxxxx  2006-04-30 03:12 EST -------
rpmlint output:
E: gcompris hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/menu

- This is a bogus warning.  rpmlint is complaining about the src
  rpm without realizing that this file is being removed, not included.

W: gcompris-devel no-dependency-on gcompris

- This is ok by design.  -devel requires -libs so that it doesn't
  need to pull in rather large (and unnecessary) base package.

W: gcompris-devel no-documentation
W: gcompris-libs no-documentation

- This can be ignored.  The developers documentation is integrated with
  the base package's documentation.

W: gcompris-flags no-documentation
W: gcompris-sound-da no-documentation
W: gcompris-sound-de no-documentation
W: gcompris-sound-en no-documentation
W: gcompris-sound-es no-documentation
W: gcompris-sound-eu no-documentation
W: gcompris-sound-fi no-documentation
W: gcompris-sound-fr no-documentation
W: gcompris-sound-hu no-documentation
W: gcompris-sound-it no-documentation
W: gcompris-sound-nl no-documentation
W: gcompris-sound-pt no-documentation
W: gcompris-sound-ru no-documentation
W: gcompris-sound-sv no-documentation

- These can be ignored.  There's no need for documentation for localized
  sound/flag files.


MUST
====
 * GPL license ok, license file included
 * Package and spec file named appropriately
 * Spec file legible and in Am. English (except for the localized parts)
 * Sources for the review came from upstream:
   d80e55da21170a98d671c98b3e582cc3  SOURCES/gcompris-7.4.tar.gz
 * Builds in mock on devel-i386, devel-x86_64
 * Locales handled appropriately
 * ldconfig called for shared libs in -libs subpackage
 * Not relocatable
 * Owns directories that it creates
 * No duplicate %files
 * Contains code and permissible content
 * $RPM_BUILD_ROOT cleaned correctly in %install and %clean
 * File permissions appear to be ok
 * headers, .so, and pkgconfig files in -devel
 * -devel requires -libs, not base (this is acceptable, see above)
 * .la files not included
 * .desktop files installed correctly
 * No need for -doc subpackage

SHOULD
=========
 * Consider adding Requires: tuxpaint for the tuxpaint launcher

I did a quick check of the english/german sounds on a handful of the games and
it worked with no obvious problems.

No blockers found.

APPROVED


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]