[digikam] [Bug 350915] New: digikam-4.12.0 fails to compile with jpeg-9a

classic Classic list List threaded Threaded
8 messages Options
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] New: digikam-4.12.0 fails to compile with jpeg-9a

Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

            Bug ID: 350915
           Summary: digikam-4.12.0 fails to compile with jpeg-9a
           Product: digikam
           Version: 4.11.0
          Platform: Exherbo Packages
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
          Assignee: [hidden email]
          Reporter: [hidden email]

Despite having a codepath for jpeg9 located at core/libs/jpegutils/libjpeg-91
digikam fails with the following error when trying to build with jpeg-9a:

[...]
Found JPEG: /usr/lib/libjpeg.so
[...]

/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:
In member function 'virtual bool Digikam::JPEGLoader::load(const QString&,
Digikam::DImgLoaderObserver*)':
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:253:34:
error: cannot convert 'bool' to 'boolean' for argument '2' to 'int
jpeg_read_header(j_decompress_ptr, boolean)'
     jpeg_read_header(&cinfo, true);
                                  ^
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:303:35:
error: cannot convert 'bool' to 'boolean' in assignment
         cinfo.do_fancy_upsampling = false;
                                   ^
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:304:35:
error: cannot convert 'bool' to 'boolean' in assignment
         cinfo.do_block_smoothing  = false;
                                   ^
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:
In member function 'virtual bool Digikam::JPEGLoader::save(const QString&,
Digikam::DImgLoaderObserver*)':
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:787:43:
error: cannot convert 'bool' to 'boolean' for argument '3' to 'void
jpeg_set_quality(j_compress_ptr, int, boolean)'
     jpeg_set_quality(&cinfo, quality, true);
                                           ^
/var/tmp/paludis/build/graphics-digikam-4.11.0/work/digikam-4.11.0/core/libs/dimg/loaders/jpegloader.cpp:788:37:
error: cannot convert 'bool' to 'boolean' for argument '2' to 'void
jpeg_start_compress(j_compress_ptr, boolean)'
     jpeg_start_compress(&cinfo, true);
                                     ^

Attached patch makes digikam-4.11.0 compile fine and a basic test adding jpegs
to a new album and displaying them worked.

Note: For getting kipi-plugins to compile with jpeg-9a I had to apply a second
patch: https://git.reviewboard.kde.org/r/123886/

Reproducible: Always

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a

Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

Timo Gurr <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|digikam-4.12.0 fails to     |digikam-4.11.0 fails to
                   |compile with jpeg-9a        |compile with jpeg-9a

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a

Timo Gurr
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

--- Comment #1 from Timo Gurr <[hidden email]> ---
Created attachment 93862
  --> https://bugs.kde.org/attachment.cgi?id=93862&action=edit
digikam-4.11.0-jpeg9.patch

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a [patch]

Gilles Caulier-4
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]
            Summary|digikam-4.11.0 fails to     |digikam-4.11.0 fails to
                   |compile with jpeg-9a        |compile with jpeg-9a
                   |                            |[patch]

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a [patch]

Gilles Caulier-4
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|general                     |Portability

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a [patch]

Gilles Caulier-4
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/b2bd2dc6440d70f4a79c0c8
                   |                            |a1ed61350e00c518b
         Resolution|---                         |FIXED
             Status|UNCONFIRMED                 |RESOLVED

--- Comment #2 from Gilles Caulier <[hidden email]> ---
Git commit b2bd2dc6440d70f4a79c0c8a1ed61350e00c518b by Gilles Caulier.
Committed on 06/08/2015 at 08:21.
Pushed by cgilles into branch 'master'.

apply patch #93862 from Timo Gurr to fix compilation with jpeg 9a
FIXED-IN : 4.13.0

M  +2    -1    NEWS
M  +5    -5    libs/dimg/loaders/jpegloader.cpp

http://commits.kde.org/digikam/b2bd2dc6440d70f4a79c0c8a1ed61350e00c518b

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a [patch]

Gilles Caulier-4
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

--- Comment #3 from Gilles Caulier <[hidden email]> ---
Git commit 10ddbc7afa500499e5d6d6449e488fc611b9c7d3 by Gilles Caulier.
Committed on 06/08/2015 at 08:25.
Pushed by cgilles into branch 'frameworks'.

backport commit #b2bd2dc6440d70f4a79c0c8a1ed61350e00c518b from git/master to
frameworks branch

M  +5    -5    libs/dimg/loaders/jpegloader.cpp

http://commits.kde.org/digikam/10ddbc7afa500499e5d6d6449e488fc611b9c7d3

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel
Reply | Threaded
Open this post in threaded view
|

[digikam] [Bug 350915] digikam-4.11.0 fails to compile with jpeg-9a [patch]

bugzilla_noreply
In reply to this post by Timo Gurr
https://bugs.kde.org/show_bug.cgi?id=350915

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|Portability                 |Bundle-Tarball

--
You are receiving this mail because:
You are the assignee for the bug.