New subject: [Bug 1208582] Review Request: trojita - IMAP e-mail client

2 Apr 2015

      https://bugzilla.redhat.com/show_bug.cgi?id=1208582
Bug ID: 1208582
           Summary: Review Request: trojita - IMAP e-mail client
           Product: Fedora
           Version: rawhide
         Component: Package Review
          Severity: medium
          Priority: medium
          Assignee: nobody@fedoraproject.org
          Reporter: projects.rg@smart.ms
        QA Contact: extras-qa@fedoraproject.org
                CC: bjoern.esser@gmail.com, extras-qa@fedoraproject.org,
                    i@cicku.me, jkt@flaska.net, kevin@tigcc.ticalc.org,
                    kvolny@redhat.com, lupinix@mailbox.org,
                    mario.blaettermann@gmail.com,
                    package-review@lists.fedoraproject.org,
                    projects.rg@smart.ms, rdieter@math.unl.edu,
                    zbyszek@in.waw.pl
        Depends On: 1080411 (trojita)
            Blocks: 928937 (qt-reviews)
   External Bug ID: KDE Software Compilation 345739
   External Bug ID: KDE Software Compilation 345739
SPEC: https://raphgro.fedorapeople.org/review/qt/trojita/trojita.spec
SRPM:
https://raphgro.fedorapeople.org/review/qt/trojita/trojita-0.5-3.fc21.src.rp...
rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9393885
Open issues:
- original requester is lost
- generation of a manpage fails (with help2man)
- reported a failed testcase of html formatting to upstream
See below for more details taken from the original review request.
******
+++ This bug was initially created as a clone of Bug #1080411 +++
Spec URL: http://kvolny.fedorapeople.org/trojita.spec
SRPM URL: http://kvolny.fedorapeople.org/trojita-0.4.1-1.fc20.src.rpm
Description:
Trojitá is a Qt IMAP e-mail client which:
  * Enables you to access your mail anytime, anywhere.
  * Does not slow you down. If we can improve the productivity of an e-mail
    user, we better do.
  * Respects open standards and facilitates modern technologies. We value
    the vendor-neutrality that IMAP provides and are committed to be as
    interoperable as possible.
  * Is efficient — be it at conserving the network bandwidth, keeping memory
    use at a reasonable level or not hogging the system's CPU.
  * Can be used on many platforms. One UI is not enough for everyone, but our
    IMAP core works fine on anything from desktop computers to cell phones
    and big ERP systems.
  * Plays well with the rest of the ecosystem. We don't like reinventing
wheels,
    but when the existing wheels quite don't fit the tracks, we're not afraid
    of making them work.
Fedora Account System Username: kvolny
Koji scratch builds:
F20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671258
EPEL7: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671502
Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671309
--- Additional comment from Karel Volný on 2014-03-25 13:00:07 CET ---
note that I've filed some bugs upstream for the things I've workarounded in
spec:
Bug 332578 - superfluous x-test language
[https://bugs.kde.org/show_bug.cgi?id=332578]
Bug 332579 - unversioned plugins library (rpmlint invalid-soname issue)
[https://bugs.kde.org/show_bug.cgi?id=332579]
--- Additional comment from Rex Dieter on 2014-04-13 23:18:12 CEST ---
Offhand looks like a good start, some initial suggestions (SHOULD are just
recommendations and/or a matter of taste, MUST are required):
1.  SHOULD add
Requires: hicolor-icon-theme
instead of:
%dir %{_datadir}/icons/hicolor
%dir %{_datadir}/icons/hicolor/*
2.  MUST add some icon scriptlets, see
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
3.  SHOULD include .spec comments clearly identifying items that are
workarounds for the bugs referenced in comment #1
Oh, just noticed the bug is assigned, but no review flag set, Christopher, was
that just an oversight?
--- Additional comment from Christopher Meng on 2014-04-14 05:07:58 CEST ---
(In reply to Rex Dieter from comment #2)
...
Oh, just noticed the bug is assigned, but no review flag set, Christopher,
was that just an oversight?
Sorry, it's not an omission. I'm busy recently, I will review this later in the
afternoon today.
--- Additional comment from Karel Volný on 2014-04-14 13:30:20 CEST ---
(In reply to Rex Dieter from comment #2)
...
Offhand looks like a good start, some initial suggestions (SHOULD are just
recommendations and/or a matter of taste, MUST are required):
thanks
...

SHOULD add

Requires: hicolor-icon-theme
instead of:
%dir %{_datadir}/icons/hicolor
%dir %{_datadir}/icons/hicolor/*
I prefer not to add dependencies that are not really needed ... multiple
ownership of a directory seems to me less evil
...

MUST add some icon scriptlets, see

https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
shame on me, I've completely forgotten about this gnomish stuff
(hm, now I wonder if I've overlooked it in the review, or such checkpoint is
missing? - RFE material? - but I'm not in the mood to investigate now with all
those errata in my queue :-/)
...

SHOULD include .spec comments clearly identifying items that are

workarounds for the bugs referenced in comment #1
added
@Christopher:
I'm going to attach new spec version here, is that okay for you, or do I have
to reupload it on the web together with new srpm?
--- Additional comment from Karel Volný on 2014-04-14 13:31:35 CEST ---
$ diff trojita.spec~ trojita.spec                                               
3c3                                                                             
< Release:        1%{?dist}                                                     
---
...
Release:        2%{?dist}
47a48
...
# https://bugs.kde.org/show_bug.cgi?id=332578
52a54,55
...
# change path for the library                                                                                                                                                                                                              
# https://bugs.kde.org/show_bug.cgi?id=332579
87a91,105
...
%post
/bin/touch --no-create %{_datadir}/icons/hicolor &>/dev/null || :
%postun
if [ $1 -eq 0 ] ; then
    /bin/touch --no-create %{_datadir}/icons/hicolor &>/dev/null
    /usr/bin/gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || :
fi
%posttrans
/usr/bin/gtk-update-icon-cache %{_datadir}/icons/hicolor &>/dev/null || :
88a107,110
...

Mon Apr 14 2014 Karel Volný kvolny@redhat.com 0.4.1-2

Fixed icon handling and added comments as per the package review
https://bugzilla.redhat.com/show_bug.cgi?id=1080411#c2

--- Additional comment from Christopher Meng on 2014-04-14 14:14:53 CEST ---
Please attach the fixed version, I will review it later.
--- Additional comment from Christopher Meng on 2014-04-16 03:08:19 CEST ---
Please also attach the fixed SRPM.
--- Additional comment from Karel Volný on 2014-04-16 13:07:47 CEST ---
Spec URL: http://kvolny.fedorapeople.org/trojita.spec
SRPM URL: http://kvolny.fedorapeople.org/trojita-0.4.1-2.fc20.src.rpm
Koji scratch builds:
F20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6745276
EPEL7: http://koji.fedoraproject.org/koji/taskinfo?taskID=6745284
Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=6745280
--- Additional comment from Christopher Meng on 2014-04-20 14:57:56 CEST ---
Package Review
==============
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
Issues:
=======
- Package installs a %{name}.desktop using desktop-file-install or desktop-
  file-validate if there is such a file.
===== MUST items =====
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
     Note: Unversioned so-files in private %_libdir subdirectory (see
     attachment). Verify they are not in ld path.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
     other legal requirements as defined in the legal section of Packaging
     Guidelines.
[x]: License field in the package spec file matches the actual license.
     Note: Checking patched sources after %prep for licenses. Licenses found:
     "GPL", "LGPL (v2)", "LGPL (v2.1 or later) (with incorrect FSF address)",
     "LGPL (v2 or later)", "Unknown or generated", "BSD (3 clause)". 65 files
     have unknown license. Detailed output of licensecheck:
BSD (3 clause)
--------------
trojita-0.4.1/src/Imap/Model/FindInterestingPart.cpp
trojita-0.4.1/src/Imap/Model/FindInterestingPart.h
trojita-0.4.1/src/Imap/Model/MailboxFinder.cpp
trojita-0.4.1/src/Imap/Model/MailboxFinder.h
trojita-0.4.1/src/XtConnect/MailSynchronizer.cpp
trojita-0.4.1/src/XtConnect/MailSynchronizer.h
trojita-0.4.1/src/XtConnect/MessageDownloader.cpp
trojita-0.4.1/src/XtConnect/MessageDownloader.h
trojita-0.4.1/src/XtConnect/SqlStorage.cpp
trojita-0.4.1/src/XtConnect/SqlStorage.h
trojita-0.4.1/src/XtConnect/XtConnect.cpp
trojita-0.4.1/src/XtConnect/XtConnect.h
trojita-0.4.1/src/XtConnect/main.cpp
GPL
---
trojita-0.4.1/src/AbookAddressbook/AbookAddressbook.cpp
trojita-0.4.1/src/AbookAddressbook/AbookAddressbook.h
trojita-0.4.1/src/AbookAddressbook/be-contacts.cpp
trojita-0.4.1/src/AbookAddressbook/be-contacts.h
trojita-0.4.1/src/AppVersion/SetCoreApplication.cpp
trojita-0.4.1/src/AppVersion/SetCoreApplication.h
trojita-0.4.1/src/Common/Application.cpp
trojita-0.4.1/src/Common/Application.h
trojita-0.4.1/src/Common/ConnectionId.cpp
trojita-0.4.1/src/Common/ConnectionId.h
trojita-0.4.1/src/Common/ConnectionMethod.h
trojita-0.4.1/src/Common/DeleteAfter.cpp
trojita-0.4.1/src/Common/DeleteAfter.h
trojita-0.4.1/src/Common/FileLogger.cpp
trojita-0.4.1/src/Common/FileLogger.h
trojita-0.4.1/src/Common/FindWithUnknown.h
trojita-0.4.1/src/Common/InvokeMethod.h
trojita-0.4.1/src/Common/Logging.h
trojita-0.4.1/src/Common/MetaTypes.cpp
trojita-0.4.1/src/Common/MetaTypes.h
trojita-0.4.1/src/Common/Paths.cpp
trojita-0.4.1/src/Common/Paths.h
trojita-0.4.1/src/Common/PortNumbers.h
trojita-0.4.1/src/Common/RingBuffer.h
trojita-0.4.1/src/Common/SettingsCategoryGuard.h
trojita-0.4.1/src/Common/SettingsNames.cpp
trojita-0.4.1/src/Common/SettingsNames.h
trojita-0.4.1/src/Common/SqlTransactionAutoAborter.h
trojita-0.4.1/src/Composer/ComposerAttachments.cpp
trojita-0.4.1/src/Composer/ComposerAttachments.h
trojita-0.4.1/src/Composer/ContentDisposition.h
trojita-0.4.1/src/Composer/MessageComposer.cpp
trojita-0.4.1/src/Composer/MessageComposer.h
trojita-0.4.1/src/Composer/PlainTextFormatter.cpp
trojita-0.4.1/src/Composer/PlainTextFormatter.h
trojita-0.4.1/src/Composer/Recipients.cpp
trojita-0.4.1/src/Composer/Recipients.h
trojita-0.4.1/src/Composer/ReplaceSignature.cpp
trojita-0.4.1/src/Composer/ReplaceSignature.h
trojita-0.4.1/src/Composer/SenderIdentitiesModel.cpp
trojita-0.4.1/src/Composer/SenderIdentitiesModel.h
trojita-0.4.1/src/Composer/SubjectMangling.cpp
trojita-0.4.1/src/Composer/SubjectMangling.h
trojita-0.4.1/src/Composer/Submission.cpp
trojita-0.4.1/src/Composer/Submission.h
trojita-0.4.1/src/Gui/AbstractAddressbook.h
trojita-0.4.1/src/Gui/AbstractPartWidget.h
trojita-0.4.1/src/Gui/AttachmentView.cpp
trojita-0.4.1/src/Gui/AttachmentView.h
trojita-0.4.1/src/Gui/AutoCompletion.cpp
trojita-0.4.1/src/Gui/AutoCompletion.h
trojita-0.4.1/src/Gui/CompleteMessageWidget.cpp
trojita-0.4.1/src/Gui/CompleteMessageWidget.h
trojita-0.4.1/src/Gui/ComposeWidget.cpp
trojita-0.4.1/src/Gui/ComposeWidget.h
trojita-0.4.1/src/Gui/ComposerAttachmentsList.cpp
trojita-0.4.1/src/Gui/ComposerAttachmentsList.h
trojita-0.4.1/src/Gui/ComposerTextEdit.cpp
trojita-0.4.1/src/Gui/ComposerTextEdit.h
trojita-0.4.1/src/Gui/EmbeddedWebView.cpp
trojita-0.4.1/src/Gui/EmbeddedWebView.h
trojita-0.4.1/src/Gui/EnvelopeView.cpp
trojita-0.4.1/src/Gui/EnvelopeView.h
trojita-0.4.1/src/Gui/ExternalElementsWidget.cpp
trojita-0.4.1/src/Gui/ExternalElementsWidget.h
trojita-0.4.1/src/Gui/FindBar.cpp
trojita-0.4.1/src/Gui/FindBar.h
trojita-0.4.1/src/Gui/FromAddressProxyModel.cpp
trojita-0.4.1/src/Gui/FromAddressProxyModel.h
trojita-0.4.1/src/Gui/IconLoader.h
trojita-0.4.1/src/Gui/LoadablePartWidget.cpp
trojita-0.4.1/src/Gui/LoadablePartWidget.h
trojita-0.4.1/src/Gui/MailBoxTreeView.cpp
trojita-0.4.1/src/Gui/MailBoxTreeView.h
trojita-0.4.1/src/Gui/MessageListWidget.cpp
trojita-0.4.1/src/Gui/MessageListWidget.h
trojita-0.4.1/src/Gui/MessageSourceWidget.cpp
trojita-0.4.1/src/Gui/MessageSourceWidget.h
trojita-0.4.1/src/Gui/MessageView.cpp
trojita-0.4.1/src/Gui/MessageView.h
trojita-0.4.1/src/Gui/MsgListView.cpp
trojita-0.4.1/src/Gui/MsgListView.h
trojita-0.4.1/src/Gui/OnePanelAtTimeWidget.cpp
trojita-0.4.1/src/Gui/OnePanelAtTimeWidget.h
trojita-0.4.1/src/Gui/OverlayWidget.cpp
trojita-0.4.1/src/Gui/OverlayWidget.h
trojita-0.4.1/src/Gui/PartWidget.cpp
trojita-0.4.1/src/Gui/PartWidget.h
trojita-0.4.1/src/Gui/PartWidgetFactory.cpp
trojita-0.4.1/src/Gui/PartWidgetFactory.h
trojita-0.4.1/src/Gui/PasswordDialog.cpp
trojita-0.4.1/src/Gui/PasswordDialog.h
trojita-0.4.1/src/Gui/ProgressPopUp.cpp
trojita-0.4.1/src/Gui/ProgressPopUp.h
trojita-0.4.1/src/Gui/ProtocolLoggerWidget.cpp
trojita-0.4.1/src/Gui/ProtocolLoggerWidget.h
trojita-0.4.1/src/Gui/ReplaceCharValidator.cpp
trojita-0.4.1/src/Gui/ReplaceCharValidator.h
trojita-0.4.1/src/Gui/SettingsDialog.cpp
trojita-0.4.1/src/Gui/SettingsDialog.h
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutConfigDialog.cpp
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutConfigDialog.h
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutConfigWidget.cpp
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutConfigWidget.h
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutHandler.cpp
trojita-0.4.1/src/Gui/ShortcutHandler/ShortcutHandler.h
trojita-0.4.1/src/Gui/SimplePartWidget.cpp
trojita-0.4.1/src/Gui/SimplePartWidget.h
trojita-0.4.1/src/Gui/Spinner.cpp
trojita-0.4.1/src/Gui/Spinner.h
trojita-0.4.1/src/Gui/TaskProgressIndicator.cpp
trojita-0.4.1/src/Gui/TaskProgressIndicator.h
trojita-0.4.1/src/Gui/UserAgentWebPage.cpp
trojita-0.4.1/src/Gui/UserAgentWebPage.h
trojita-0.4.1/src/Gui/Util.cpp
trojita-0.4.1/src/Gui/Util.h
trojita-0.4.1/src/Gui/Window.cpp
trojita-0.4.1/src/Gui/Window.h
trojita-0.4.1/src/Gui/main.cpp
trojita-0.4.1/src/Harmattan/main.cpp
trojita-0.4.1/src/Imap/ConnectionState.cpp
trojita-0.4.1/src/Imap/ConnectionState.h
trojita-0.4.1/src/Imap/Encoders.cpp
trojita-0.4.1/src/Imap/Encoders.h
trojita-0.4.1/src/Imap/Exceptions.cpp
trojita-0.4.1/src/Imap/Exceptions.h
trojita-0.4.1/src/Imap/Model/Cache.cpp
trojita-0.4.1/src/Imap/Model/Cache.h
trojita-0.4.1/src/Imap/Model/CacheLoadingMode.h
trojita-0.4.1/src/Imap/Model/CatenateData.h
trojita-0.4.1/src/Imap/Model/CombinedCache.cpp
trojita-0.4.1/src/Imap/Model/CombinedCache.h
trojita-0.4.1/src/Imap/Model/CopyMoveOperation.h
trojita-0.4.1/src/Imap/Model/DiskPartCache.cpp
trojita-0.4.1/src/Imap/Model/DiskPartCache.h
trojita-0.4.1/src/Imap/Model/DragAndDrop.cpp
trojita-0.4.1/src/Imap/Model/DragAndDrop.h
trojita-0.4.1/src/Imap/Model/FlagsOperation.cpp
trojita-0.4.1/src/Imap/Model/FlagsOperation.h
trojita-0.4.1/src/Imap/Model/FullMessageCombiner.cpp
trojita-0.4.1/src/Imap/Model/FullMessageCombiner.h
trojita-0.4.1/src/Imap/Model/ImapAccess.cpp
trojita-0.4.1/src/Imap/Model/ImapAccess.h
trojita-0.4.1/src/Imap/Model/ItemRoles.h
trojita-0.4.1/src/Imap/Model/MailboxMetadata.cpp
trojita-0.4.1/src/Imap/Model/MailboxMetadata.h
trojita-0.4.1/src/Imap/Model/MailboxModel.cpp
trojita-0.4.1/src/Imap/Model/MailboxModel.h
trojita-0.4.1/src/Imap/Model/MailboxTree.cpp
trojita-0.4.1/src/Imap/Model/MailboxTree.h
trojita-0.4.1/src/Imap/Model/MailboxTreeFwd.h
trojita-0.4.1/src/Imap/Model/MemoryCache.cpp
trojita-0.4.1/src/Imap/Model/MemoryCache.h
trojita-0.4.1/src/Imap/Model/Model.cpp
trojita-0.4.1/src/Imap/Model/Model.h
trojita-0.4.1/src/Imap/Model/ModelWatcher.cpp
trojita-0.4.1/src/Imap/Model/ModelWatcher.h
trojita-0.4.1/src/Imap/Model/MsgListModel.cpp
trojita-0.4.1/src/Imap/Model/MsgListModel.h
trojita-0.4.1/src/Imap/Model/NetworkPolicy.h
trojita-0.4.1/src/Imap/Model/NetworkWatcher.cpp
trojita-0.4.1/src/Imap/Model/NetworkWatcher.h
trojita-0.4.1/src/Imap/Model/OneMessageModel.cpp
trojita-0.4.1/src/Imap/Model/OneMessageModel.h
trojita-0.4.1/src/Imap/Model/ParserState.cpp
trojita-0.4.1/src/Imap/Model/ParserState.h
trojita-0.4.1/src/Imap/Model/PrettyMailboxModel.cpp
trojita-0.4.1/src/Imap/Model/PrettyMailboxModel.h
trojita-0.4.1/src/Imap/Model/PrettyMsgListModel.cpp
trojita-0.4.1/src/Imap/Model/PrettyMsgListModel.h
trojita-0.4.1/src/Imap/Model/QAIM_reset.h
trojita-0.4.1/src/Imap/Model/SQLCache.cpp
trojita-0.4.1/src/Imap/Model/SQLCache.h
trojita-0.4.1/src/Imap/Model/SpecialFlagNames.cpp
trojita-0.4.1/src/Imap/Model/SpecialFlagNames.h
trojita-0.4.1/src/Imap/Model/SubscribeUnSubscribeOperation.h
trojita-0.4.1/src/Imap/Model/SubtreeModel.cpp
trojita-0.4.1/src/Imap/Model/SubtreeModel.h
trojita-0.4.1/src/Imap/Model/TaskFactory.cpp
trojita-0.4.1/src/Imap/Model/TaskFactory.h
trojita-0.4.1/src/Imap/Model/TaskPresentationModel.cpp
trojita-0.4.1/src/Imap/Model/TaskPresentationModel.h
trojita-0.4.1/src/Imap/Model/ThreadingMsgListModel.cpp
trojita-0.4.1/src/Imap/Model/ThreadingMsgListModel.h
trojita-0.4.1/src/Imap/Model/UidSubmitData.h
trojita-0.4.1/src/Imap/Model/Utils.cpp
trojita-0.4.1/src/Imap/Model/Utils.h
trojita-0.4.1/src/Imap/Model/VisibleTasksModel.cpp
trojita-0.4.1/src/Imap/Model/VisibleTasksModel.h
trojita-0.4.1/src/Imap/Network/FileDownloadManager.cpp
trojita-0.4.1/src/Imap/Network/FileDownloadManager.h
trojita-0.4.1/src/Imap/Network/ForbiddenReply.cpp
trojita-0.4.1/src/Imap/Network/ForbiddenReply.h
trojita-0.4.1/src/Imap/Network/MsgPartNetAccessManager.h
trojita-0.4.1/src/Imap/Network/MsgPartNetworkReply.cpp
trojita-0.4.1/src/Imap/Network/MsgPartNetworkReply.h
trojita-0.4.1/src/Imap/Parser/Command.cpp
trojita-0.4.1/src/Imap/Parser/Command.h
trojita-0.4.1/src/Imap/Parser/Data.cpp
trojita-0.4.1/src/Imap/Parser/Data.h
trojita-0.4.1/src/Imap/Parser/LowLevelParser.cpp
trojita-0.4.1/src/Imap/Parser/LowLevelParser.h
trojita-0.4.1/src/Imap/Parser/MailAddress.cpp
trojita-0.4.1/src/Imap/Parser/MailAddress.h
trojita-0.4.1/src/Imap/Parser/Message.cpp
trojita-0.4.1/src/Imap/Parser/Message.h
trojita-0.4.1/src/Imap/Parser/Parser.cpp
trojita-0.4.1/src/Imap/Parser/Parser.h
trojita-0.4.1/src/Imap/Parser/Response.cpp
trojita-0.4.1/src/Imap/Parser/Response.h
trojita-0.4.1/src/Imap/Parser/Rfc5322HeaderParser.cpp
trojita-0.4.1/src/Imap/Parser/Rfc5322HeaderParser.generated.cpp
trojita-0.4.1/src/Imap/Parser/Rfc5322HeaderParser.h
trojita-0.4.1/src/Imap/Parser/Sequence.cpp
trojita-0.4.1/src/Imap/Parser/Sequence.h
trojita-0.4.1/src/Imap/Parser/ThreadingNode.cpp
trojita-0.4.1/src/Imap/Parser/ThreadingNode.h
trojita-0.4.1/src/Imap/Tasks/AppendTask.cpp
trojita-0.4.1/src/Imap/Tasks/AppendTask.h
trojita-0.4.1/src/Imap/Tasks/CopyMoveMessagesTask.cpp
trojita-0.4.1/src/Imap/Tasks/CopyMoveMessagesTask.h
trojita-0.4.1/src/Imap/Tasks/CreateMailboxTask.cpp
trojita-0.4.1/src/Imap/Tasks/CreateMailboxTask.h
trojita-0.4.1/src/Imap/Tasks/DeleteMailboxTask.cpp
trojita-0.4.1/src/Imap/Tasks/DeleteMailboxTask.h
trojita-0.4.1/src/Imap/Tasks/EnableTask.cpp
trojita-0.4.1/src/Imap/Tasks/EnableTask.h
trojita-0.4.1/src/Imap/Tasks/ExpungeMailboxTask.cpp
trojita-0.4.1/src/Imap/Tasks/ExpungeMailboxTask.h
trojita-0.4.1/src/Imap/Tasks/ExpungeMessagesTask.cpp
trojita-0.4.1/src/Imap/Tasks/ExpungeMessagesTask.h
trojita-0.4.1/src/Imap/Tasks/Fake_ListChildMailboxesTask.cpp
trojita-0.4.1/src/Imap/Tasks/Fake_ListChildMailboxesTask.h
trojita-0.4.1/src/Imap/Tasks/Fake_OpenConnectionTask.cpp
trojita-0.4.1/src/Imap/Tasks/Fake_OpenConnectionTask.h
trojita-0.4.1/src/Imap/Tasks/FetchMsgMetadataTask.cpp
trojita-0.4.1/src/Imap/Tasks/FetchMsgMetadataTask.h
trojita-0.4.1/src/Imap/Tasks/FetchMsgPartTask.cpp
trojita-0.4.1/src/Imap/Tasks/FetchMsgPartTask.h
trojita-0.4.1/src/Imap/Tasks/GenUrlAuthTask.cpp
trojita-0.4.1/src/Imap/Tasks/GenUrlAuthTask.h
trojita-0.4.1/src/Imap/Tasks/GetAnyConnectionTask.cpp
trojita-0.4.1/src/Imap/Tasks/GetAnyConnectionTask.h
trojita-0.4.1/src/Imap/Tasks/IdTask.cpp
trojita-0.4.1/src/Imap/Tasks/IdTask.h
trojita-0.4.1/src/Imap/Tasks/IdleLauncher.cpp
trojita-0.4.1/src/Imap/Tasks/IdleLauncher.h
trojita-0.4.1/src/Imap/Tasks/ImapTask.cpp
trojita-0.4.1/src/Imap/Tasks/ImapTask.h
trojita-0.4.1/src/Imap/Tasks/KeepMailboxOpenTask.cpp
trojita-0.4.1/src/Imap/Tasks/KeepMailboxOpenTask.h
trojita-0.4.1/src/Imap/Tasks/ListChildMailboxesTask.cpp
trojita-0.4.1/src/Imap/Tasks/ListChildMailboxesTask.h
trojita-0.4.1/src/Imap/Tasks/NoopTask.cpp
trojita-0.4.1/src/Imap/Tasks/NoopTask.h
trojita-0.4.1/src/Imap/Tasks/NumberOfMessagesTask.cpp
trojita-0.4.1/src/Imap/Tasks/NumberOfMessagesTask.h
trojita-0.4.1/src/Imap/Tasks/ObtainSynchronizedMailboxTask.cpp
trojita-0.4.1/src/Imap/Tasks/ObtainSynchronizedMailboxTask.h
trojita-0.4.1/src/Imap/Tasks/OfflineConnectionTask.cpp
trojita-0.4.1/src/Imap/Tasks/OfflineConnectionTask.h
trojita-0.4.1/src/Imap/Tasks/OpenConnectionTask.cpp
trojita-0.4.1/src/Imap/Tasks/OpenConnectionTask.h
trojita-0.4.1/src/Imap/Tasks/SortTask.cpp
trojita-0.4.1/src/Imap/Tasks/SortTask.h
trojita-0.4.1/src/Imap/Tasks/SubscribeUnsubscribeTask.cpp
trojita-0.4.1/src/Imap/Tasks/SubscribeUnsubscribeTask.h
trojita-0.4.1/src/Imap/Tasks/ThreadTask.cpp
trojita-0.4.1/src/Imap/Tasks/ThreadTask.h
trojita-0.4.1/src/Imap/Tasks/UidSubmitTask.cpp
trojita-0.4.1/src/Imap/Tasks/UidSubmitTask.h
trojita-0.4.1/src/Imap/Tasks/UnSelectTask.cpp
trojita-0.4.1/src/Imap/Tasks/UnSelectTask.h
trojita-0.4.1/src/Imap/Tasks/UpdateFlagsOfAllMessagesTask.cpp
trojita-0.4.1/src/Imap/Tasks/UpdateFlagsOfAllMessagesTask.h
trojita-0.4.1/src/Imap/Tasks/UpdateFlagsTask.cpp
trojita-0.4.1/src/Imap/Tasks/UpdateFlagsTask.h
trojita-0.4.1/src/MSA/AbstractMSA.cpp
trojita-0.4.1/src/MSA/AbstractMSA.h
trojita-0.4.1/src/MSA/FakeMSA.cpp
trojita-0.4.1/src/MSA/FakeMSA.h
trojita-0.4.1/src/MSA/ImapSubmit.cpp
trojita-0.4.1/src/MSA/ImapSubmit.h
trojita-0.4.1/src/MSA/SMTP.cpp
trojita-0.4.1/src/MSA/SMTP.h
trojita-0.4.1/src/MSA/Sendmail.cpp
trojita-0.4.1/src/MSA/Sendmail.h
trojita-0.4.1/src/Plugins/AddressbookPlugin.cpp
trojita-0.4.1/src/Plugins/AddressbookPlugin.h
trojita-0.4.1/src/Plugins/PasswordPlugin.cpp
trojita-0.4.1/src/Plugins/PasswordPlugin.h
trojita-0.4.1/src/Plugins/PluginInterface.h
trojita-0.4.1/src/Plugins/PluginJob.cpp
trojita-0.4.1/src/Plugins/PluginJob.h
trojita-0.4.1/src/Plugins/PluginManager.cpp
trojita-0.4.1/src/Plugins/PluginManager.h
trojita-0.4.1/src/Streams/DeletionWatcher.cpp
trojita-0.4.1/src/Streams/DeletionWatcher.h
trojita-0.4.1/src/Streams/FakeSocket.cpp
trojita-0.4.1/src/Streams/FakeSocket.h
trojita-0.4.1/src/Streams/IODeviceSocket.cpp
trojita-0.4.1/src/Streams/IODeviceSocket.h
trojita-0.4.1/src/Streams/Socket.cpp
trojita-0.4.1/src/Streams/Socket.h
trojita-0.4.1/src/Streams/SocketFactory.cpp
trojita-0.4.1/src/Streams/SocketFactory.h
trojita-0.4.1/src/Streams/TrojitaZlibStatus.h
trojita-0.4.1/src/XtConnect/XtCache.cpp
trojita-0.4.1/src/XtConnect/XtCache.h
trojita-0.4.1/src/be.contacts/main.cpp
trojita-0.4.1/tests/Composer/test_Composer_Submission.cpp
trojita-0.4.1/tests/Composer/test_Composer_Submission.h
trojita-0.4.1/tests/Composer/test_Composer_responses.cpp
trojita-0.4.1/tests/Composer/test_Composer_responses.h
trojita-0.4.1/tests/Composer/test_Html_formatting.cpp
trojita-0.4.1/tests/Composer/test_Html_formatting.h
trojita-0.4.1/tests/Imap/test_Imap_BodyParts.cpp
trojita-0.4.1/tests/Imap/test_Imap_BodyParts.h
trojita-0.4.1/tests/Imap/test_Imap_CopyAndFlagOperations.cpp
trojita-0.4.1/tests/Imap/test_Imap_CopyAndFlagOperations.h
trojita-0.4.1/tests/Imap/test_Imap_DisappearingMailboxes.cpp
trojita-0.4.1/tests/Imap/test_Imap_DisappearingMailboxes.h
trojita-0.4.1/tests/Imap/test_Imap_Idle.cpp
trojita-0.4.1/tests/Imap/test_Imap_Idle.h
trojita-0.4.1/tests/Imap/test_Imap_LowLevelParser.cpp
trojita-0.4.1/tests/Imap/test_Imap_LowLevelParser.h
trojita-0.4.1/tests/Imap/test_Imap_Message.cpp
trojita-0.4.1/tests/Imap/test_Imap_Message.h
trojita-0.4.1/tests/Imap/test_Imap_Model.cpp
trojita-0.4.1/tests/Imap/test_Imap_Model.h
trojita-0.4.1/tests/Imap/test_Imap_Offline.cpp
trojita-0.4.1/tests/Imap/test_Imap_Offline.h
trojita-0.4.1/tests/Imap/test_Imap_Parser_parse.cpp
trojita-0.4.1/tests/Imap/test_Imap_Parser_parse.h
trojita-0.4.1/tests/Imap/test_Imap_Responses.cpp
trojita-0.4.1/tests/Imap/test_Imap_Responses.h
trojita-0.4.1/tests/Imap/test_Imap_SelectedMailboxUpdates.cpp
trojita-0.4.1/tests/Imap/test_Imap_SelectedMailboxUpdates.h
trojita-0.4.1/tests/Imap/test_Imap_Tasks_CreateMailbox.cpp
trojita-0.4.1/tests/Imap/test_Imap_Tasks_CreateMailbox.h
trojita-0.4.1/tests/Imap/test_Imap_Tasks_DeleteMailbox.cpp
trojita-0.4.1/tests/Imap/test_Imap_Tasks_DeleteMailbox.h
trojita-0.4.1/tests/Imap/test_Imap_Tasks_ListChildMailboxes.cpp
trojita-0.4.1/tests/Imap/test_Imap_Tasks_ListChildMailboxes.h
trojita-0.4.1/tests/Imap/test_Imap_Tasks_ObtainSynchronizedMailbox.cpp
trojita-0.4.1/tests/Imap/test_Imap_Tasks_ObtainSynchronizedMailbox.h
trojita-0.4.1/tests/Imap/test_Imap_Tasks_OpenConnection.cpp
trojita-0.4.1/tests/Imap/test_Imap_Tasks_OpenConnection.h
trojita-0.4.1/tests/Imap/test_Imap_Threading.cpp
trojita-0.4.1/tests/Imap/test_Imap_Threading.h
trojita-0.4.1/tests/Misc/test_Rfc5322.cpp
trojita-0.4.1/tests/Misc/test_Rfc5322.h
trojita-0.4.1/tests/Misc/test_RingBuffer.cpp
trojita-0.4.1/tests/Misc/test_RingBuffer.h
trojita-0.4.1/tests/Misc/test_SenderIdentitiesModel.cpp
trojita-0.4.1/tests/Misc/test_SenderIdentitiesModel.h
trojita-0.4.1/tests/Misc/test_SqlCache.cpp
trojita-0.4.1/tests/Misc/test_SqlCache.h
trojita-0.4.1/tests/Misc/test_algorithms.cpp
trojita-0.4.1/tests/Misc/test_algorithms.h
trojita-0.4.1/tests/Utils/FakeCapabilitiesInjector.h
trojita-0.4.1/tests/Utils/LibMailboxSync.cpp
trojita-0.4.1/tests/Utils/LibMailboxSync.h
trojita-0.4.1/tests/Utils/ModelEvents.cpp
trojita-0.4.1/tests/Utils/ModelEvents.h
trojita-0.4.1/tests/Utils/TagGenerator.h
trojita-0.4.1/tests/Utils/headless_test.h
LGPL (v2 or later)
------------------
trojita-0.4.1/src/Imap/Model/kdeui-itemviews/kbihash_p.h
trojita-0.4.1/src/Imap/Model/kdeui-itemviews/kdescendantsproxymodel.cpp
trojita-0.4.1/src/Imap/Model/kdeui-itemviews/kdescendantsproxymodel.h
trojita-0.4.1/src/Imap/Parser/3rdparty/rfccodecs.cpp
trojita-0.4.1/src/Imap/Parser/3rdparty/rfccodecs.h
trojita-0.4.1/src/QmlSupport/QNAMWebView/plugin.cpp
trojita-0.4.1/src/QmlSupport/QNAMWebView/plugin.h
trojita-0.4.1/src/QmlSupport/QNAMWebView/qdeclarativewebview.cpp
trojita-0.4.1/src/QmlSupport/QNAMWebView/qdeclarativewebview_p.h
trojita-0.4.1/tests/Misc/test_rfccodecs.h
LGPL (v2)
---------
trojita-0.4.1/src/Imap/Parser/3rdparty/kcodecs.cpp
trojita-0.4.1/src/Imap/Parser/3rdparty/kcodecs.h
trojita-0.4.1/tests/Misc/test_rfccodecs.cpp
LGPL (v2.1 or later) (with incorrect FSF address)
-------------------------------------------------
trojita-0.4.1/src/XtConnect/xsqlquery.cpp
trojita-0.4.1/src/XtConnect/xsqlquery.h
Unknown or generated
--------------------
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-alignof.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-auto.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-auto_fail_compile.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-class_override_final.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-class_override_final_fail_compile.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-constexpr.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-cstdint_header.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-decltype.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-defaulted_functions.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-deleted_functions.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-deleted_functions_fail_compile.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-func_identifier.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-initializer_list.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-lambda.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-long_long.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-nullptr.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-nullptr_fail_compile.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-rvalue_references.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-sizeof_member.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-static_assert.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-static_assert_fail_compile.cxx
trojita-0.4.1/cmake/FindCXXFeatures/cxx11-test-variadic_templates.cxx
trojita-0.4.1/src/Gui/FlowLayout.cpp
trojita-0.4.1/src/Gui/FlowLayout.h
trojita-0.4.1/src/Gui/LineEdit.cpp
trojita-0.4.1/src/Gui/LineEdit.h
trojita-0.4.1/src/Gui/TagListWidget.cpp
trojita-0.4.1/src/Gui/TagListWidget.h
trojita-0.4.1/src/Gui/TagWidget.cpp
trojita-0.4.1/src/Gui/TagWidget.h
trojita-0.4.1/src/Harmattan/qmlapplicationviewer/qmlapplicationviewer.cpp
trojita-0.4.1/src/Harmattan/qmlapplicationviewer/qmlapplicationviewer.h
trojita-0.4.1/src/Imap/Model/ModelTest/modeltest.cpp
trojita-0.4.1/src/Imap/Model/ModelTest/modeltest.h
trojita-0.4.1/src/Imap/Network/MsgPartNetAccessManager.cpp
trojita-0.4.1/src/Imap/Parser/regenerate-ragel.sh
trojita-0.4.1/src/Streams/3rdparty/rfc1951.cpp
trojita-0.4.1/src/Streams/3rdparty/rfc1951.h
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths.cpp
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths.h
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths_mac.cpp
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths_os2.cpp
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths_unix.cpp
trojita-0.4.1/src/mimetypes-qt4/io/qstandardpaths_win.cpp
trojita-0.4.1/src/mimetypes-qt4/io/qsystemlibrary_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimedatabase.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimedatabase.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimedatabase_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimeglobpattern.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimeglobpattern_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimemagicrule.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimemagicrule_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimemagicrulematcher.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimemagicrulematcher_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimeprovider.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimeprovider_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimetype.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimetype.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimetype_p.h
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimetypeparser.cpp
trojita-0.4.1/src/mimetypes-qt4/mimetypes/qmimetypeparser_p.h
trojita-0.4.1/src/qwwsmtpclient/qwwsmtpclient.cpp
trojita-0.4.1/src/qwwsmtpclient/qwwsmtpclient.h
trojita-0.4.1/tests/imap-folder-sync/1.sh
trojita-0.4.1/tests/imap-folder-sync/2.sh
[x]: If the package is under multiple licenses, the licensing breakdown must
     be documented in the spec.
[!]: Package does not own files or directories owned by other packages.
     Note: Dirs in package are owned also by:
     /usr/share/icons/hicolor/32x32/apps(hicolor-icon-theme, fedora-logos,
     eom), /usr/share/icons/hicolor/scalable/apps(hicolor-icon-theme, fedora-
     logos, eom, hexchat), /usr/share/icons/hicolor/32x32(hicolor-icon-theme,
     fedora-logos, eom), /usr/share/icons/hicolor(hicolor-icon-theme, fedora-
     logos), /usr/share/icons/hicolor/scalable(hicolor-icon-theme, fedora-
     logos, eom, hexchat)
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
     Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: gtk-update-icon-cache is invoked in %postun and %posttrans if package
     contains icons.
     Note: icons in trojita
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
     (~1MB) or number of files.
     Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
     supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
     Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
     in its own file, then that file, containing the text of the license(s)
     for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: All build dependencies are listed in BuildRequires, except for any that
     are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
     beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
     work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
     in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
     %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
===== SHOULD items =====
Generic:
[x]: If the source package does not include license text(s) as a separate file
     from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Description and summary sections in the package spec file contains
     translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
     architectures.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
     $RPM_BUILD_ROOT)
[x]: Dist tag is present (not strictly required in GL).
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Uses parallel make %{?_smp_mflags} macro.
[x]: SourceX is a working URL.
[x]: Spec use %global instead of %define unless justified.
===== EXTRA items =====
Generic:
[ ]: Large data in /usr/share should live in a noarch subpackage if package is
     arched.
     Note: Arch-ed rpms have a total of 2181120 bytes in /usr/share
[x]: Rpmlint is run on all installed packages.
     Note: There are rpmlint messages (see attachment).
Rpmlint
-------
Checking: trojita-0.4.1-2.fc21.i686.rpm
          trojita-0.4.1-2.fc21.src.rpm
trojita.i686: W: spelling-error %description -l en_US Trojitá -> Trojan
trojita.i686: W: spelling-error %description -l en_US interoperable -> inter
operable, inter-operable, inoperable
trojita.i686: W: no-manual-page-for-binary be.contacts
trojita.i686: W: no-manual-page-for-binary trojita
trojita.src: W: spelling-error %description -l en_US Trojitá -> Trojan
trojita.src: W: spelling-error %description -l en_US interoperable -> inter
operable, inter-operable, inoperable
2 packages and 0 specfiles checked; 0 errors, 6 warnings.
Rpmlint (installed packages)
----------------------------
# rpmlint trojita
trojita.i686: W: spelling-error %description -l en_US Trojitá -> Trojan
trojita.i686: W: spelling-error %description -l en_US interoperable -> inter
operable, inter-operable, inoperable
trojita.i686: W: no-manual-page-for-binary be.contacts
trojita.i686: W: no-manual-page-for-binary trojita
1 packages and 0 specfiles checked; 0 errors, 4 warnings.
# echo 'rpmlint-done:'
Requires
--------
trojita (rpmlib, GLIBC filtered):
    /bin/sh
    libQtCore.so.4
    libQtGui.so.4
    libQtNetwork.so.4
    libQtSql.so.4
    libQtWebKit.so.4
    libc.so.6
    libgcc_s.so.1
    libgcc_s.so.1(GCC_3.0)
    libm.so.6
    libstdc++.so.6
    libstdc++.so.6(CXXABI_1.3)
    libtrojita_plugins.so
    libz.so.1
    rtld(GNU_HASH)
Provides
--------
trojita:
    application()
    application(trojita.desktop)
    libtrojita_plugins.so
    trojita
    trojita(x86-32)
Unversioned so-files
--------------------
trojita: /usr/lib/trojita/libtrojita_plugins.so
Source checksums
----------------
http://downloads.sourceforge.net/trojita/trojita-0.4.1.tar.bz2 :
  CHECKSUM(SHA256) this package     :
68a17498e70fd66966917fe4ecb90e6fe6f36f439c714552b8a97c004fe687de
  CHECKSUM(SHA256) upstream package :
68a17498e70fd66966917fe4ecb90e6fe6f36f439c714552b8a97c004fe687de
Generated by fedora-review 0.5.1 (bb9bf27) last change: 2013-12-13
Command line :/bin/fedora-review -rvn trojita-0.4.1-2.fc20.src.rpm
Buildroot used: fedora-rawhide-i386
Active plugins: Generic, Shell-api, C/C++
Disabled plugins: Java, Python, fonts, SugarActivity, Ocaml, Perl, Haskell, R,
PHP, Ruby
Disabled flags: EXARCH, EPEL5, BATCH, DISTTAG
--------------------------------------------
Besides above issues, you have:
1. From the koji build log:
-- The following features have been disabled:
 * WITH_HARMATTAN , Build MeeGo Harmattan version
 * WITH_QT5 , Build with Qt5 library
 * DEV_FATAL_WARNINGS , Turn build warnings into errors (developers only)
 * WITH_RAGEL , Build with Ragel library
WITH_RAGEL option is needed for building the parser from source, right?
ragel is in Fedora repo, you should use that one as BR.
2. %check section, since upstream has stated:
$ ctest -j 4 --output-on-failure
So I think you can use smp flags here(you can have a try)
3. Appdata file should be created.
--- Additional comment from Jan Kundrát on 2014-04-25 11:37:10 CEST ---
...
ragel is in Fedora repo, you should use that one as BR.
There's a small problem with that -- Ragel uses host's type sizes and
signedness when generating the .cpp file, which breaks the build when
cross-compiling on amd64 for arm because amd64's "char" can store 0-255, while
the ARM's one is -128-127. I reported this to Ragel developers [1], but the fix
is still not integrated AFAIK. I don't think that would affect native builds,
though, so perhaps it's OK to use this in Fedora.
...
$ ctest -j 4 --output-on-failure
So I think you can use smp flags here(you can have a try)
Note that it uses a different syntax from make -- "-j 4" vs make's "-j4".
Also be careful and do not activate parallel tests on build environments where
the system is under heavy load. Some of these tests operate on QTimers with a
deadline on the order of 20ms. I've seen problems with Ubuntu's build farm
which was fixed by switching to serial invocation of these tests.
...
Bug 332578 - superfluous x-test language
[https://bugs.kde.org/show_bug.cgi?id=332578]
Fixed now, will be fixed in the next release.
...
Bug 332579 - unversioned plugins library (rpmlint invalid-soname issue)
[https://bugs.kde.org/show_bug.cgi?id=332579]
Patches for this are welcome; I'm not that much familiar with cmake to know
what a perfect fix is.
[1] http://thread.gmane.org/gmane.comp.parsers.ragel.user/933
--- Additional comment from Karel Volný on 2014-05-05 22:38:51 CEST ---
thanks Jan for the update
sorry, I'm overloaded now, there was vacation on May 1st and there will be on
May 8th, things should settle after the next RHSCL release, dont't expect me to
get back to this earlier than second half of May :-(
--- Additional comment from Kevin Kofler on 2014-05-30 01:17:24 CEST ---
We indeed do not care about cross builds for Fedora packages.
--- Additional comment from Christopher Meng on 2014-07-29 05:49:55 CEST ---
Needinfo on me when you ready.
--- Additional comment from Karel Volný on 2014-10-27 15:59:39 CET ---
oops, this took longer than expected :-(
here's the new version:
Spec URL: http://kvolny.fedorapeople.org/trojita.spec
SRPM URL: http://kvolny.fedorapeople.org/trojita-0.4.1-3.fc20.src.rpm
it only adds ragel dependency ...
0) "above issues" - I see just
[!]: Package does not own files or directories owned by other packages.
- anything else?
as for the icon dirs ownership, I've already responded in comment #4 and after
re-reading
http://fedoraproject.org/wiki/Packaging:Guidelines#File_and_Directory_Owners...
I believe not requiring otherwise unneeded package is the correct solution
1) ragel is now used
2) following the discussion in comment #10, I believe it is better not to mess
with it; when I've done my testbuild, tests took just 11 seconds so there's not
much to gain here
3) appdata got added upstream:
http://quickgit.kde.org/?p=trojita.git&a=commit&h=66cf5af897741d9655...
so I'd prefer to just wait for the next version and pick it up from the
upstream tarball rather than to add own copy now
--- Additional comment from Kevin Kofler on 2014-10-27 18:48:51 CET ---
I still disagree about your handling of the hicolor icon theme.
The hicolor-icon-theme package is a "*filesystem"-type package. Therefore, this
section of the guidelines applies:
...
An exception
Sometimes, it may be preferable for such directories to be owned by an
"artificial filesystem" package, such as mozilla-filesystem. These packages are
designed to be explicitly required when other packages store files in their
directories, thus, in such situations, these packages should explicitly Require
the artificial filesystem package and not multiply own those directories.
Packagers should consider the number of affected directories and packages when
determining whether to create artificial filesystem packages, and use their own
best judgement to determine if this is necessary or not.
--- Additional comment from Karel Volný on 2014-10-29 14:39:31 CET ---
(In reply to Kevin Kofler from comment #15)
...
The hicolor-icon-theme package is a "*filesystem"-type package.
the catch is that it is "*filesystem"-type but not "*-filesystem"
it carries non-empty payload (I mean actual files of nonzero size, not just the
directory structure)
this is Qt application - it doesn't need 100+ MB GTK icon cache, and more
generally the post scriptlets are even allowed to ignore if
gtk-update-icon-cache is missing
if something like "icons-filesystem" is going to be created, I'll gladly add a
dependency on it, but the current implementation of "hicolor-icon-theme"
doesn't match what I think the abovequoted exception refers to, despite it is
very close ... if it'd be added to the guidelines along the reference to the
"man" package, there would be no space for doubt of course :-)
--- Additional comment from Kevin Kofler on 2014-10-30 00:32:25 CET ---
The icon-theme.cache file is actually ghosted, which means it is NOT part of
the installed size of the hicolor-icon-theme package. The file is NOT created
by installing the hicolor-icon-theme package, but it is created on your system
by gtk-update-icon-cache, if (and only if) gtk-update-icon-cache is installed.
And gtk-update-icon-cache will create that file whether you have the
hicolor-icon-theme package installed or not.
The actual size of hicolor-icon-theme is only these files:
-rw-r--r--   1 root root     27095 27. Feb 2014 
/usr/share/icons/hicolor/index.theme
required by the freedesktop.org icon theme spec. Strictly speaking, installing
an icon to hicolor without that file being present is meaningless.
-rw-r--r--   1 root root      2688 17. Dez 2013 
/usr/share/doc/hicolor-icon-theme-0.13/ChangeLog
-rw-r--r--   1 root root     17992 17. Dez 2013 
/usr/share/doc/hicolor-icon-theme-0.13/COPYING
-rw-r--r--   1 root root       433 17. Dez 2013 
/usr/share/doc/hicolor-icon-theme-0.13/README
Documentation and license files, as per the packaging guidelines.
The total installed size of hicolor-icon-theme is a whopping 48208 bytes. I
would understand your complaint if we were still shipping the original
hicolor-icon-theme from KDE 2, which contained actual icons (that's where the
"hicolor" name historically comes from), but as it stands, you are just
splitting hairs (or not understanding the meaning of %ghost in RPM).
--- Additional comment from Karel Volný on 2014-11-03 17:47:59 CET ---
(In reply to Kevin Kofler from comment #17)
...
The icon-theme.cache file is actually ghosted, which means it is NOT part of
the installed size of the hicolor-icon-theme package.
okay, sorry, I've used improper logical shortcut no file installed = no space
eaten
...
And gtk-update-icon-cache will create that file whether you have the
hicolor-icon-theme package installed or not.
well, in that case, wouldn't that be better if the file is owned by gtk2, which
provides gtk-update-icon-cache?
...
-rw-r--r--   1 root root     27095 27. Feb 2014  /usr/share/icons/hicolor/index.theme
required by the freedesktop.org icon theme spec. Strictly speaking, installing
an icon to hicolor without that file being present is meaningless.
strictly speaking yes, but in case you're not interested in icons, it's still a
little bit better to just install the unwanted icon (when it is not in separate
package) rather than to install icon+theme
...
The total installed size of hicolor-icon-theme is a whopping 48208 bytes. I
would understand your complaint if we were still shipping the original
hicolor-icon-theme from KDE 2, which contained actual icons (that's where
the "hicolor" name historically comes from), but as it stands, you are just
splitting hairs (or not understanding the meaning of %ghost in RPM).
I don't think the approach "it's just X bytes, next to nothing, let's waste it"
is the best ...
but that would be a different point
I'm not quite sure about the interpretation of the packaging guidelines ... if
you say depending on hicolor-icon-theme is the best thing to do (and I don't
say it isn't, just that in my POV it's not that clear), please file a request
to change the guidelines to mandate hicolor-icon-theme dependency for packages
installing icons, and once it gets changed, we are over with this discussion -
and as a bonus, bugs for other packages owning those directories could be filed
then so that the mess gets cleaned up
wouldn't it be better to make it absolutely clear for all the packagers rather
than arguing here?
--- Additional comment from Rex Dieter on 2014-12-31 22:58:34 CET ---
Ping, Christopher, can you continue with the review?  it's been a couple of
months since comment #14 requesting feedback.
--- Additional comment from Björn "besser82" Esser on 2015-02-06 18:32:25 CET
---
Taking over here…
--- Additional comment from Mario Blättermann on 2015-02-07 17:44:09 CET ---
%doc LICENSE README
Please move the license text to %license:
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/Li...
--- Additional comment from Raphael Groner on 2015-02-07 21:13:21 CET ---
(In reply to Mario Blättermann from comment #21)
...
%doc LICENSE README
Please move the license text to %license:
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/
LicensingGuidelines#License_Text
This won't work properly when a package for f20 is needed.
Karel, do you plan to package also for f20? If yes, you have to use a %if
conditional to exclude %license and keep %doc.
--- Additional comment from Zbigniew Jędrzejewski-Szmek on 2015-02-08 01:28:01
CET ---
(In reply to Raphael Groner from comment #22)
...
Karel, do you plan to package also for f20? If yes, you have to use a %if
conditional to exclude %license and keep %doc.
No. See https://fedorahosted.org/fpc/ticket/411#comment:11 for a better
solution.
--- Additional comment from Karel Volný on 2015-02-09 13:32:37 CET ---
(In reply to Raphael Groner from comment #22)
...
(In reply to Mario Blättermann from comment #21)
...
%doc LICENSE README
Please move the license text to %license:
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines?rd=Packaging/
LicensingGuidelines#License_Text
thanks for heads up
...
This won't work properly when a package for f20 is needed.
Karel, do you plan to package also for f20? If yes, you have to use a %if
conditional to exclude %license and keep %doc.
no, I do not plan to get it into any older releases
btw, has anyone taken the hicolor-icon-theme issue anywhere, have I missed
something?
--- Additional comment from Raphael Groner on 2015-02-09 14:31:02 CET ---
(In reply to Karel Volný from comment #24)
…
...
btw, has anyone taken the hicolor-icon-theme issue anywhere, have I missed
something?
...
From your spec file:
%dir %{_datadir}/icons/hicolor
%dir %{_datadir}/icons/hicolor/*
%dir %{_datadir}/icons/hicolor/*/apps
Remove these lines and put "Requires: hicolor-icon-theme". The guideline says
to not own folders from a filesystem package, hicolor-icon-theme owns those
folders already. This is the common way to do it.
Further, insert the ScriptletSnippets from your comment #5, for proper handling
of icon cache as needed in several desktops.
https://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache
--- Additional comment from Karel Volný on 2015-02-09 15:09:59 CET ---
(In reply to Raphael Groner from comment #25)
...
Remove these lines and put "Requires: hicolor-icon-theme". The guideline
says to not own folders from a filesystem package, hicolor-icon-theme owns
those folders already. This is the common way to do it.
please re-read comments 16-18
the fact that a package owns directories doesn't automatically make it "a
filesystem package" - in fact the guidelines[*] give an example where a package
(gtk-doc) owns a directory (/usr/share/gtk-doc/) and yet the second package
(evolution) _should own_ that directory
[*]
https://fedoraproject.org/wiki/Packaging:Guidelines#The_directory_is_owned_b...
give me some official statement that hicolor-icon-theme is an exception both to
naming scheme (it's not *-filesystem) and both that it can include not just
(empty) directories
e.g., the easiest, have this bullet:
"any directories owned by the filesystem, man, or other explicitly created
-filesystem packages"
changed to
"any directories owned by the filesystem, man, hicolor-icon-theme, or other
explicitly created -filesystem packages"
otherwise, formally, I cannot agree with your interpretation of the guidelines,
and personally, I do not like pulling in unnecessary dependencies
--- Additional comment from Michael Schwendt (Fedora Packager Sponsors Group)
on 2015-02-09 18:52:15 CET ---
Please don't be so stubborn.
There is general agreement about explicitly requiring hicolor-icon-theme
package instead of _trying_ to "own" all used directories. See e.g.
  https://lists.fedoraproject.org/pipermail/devel/2012-January/161098.html
One(/the?) reason why it's still not in the guidelines could be that the FPC
often expects a ticket to be opened and a guidelines change draft to be
attached before anything would happen at all. 
  https://lists.fedoraproject.org/pipermail/devel/2012-January/161101.html
And when that doesn't happen, an issue is forgotten, and nobody from the FPC
drives it forward "automatically" either.
--- Additional comment from Kevin Kofler on 2015-02-25 19:02:20 CET ---
As I already said in comment #15 and explained in comment #17, I also agree
that you MUST require hicolor-icon-theme instead of owning the same
directories.
--- Additional comment from Raphael Groner on 2015-02-28 22:42:37 CET ---
SPEC: https://raphgro.fedorapeople.org/review/qt/trojita/trojita.spec
SRPM:
https://raphgro.fedorapeople.org/review/qt/trojita/trojita-0.5-1.fc21.src.rp...
rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9102479
* Sat Feb 28 2015 Raphael Groner <projects.rg (AT) smart.ms> - 0.5-1
- clean files section and R: hicolor-icon-theme
- introduce license macro
- use name macro generally
- new upstream version 0.5
- distribute doxygen files
--- Additional comment from Raphael Groner on 2015-02-28 23:12:02 CET ---
Fixed help2man. New rawhide scratch:
http://koji.fedoraproject.org/koji/taskinfo?taskID=9102581
--- Additional comment from Raphael Groner on 2015-03-30 15:59:39 CEST ---
any news here?
--- Additional comment from Raphael Groner on 2015-04-01 01:07:02 CEST ---
HTML formatting seems to fail with Qt5, see upstream bug 345739.
Release #2 with enabled Qt5.
SPEC: https://raphgro.fedorapeople.org/review/qt/trojita/trojita.spec
SRPM:
https://raphgro.fedorapeople.org/review/qt/trojita/trojita-0.5-2.fc21.src.rp...
rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9386480
--- Additional comment from Christopher Meng on 2015-04-01 06:39:52 CEST ---
Taking back.
So, who is gonna submit the final package?
--- Additional comment from Raphael Groner on 2015-04-01 11:26:34 CEST ---
Not sure how long to wait for Karel as the original requester. I could take
this package and open a new bug for the request.
--- Additional comment from Christian Dersch on 2015-04-01 16:38:43 CEST ---
(In reply to Raphael Groner from comment #29)
...
SPEC: https://raphgro.fedorapeople.org/review/qt/trojita/trojita.spec
SRPM:
https://raphgro.fedorapeople.org/review/qt/trojita/trojita-0.5-1.fc21.src.rp...
rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9102479

Sat Feb 28 2015 Raphael Groner <projects.rg (AT) smart.ms> - 0.5-1

clean files section and R: hicolor-icon-theme
introduce license macro
use name macro generally
new upstream version 0.5
distribute doxygen files

Any need for the doxygen stuff (I get a -doc package with >300mb size)? It
generates the developer documentation, no user documentation. As trojita is a
simple GUI application and no library for development I would remove it.
Greetings,
Christian
--- Additional comment from Raphael Groner on 2015-04-01 19:12:10 CEST ---
* Wed Apr 01 2015 Raphael Groner <> - 0.5-3
- ease switching build with qt4 or qt5
- disable doxygen
- remove toolkit from summary
- use build subfolder
- improve tests execution
SPEC: https://raphgro.fedorapeople.org/review/qt/trojita/trojita.spec
SRPM:
https://raphgro.fedorapeople.org/review/qt/trojita/trojita-0.5-3.fc21.src.rp...
rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9393885
--- Additional comment from Jan Kundrát on 2015-04-01 19:26:37 CEST ---
I got curious what's in the generated manpage. It says "QXcbConnection: Could
not connect to display". Does it try to e.g. parse output of `trojita --help`?
That requires X11.
With my upstream hat on, I strongly prefer to always run the whole test suite
when doing builds. If a test fails, that's a bug which should be fixed and not
papered over. That's what I am not thrilled by disabling the
test_Html_formatting test, but maybe it's just a temporary thing. I believe
that the bug is in the interaction of Fedora's Qt packaging and Fedora's Xvfb,
because the tests work on CentOS 7 (Trojita's CI) and some ancient Ubuntu
(KDE's CI). My suggestion here is to ask Fedora Qt5 packagers for help.
--- Additional comment from Rex Dieter on 2015-04-01 19:30:07 CEST ---
Re: test suite
We can/will certainly investigate further, but it's not worth blocking the pkg
review for.  Once imported, testing will be much easier (and collaborative).
--- Additional comment from Raphael Groner on 2015-04-01 19:48:58 CEST ---
It is really strange with the html test case. I tried now to only execute this
one but it works as alone!??
http://koji.fedoraproject.org/koji/watchlogs?taskID=9394112
Maybe it happens only cause of running out of memory, e.g. when doxygen is
enabled what we do not need.
--- Additional comment from Jan Kundrát on 2015-04-01 20:00:16 CEST ---
@Rex, OK, sorry for noise.
@Raphael, I'm a bit confused -- the log you pointed at says that you asked
ctest to skip that test, so it isn't laucnhed at all. Ctest reports 26 tests in
total, while we've 27 tests -- as expected, given the -E option.
--- Additional comment from Kevin Kofler on 2015-04-01 20:06:50 CEST ---
Many Fedora packages don't run any test suite at all, or ignore failures. It's
quite common that tests don't pass in Koji for whatever reason.
--- Additional comment from Raphael Groner on 2015-04-01 20:07:07 CEST ---
--- Additional comment from Raphael Groner on 2015-04-01 20:55:47 CEST ---
The relevant source snippet seems to be that:
https://webcache.googleusercontent.com/search?q=cache:AY5C2IQNJYQJ:https://q...
(using google's webcache cause gitorious is not really usable)
--- Additional comment from Raphael Groner on 2015-04-02 17:05:47 CEST ---
Closing here cause the original requester is missed since weeks (its last
comment #26 in February).
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=928937
[Bug 928937] Qt-related package review tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1080411
[Bug 1080411] Review Request: trojita - IMAP e-mail client
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component

[Bug 1208582] New: Review Request: trojita - IMAP e-mail client