[Bug 185951] Review Request: amsn : msn messenger clone

[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: amsn : msn messenger clone


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





------- Additional Comments From wart@xxxxxxxxxx  2006-06-18 18:54 EST -------
MUST
====
* rpmlint output:
  W: amsn-plugins no-documentation

* Package named appropriately
* GPL license ok, license file included
* Spec file legible
* compiles and packages on FC5 i386 and FC5 x86_64.
* No language files in %{_datadir}/locale, no need for %find_lang
  All translations are stored in %{_datadir}/%{name}/lang
* No shared libs in the default linker path
* Not relocatable
* File permissions look ok.
* $RPM_BUILD_ROOT cleaned where needed.
* Contains code, not content
* Not enough docs to warrant a -doc subpackage
* No reason for a -devel subpackage (no headers, libs, pkgconfig files)
* Does not own directories owned by other packages

MUSTFIX
=======
* Version comparison for tk-devel is not necessary since FC4 and later all
  use tk 8.4.
* The URL in %description isn't necessary since it's already in the URL:
  tag.
* Remove the Version: tag from the plugins subpackage so that it inherits
  the full %{version}-%{release} from the base package.  Otherwise you run
  the risk of building a second, slightly different plugins subpackage later
  that has the same version.
* The 48x48 icon for the .desktop file should go in
  ${RPM_BUILD_ROOT}%{_datadir}/icons/hicolor/48x48/apps/
  and you need to update the icon cache in both %post and %postun:

  touch --no-create %{_datadir}/icons/hicolor || :
  if [ -x %{_bindir}/gtk-update-icon-cache ]; then
     %{_bindir}/gtk-update-icon-cache --quiet %{_datadir}/icons/hicolor || :
  fi
* Missing BuildRequires: autoconf
* BuildRequires: libXt-devel is redundant since it's also required by tk-devel.
* The binaries are installed into /usr/share/amsn/ and linked into /usr/bin.
* There are a number of duplicate files in %{_datadir}/amsn and %doc:
  AGREEMENT, CREDITS, ... These should be removed from %{_datadir}/amsn.
* amsn/lang/LANG-HOWTO should be included in %doc
* The desktop file is placed in %{_datadir}/applications and %{_datadir}/amsn.
  Remove the extra copy in %{_datadir}/amsn since it's not needed there.
* Missing "Requires: mozilla".  I would recommend, however, changing the
  default browser command from 'mozilla $url' to 'htmlview $url' and
  adding "Requires: htmlview".  This will launch the url in the
  user's preferred web browser.
* Missing "Requires: sox" for /usr/bin/play for playing sounds.
* As pointed out above, be consistent about $RPM_BUILD_ROOT vs.
  ${RPM_BUILD_ROOT}
* The %description for the -plugins subpackage doesn't need to include the
  full description for the base package.  Just describe the purpose
  of the subpackage, such as "Extra plugins for amsn to enable foo, bar,
  and baz."

QUESTIONS
=========
* Official upstream sources come from sourceforge.net, but this rc release
  comes from elsewhere.  Since I know you're one of the upstream developers,
  I'm not too concerned about this, but it would be better if Source0:
  pointed to upstream's official download page.

* You might consider submitting cximage as a separate package instead of
  including it in amsn.  Since Fedora doesn't include cximage already,
  this won't block the review.

* Is there a special reason why the amsn commands (amsn, amsn-remote,
  amsn-remote-CLI) are located in %{_datadir}/amsn and only linked to
  %{_bindir}?  Why can't they just be installed directly into %{_bindir}?

* Is it necessary to include the %{_datadir}/amsn/lang/missing.py, convert.tcl,
  and sortlang utilities in the package?


-- 
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]