[Bug 234830] Review Request: audacious-show - XChat plugin for audacious
bugzilla at redhat.com
bugzilla at redhat.com
Wed Jun 20 03:39:48 UTC 2007
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: audacious-show - XChat plugin for audacious
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234830
------- Additional Comments From lxtnow at gmail.com 2007-06-19 23:39 EST -------
=== REQUIRED ITEMS ===
[ ? ] Package is named according to the Package Naming Guidelines.
[ ? ] Spec file name must match the base package %{name}
[ OK ] Package meets the Packaging Guidelines.
[ OK ] Package successfully compiles and builds.
[ Ok ] Mock built on x86_64 (F-7, devel).
[ Ok ] Package is not relocatable.
[ Ok ] Buildroot is correct
[ Ok ] Package is licensed with an open-source compatible license.
[ OK ] License field in the package spec file matches the actual license.
[ Ok ] License type: GPL
[ OK ] License text is included in package.
[ Ok ] Spec file is legible and written in American English.
[ OK ] Package is not known to require ExcludeArch
[ OK ] All build dependencies are listed in BuildRequires
[ OK ] The spec file handles locales properly.
[ SKIP ] ldconfig called in %post and %postun if required.
[ OK ] Package must own all directories that it creates.
[ OK ] Package requires other packages for directories it uses.
[ OK ] Package does not contain duplicates in %files.
[ OK ] Permissions on files are set properly.
[ OK ] Package has a %clean section.
[ OK ] Package consistently uses macros.
[ OK ] Package contains code, or permissable content.
[ SKIP ] Large documentation files are in a -doc subpackage, if required.
[ OK ] Package uses nothing in %doc for runtime.
[ SKIP ] Header files in -devel subpackage, if present.
[ SKIP ] Static libraries in -devel subpackage, if present.
[ SKIP ] Package requires pkgconfig, if .pc files are present.
[ SKIP ] Development .so files in -devel subpackage, if present.
[ SKIP ] Fully versioned dependency in subpackages, if present.
[ OK ] Package does not contain any libtool archives (.la).
[ SKIP ] Package contains a properly installed %{name}.desktop file.
[ OK ] Package does not own files or directories owned by other packages.
[ CHECK ] Requires:
hicolor-icon-theme libQtCore.so.4()(64bit) libQtGui.so.4()(64bit)
libQtXml.so.4()(64bit) libX11.so.6()(64bit) libXtst.so.6()(64bit)
libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit) libc.so.6(GLIBC_2.4)(64bit)
libgcc_s.so.1()(64bit) libgcc_s.so.1(GCC_3.0)(64bit) libm.so.6()(64bit)
libpthread.so.0()(64bit) libpthread.so.0(GLIBC_2.2.5)(64bit)
libstdc++.so.6()(64bit) libstdc++.so.6(CXXABI_1.3)(64bit)
libstdc++.so.6(GLIBCXX_3.4)(64bit) rtld(GNU_HASH)
== Rpmlint output: ==
[ OK ] Silent on both.
=== some comments ===
* you can easly export and set your variable by one line in spec file as follow:
export CFLAGS="RPM_OPT_FLAGS"
* 'bout package naming.
As this package is a plugin of x-chat, the package name should be renaming to
follow all plugins packages naming.
So your package name should look like this:
x-chat-audacious-show or x-chat-audacious instead of just audacious.
* INSTALL file should be removed from %doc.
--
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.
More information about the package-review
mailing list