It was not optional
=> so it's not a bug.

"
  find_package(Baloo 4.14.0 QUIET CONFIG)
  set_package_properties(Baloo PROPERTIES DESCRIPTION "The Baloo libraries" URL "http://www.kde.org" TYPE REQUIRED PURPOSE "Baloo provides search capabilities in KMail and Akonadi")
"
=> it's required

(In reply to Laurent Montel from comment #1)
> It was not optional
> => so it's not a bug.

ahem, I've referenced a comment from *Baloo developer* who says that Baloo should be uninstallable

that implies that the compilation should not depend on it; either as a direct implication (if the "uninstallability" is excercised before the compilation) and either as indirect implication (if compiled with Baloo and then it gets removed, you'd get executables broken due to missing libraries)

so in no way I can see how this could _not_ be a bug


> "
>   find_package(Baloo 4.14.0 QUIET CONFIG)
>   set_package_properties(Baloo PROPERTIES DESCRIPTION "The Baloo libraries"
> URL "http://www.kde.org" TYPE REQUIRED PURPOSE "Baloo provides search
> capabilities in KMail and Akonadi")
> "
> => it's required

then, in addition, there's some bug with cmake if the configure phase output lists it  in the section of optional and not required packages ... feel free to open a new report for it

the full detection output is as follows:


-- The following OPTIONAL packages have been found:

 * Perl
 * KdepimLibs (required version >= 4.14.3) , The KDEPIM libraries , <http://www.kde.org>
 * Boost (required version >= 1.34.0) , Boost C++ Libraries , <http://www.boost.org>
   Boost is required for building most KDEPIM applications
 * Akonadi (required version >= 1.12.90) , Akonadi server libraries , <http://pim.kde.org/akonadi>
   Akonadi is required to build KDEPIM
 * ZLIB , The Zlib compression library , <http://www.zlib.net>
 * Gpgme
 * Threads
 * PkgConfig
 * OpenSSL
 * Grantlee (required version >= 0.3.0) , The Grantlee Template System , <http://www.gitorious.org/grantlee/pages/Home>
   Grantlee is requires for kmail and templating, theming for KJots, KaddressBook, KNotes and MessageViewer(KMail).
 * Xsltproc , XSLT processor from libxslt
   Required to generate D-Bus interfaces.
 * QJSON , QJSON , <http://qjson.sourceforge.net/>
   Qt library for handling JSON data
 * LibKGAPI2 (required version >= 2.2.0) , KDE-based library for accessing various Google services , <https://projects.kde.org/libkgapi>
   LibKGAPI is required to build Google Drive Storage Service
 * Assuan2

-- The following RECOMMENDED packages have been found:

 * QGpgme , The QGpgMe library , <http://www.kde.org>
   QGpgME is required to build KMail, KOrganizer and Kleopatra

-- The following REQUIRED packages have been found:

 * Automoc4
 * Phonon (required version >= 4.3.80)
 * KDE4Internal
 * KDE4 (required version >= 4.13.0)
 * X11
 * Qt4

-- The following OPTIONAL packages have not been found:

 * Baloo (required version >= 4.14.0) , The Baloo libraries , <http://www.kde.org>
   Baloo provides search capabilities in KMail and Akonadi
 * Prison , The Prison library , <http://projects.kde.org/prison>
   Needed to show mobile barcodes of your contacts

(In reply to kavol from comment #2)
> (In reply to Laurent Montel from comment #1)
> > It was not optional
> > => so it's not a bug.
> 
> ahem, I've referenced a comment from *Baloo developer* who says that Baloo
> should be uninstallable
> 
> that implies that the compilation should not depend on it; either as a
> direct implication (if the "uninstallability" is excercised before the
> compilation) and either as indirect implication (if compiled with Baloo and
> then it gets removed, you'd get executables broken due to missing libraries)
> 
> so in no way I can see how this could _not_ be a bug

Baloo developper has created this patch.
So it was required and there is never optional so no it's not a bug.
Perhaps you need a feature but it's not a bug because never supported.

So I will look at for kf5 but not for 4.14. I will not break all for it.
If you don't want it. remove akonadi_baloo_indexer.

> 
> 
> > "
> >   find_package(Baloo 4.14.0 QUIET CONFIG)
> >   set_package_properties(Baloo PROPERTIES DESCRIPTION "The Baloo libraries"
> > URL "http://www.kde.org" TYPE REQUIRED PURPOSE "Baloo provides search
> > capabilities in KMail and Akonadi")
> > "
> > => it's required
> 
> then, in addition, there's some bug with cmake if the configure phase output
> lists it  in the section of optional and not required packages ... feel free
> to open a new report for it

I don't know why it reported it as optional but it's not my problem.
If you want to open a bug report about it, report to kdelibs

Regards

> 
> the full detection output is as follows:
> 
> 
> -- The following OPTIONAL packages have been found:
> 
>  * Perl
>  * KdepimLibs (required version >= 4.14.3) , The KDEPIM libraries ,
> <http://www.kde.org>
>  * Boost (required version >= 1.34.0) , Boost C++ Libraries ,
> <http://www.boost.org>
>    Boost is required for building most KDEPIM applications
>  * Akonadi (required version >= 1.12.90) , Akonadi server libraries ,
> <http://pim.kde.org/akonadi>
>    Akonadi is required to build KDEPIM
>  * ZLIB , The Zlib compression library , <http://www.zlib.net>
>  * Gpgme
>  * Threads
>  * PkgConfig
>  * OpenSSL
>  * Grantlee (required version >= 0.3.0) , The Grantlee Template System ,
> <http://www.gitorious.org/grantlee/pages/Home>
>    Grantlee is requires for kmail and templating, theming for KJots,
> KaddressBook, KNotes and MessageViewer(KMail).
>  * Xsltproc , XSLT processor from libxslt
>    Required to generate D-Bus interfaces.
>  * QJSON , QJSON , <http://qjson.sourceforge.net/>
>    Qt library for handling JSON data
>  * LibKGAPI2 (required version >= 2.2.0) , KDE-based library for accessing
> various Google services , <https://projects.kde.org/libkgapi>
>    LibKGAPI is required to build Google Drive Storage Service
>  * Assuan2
> 
> -- The following RECOMMENDED packages have been found:
> 
>  * QGpgme , The QGpgMe library , <http://www.kde.org>
>    QGpgME is required to build KMail, KOrganizer and Kleopatra
> 
> -- The following REQUIRED packages have been found:
> 
>  * Automoc4
>  * Phonon (required version >= 4.3.80)
>  * KDE4Internal
>  * KDE4 (required version >= 4.13.0)
>  * X11
>  * Qt4
> 
> -- The following OPTIONAL packages have not been found:
> 
>  * Baloo (required version >= 4.14.0) , The Baloo libraries ,
> <http://www.kde.org>
>    Baloo provides search capabilities in KMail and Akonadi
>  * Prison , The Prison library , <http://projects.kde.org/prison>
>    Needed to show mobile barcodes of your contacts

(In reply to Laurent Montel from comment #3)
...
> Baloo developper has created this patch.
> So it was required and there is never optional so no it's not a bug.
> Perhaps you need a feature but it's not a bug because never supported.

ahem, sorry, but I still do not follow ...

there is a discrepancy between the statement and the actual code

so one of them is wrong - "buggy"

and as the statement is part of a broader discussion, not just a random thought, I'd say the bug is in the code and not in the statement

it's not a feature in the sense of "new functionality", it is a problem with already existing code

so probably reassigning this to the actual author of the code to make it compliant with what he says the code should do would be better than just closing the bugreport with something like "he coded it this way so it should be so" - if I get your words right

(if not, I apologise; also, I don't know much about processes in this bugzilla instance, if the assignee is tied to the component and cannot be changed or whatever ... I just present my POV which doesn't have to be best due to missing context)

> So I will look at for kf5

cool, thanks

> but not for 4.14. I will not break all for it.
> If you don't want it. remove akonadi_baloo_indexer.

ok, I guess I can live with that as long as `rm /usr/bin/baloo*` doesn't break anything I use

...
> I don't know why it reported it as optional but it's not my problem.
> If you want to open a bug report about it, report to kdelibs

ok, bug 342477

*** Bug 342477 has been marked as a duplicate of this bug. ***

As it's still not clear to you: Baloo is mandatory for kdepimlibs. That's not a bug.

s/kdepimlibs/kdepim/

It's the gentoo build system that changes it to optional.

(In reply to Christophe Giboudeaux from comment #6)
> As it's still not clear to you: Baloo is mandatory for kdepimlibs.

that part is clear to me just by looking at the code, however correctness of the code cannot be proved by referencing itself

> That's not a bug.

why?

as already said, the code contradicts to what was said how it should behave

so the code not doing what it should do according to its developer is not a bug because ...?

(In reply to Christophe Giboudeaux from comment #7)
> It's the gentoo build system that changes it to optional.

oops, I apologize, you're right ... I must have mistaken directories when examining the sources and I didn't have any suspicion as there isn't any such patch and I hadn't realized it can get changed elsewhere; running it through diff I see the REQUIRED => OPTIONAL change now ... sorry :-(