Bug 781260 (leechcraft) - Review Request: leechcraft - A Free Open Source Cross-Platform Modular Internet-Client
Summary: Review Request: leechcraft - A Free Open Source Cross-Platform Modular Intern...
Alias: leechcraft
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Rex Dieter
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: kde-reviews
TreeView+ depends on / blocked
Reported: 2012-01-13 05:38 UTC by Minh Ngo
Modified: 2013-03-07 13:16 UTC (History)
6 users (show)

Fixed In Version: leechcraft-0.5.60-5.fc17
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-04-01 22:58:03 UTC
Type: ---
rdieter: fedora-review+

Attachments (Terms of Use)

Description Minh Ngo 2012-01-13 05:38:32 UTC
Spec URL: https://github.com/Ignotus/leechcraft-fedora/blob/e4493277dfe2fa3a7cc57cd193e92a0e7ab4ccd8/leechcraft.spec
SRPM URL: https://github.com/Ignotus/leechcraft-fedora/blob/e4493277dfe2fa3a7cc57cd193e92a0e7ab4ccd8/leechcraft-0.4.97-1.fc16.src.rpm?raw=true
Description: LeechCraft is a free modular "Internet client" application.

LeechCraft allows to browse the web, read RSS/Atom feeds, download
files via BitTorrent, HTTP, FTP and DC, automatically stream,
download or play podcasts and other media files and much more.

Features can be easily added via plugins that can be integrated with
each other with no effort while staying abstract from the exact

This package contains the main LeechCraft executable, which connects
all the plugins with each other, routes requests between them, tracks
dependencies and performs several other housekeeping tasks.

To build it in koji please add qxmpp-dev- by "Buildroot Overrides" in bodhi if it's not already in the stable. ( https://bugzilla.redhat.com/show_bug.cgi?id=753262 )

There is the complete scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3645191

Comment 1 Rex Dieter 2012-01-13 13:34:51 UTC
Offhand, I see a *lot* of sub-packages for plugins, which greatly adds complexity, for both packager (complicating .spec) and end-user (who's bombarded with an incredible number of choices about which ones to install.  I hope you have good reason do want to do it this way?

Wrt buildroot overrides, you can do it yourself via bodhi,

Comment 2 Minh Ngo 2012-01-13 15:29:33 UTC
I'm maintained these packages in openSUSE. And they are not very complexity for supporting.

A end-user can read a description of a package and decide to install it.

Every leechcraft package is a separate plugin that adds some functionality to the program. So they should not be in the single package.

Comment 3 Minh Ngo 2012-01-13 15:30:21 UTC
s/I'm maintained/I'm maintaining/

Comment 4 Kevin Kofler 2012-01-13 15:45:36 UTC
It looks indeed like that application is designed the Emacs way, where almost everything is a plugin…

I think there should be some metapackage dragging in the whole package with all the plugins, like eclipse-platform, netbeans-platform, wine (as opposed to wine-core) etc.

Comment 10 Rex Dieter 2012-02-14 18:45:00 UTC
I was about to start review, but will be busy until at least the 20th, so I'll pick it up again then if no one beats me to it.

Comment 12 Rex Dieter 2012-03-06 17:33:05 UTC
OK, sorry for the delay.

1.  Naming: mostly ok, but subpkg dependencies SHOULD be fully versioned, for example, instead of:
Requires: %{name}-iconset
Requires: %{name}-iconset = %{full_version}

and in -iconset-oxygen subpkg,

Provides: %{name}-iconset = %{full_version}

*unless*, you ever intend to provide alternative iconset's provided elsewhere?  If so, then best leave off the versioning.

2.  macros.  SHOULD remove
%if "%{_lib}" == "lib64"
    -DLIB_SUFFIX=64 \
snippet, default %{cmake} macro already includes that.  see /etc/rpm/macros.cmake

3.  scriptlets.  MUST
main pkg, iconset-oxygen, iconset-tango are lacking icon-related scriptlets, see

4.  qm translation files SHOULD be marked with %lang(locale) tags, but I'm not sure if that's practical here.  if you can figure out a way to do that in an easy way, great.  if not, don't worry about it.

Sources: ok
7daefb3130a475f9260eac069942076e  leechcraft-0.5.0.tar.xz

Licensing: ok
most code is GPLv3+, sprinkled with some LGPLv2+ and GPLv2+, so combined result is indeed GPLv2+

Address items 1-4, and we're looking good.

Comment 13 Kevin Kofler 2012-03-06 17:51:37 UTC
As I understand the guidelines, proper handling of translations (point 4) is a MUST, not a SHOULD.

You should just need multiple %find_lang --with-qt --without-mo calls at the end of %install. See how I did this in the Qt package itself:

You should also be able to use %find_lang for the eiskaltdcpp translations instead of listing every language by hand. (And by the way, those are gettext translations, why aren't they in /usr/share/locale?)

Comment 14 Minh Ngo 2012-03-10 22:32:56 UTC
translation files are not in the standard directory, so I can't find their with %find_lang macro.

Comment 15 Minh Ngo 2012-03-10 22:40:16 UTC
I don't know why they aren't in /usr/share/locale, but I can fix with something like:
root_dir=`echo ${RPM_BUILD_ROOT} | sed 's,\/,\\\/,g'`
find ${RPM_BUILD_ROOT}%{translations_dir} -name libeiskaltdcpp.mo | sed 's/^.*ions\/\([a-z]*\)\/LC.*/%lang(\1) \0/g;s/'$root_dir'//g' > libeiskaltdcpp.lang

Comment 19 Rex Dieter 2012-03-11 21:12:09 UTC
Well, a minor nit... if you aren't shipping alternative iconsets (ie, tango has been remove), why bother splitting out oxygen-iconset at all?  think about it. :)

Anyway, my other issues have been resolved,  APPROVED.

Comment 20 Minh Ngo 2012-03-11 21:20:00 UTC
New Package SCM Request
Package Name: leechcraft
Short Description: LeechCraft internet client
Owners: ignotusp
Branches: f15 f16 el6

Comment 21 Gwyn Ciesla 2012-03-12 12:07:03 UTC
Git done (by process-git-requests).

Added f17.

Comment 22 Fedora Update System 2012-03-12 18:30:40 UTC
leechcraft-0.5.0-3.fc16 has been submitted as an update for Fedora 16.

Comment 23 Fedora Update System 2012-03-12 18:31:59 UTC
leechcraft-0.5.0-3.fc15 has been submitted as an update for Fedora 15.

Comment 24 Fedora Update System 2012-03-14 19:46:11 UTC
leechcraft-0.5.0-4.fc17 has been submitted as an update for Fedora 17.

Comment 25 Fedora Update System 2012-03-15 02:22:17 UTC
leechcraft-0.5.0-3.fc16 has been pushed to the Fedora 16 testing repository.

Comment 26 Fedora Update System 2012-03-18 07:09:55 UTC
leechcraft-0.5.60-1.fc17 has been submitted as an update for Fedora 17.

Comment 27 Fedora Update System 2012-03-18 08:11:36 UTC
leechcraft-0.5.60-2.fc15 has been submitted as an update for Fedora 15.

Comment 28 Fedora Update System 2012-03-18 08:12:32 UTC
leechcraft-0.5.60-2.fc16 has been submitted as an update for Fedora 16.

Comment 29 Fedora Update System 2012-03-18 08:55:53 UTC
leechcraft-0.5.60-3.fc16 has been submitted as an update for Fedora 16.

Comment 30 Fedora Update System 2012-03-18 08:56:39 UTC
leechcraft-0.5.60-3.fc15 has been submitted as an update for Fedora 15.

Comment 31 Fedora Update System 2012-03-24 07:02:48 UTC
leechcraft-0.5.60-5.fc17 has been submitted as an update for Fedora 17.

Comment 32 Fedora Update System 2012-03-24 16:34:23 UTC
leechcraft-0.5.60-5.fc15 has been submitted as an update for Fedora 15.

Comment 33 Fedora Update System 2012-04-01 22:58:03 UTC
leechcraft-0.5.60-5.fc15 has been pushed to the Fedora 15 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 34 Fedora Update System 2012-04-12 02:03:51 UTC
leechcraft-0.5.60-5.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 35 Minh Ngo 2013-03-07 10:02:20 UTC
Please unretired f17 f18 branches

Comment 36 Gwyn Ciesla 2013-03-07 13:16:00 UTC
This requires a re-review.

Note You need to log in before you can comment on or make changes to this bug.