Bug 146436

Summary:	digikam try to include Exiv2 library headers
Product:	[Applications] digikam	Reporter:	Marcin Garski <mgarski>
Component:	Bundle-Tarball	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	caulier.gilles
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:	0.9.2
Sentry Crash Report:

Description Marcin Garski 2007-06-06 00:37:00 UTC

Version:            (using KDE Devel)
Installed from:    Compiled sources
OS:                Linux

On digiKam website I can see such text about 0.9.2 release:
"- Removing direct Exiv2 library depency. libkexiv2 interface is used everywhere instead."

But if we look into the libs/widgets/metadata/makernotewidget.cpp file you will notice:
// LibExiv2 includes.

#include <exiv2/exif.hpp>
#include <exiv2/tags.hpp>
#include <exiv2/ifd.hpp>

So digiKam directly requires Exiv2 and won't compile if this headers will not be installed.

Comment 1 caulier.gilles 2007-06-06 00:45:45 UTC

SVN commit 671981 by cgilles:

fix broken compilation duing obsolete Exiv2 depency
BUG: 146436


 M  +0 -13     makernotewidget.cpp  


--- branches/extragear/kde3/graphics/digikam/libs/widgets/metadata/makernotewidget.cpp #671980:671981
@@ -22,13 +22,6 @@
  * 
  * ============================================================ */
 
-// C++ includes.
-
-#include <cstdlib>
-#include <cstdio>
-#include <cassert>
-#include <string>
-
 // Qt includes.
 
 #include <qmap.h>
@@ -38,12 +31,6 @@
 
 #include <klocale.h>
 
-// LibExiv2 includes.
-
-#include <exiv2/exif.hpp>
-#include <exiv2/tags.hpp>
-#include <exiv2/ifd.hpp>
-
 // Local includes.
 
 #include "ddebug.h"