Bug 155271 - configure suggests wrong parameter for libjasper
Summary: configure suggests wrong parameter for libjasper
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Portability-Runtime (show other bugs)
Version: unspecified
Platform: RedHat Enterprise Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-08 12:34 UTC by raspl
Modified: 2017-07-16 04:57 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In: 0.9.4


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description raspl 2008-01-08 12:34:23 UTC
Version:           0.9.3 (using KDE KDE 3.5.4)
Installed from:    RedHat RPMs
Compiler:          gcc 
OS:                Linux

If libjasper is not present on the system, the following error is reported by the configure script:

  -- libjasper library found........ NO

  digiKam needs libjasper to support JPEG2000.
  You need to install the correct version (>= 1.7.0).
  Libjasper website is at http://www.ece.uvic.ca/~mdadams/jasper
  Important note: libjaspeer has to be configured with --enable-dynamic
  as otherwise the required dynamic libraries are not created.

Besides a minor spelling error (libjaspe_e_r), the switch '--enable-dynamic' does not exist in libjasper's configure script. The correct switch is '--enable-shared=yes'. Since libjasper's configure script doesn't report superfluous or noexisting options, this will go unnoticed and can cause headaches...

Here's a trivial patch:

--- configure.old       2008-01-08 12:33:14.000000000 +0100
+++ configure   2008-01-08 12:33:36.000000000 +0100
@@ -44210,7 +44210,7 @@ if test "x$have_jasper" != "xyes"; then
   echo "digiKam needs libjasper to support JPEG2000. "
   echo "You need to install the correct version (>= 1.7.0)."
   echo "Libjasper website is at http://www.ece.uvic.ca/~mdadams/jasper"
-  echo "Important note: libjaspeer has to be configured with --enable-dynamic"
+  echo "Important note: libjasper has to be configured with --enable-shared=yes"
   echo "as otherwise the required dynamic libraries are not created."
   echo ""
   all_tests=bad
Comment 1 Arnd Baecker 2008-01-08 12:52:12 UTC
Raspl, thanks a lot for the report and patch.
I applied it to configure.in.bot in current svn. 
Best, Arnd
Comment 2 Arnd Baecker 2008-01-08 12:53:02 UTC
SVN commit 758580 by abaecker:

Correction of the configure script to suggest the right option for libjasper

CCBUGS: 155271
TODO:KDE4PORT



 M  +2 -1      NEWS  
 M  +1 -1      configure.in.bot  


WebSVN link: http://websvn.kde.org/?view=rev&revision=758580
Comment 3 caulier.gilles 2008-03-18 12:47:37 UTC
This file is already fixed...

Gilles Caulier