Bug 288679

Summary: vkontakte does not compile
Product: [Applications] digikam Reporter: Jonathan Riddell <jr>
Component: Plugin-WebService-VkontakteAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED UNMAINTAINED    
Severity: normal CC: caulier.gilles, cfeck, rakuco
Priority: NOR    
Version: 1.0.0   
Target Milestone: ---   
Platform: Ubuntu   
OS: Linux   
Latest Commit: Version Fixed In: 8.0.0
Attachments: gcc error

Description Jonathan Riddell 2011-12-10 19:53:19 UTC
Created attachment 66602 [details]
gcc error

Version:           2.4.0 (using KDE 4.7.3) 
OS:                Linux

libkvkontakte does not compile against qjson with gcc 4:4.6.1-2ubuntu5 from Ubuntu development version


Reproducible: Didn't try

Steps to Reproduce:
compile extras/libkvkontakte


Expected Results:  
it would compile
Comment 1 Jonathan Riddell 2011-12-10 22:27:56 UTC
It's more likely due to GNU ld (GNU Binutils for Ubuntu) 2.22 which is new in ubuntu development versions (12.04).  
libkvkontakte does link successfully in 11.10 which uses ld 2.21.53.20110810
Comment 2 Jonathan Riddell 2011-12-10 22:36:20 UTC
The other reason might be that libQJson has been changed to multiarch in ubuntu 12.04 so it installed in /usr/lib/i386-linux-gnu/libqjson.so (etc) rather than /usr/lib/libqjson.so
Comment 3 Jonathan Riddell 2011-12-10 22:52:33 UTC
It successfully links if libqjson is installed to /usr/lib and not /usr/lib/i386-linux-gnu/
Comment 4 Jonathan Riddell 2011-12-10 23:28:01 UTC
If i add   ${qjson_LIBRARIES} into extra/libkvkontakte/libkvkontakte/CMakeLists.txt it works fine, for some reason ${QJSON_LIBRARIES} is blank there
Comment 5 Alexander Potashev 2011-12-11 07:15:17 UTC
Git commit 42b3652322b63fa6e11b426e40351f7138f51292 by Alexander Potashev.
Committed on 11/12/2011 at 08:13.
Pushed by aspotashev into branch 'master'.

Also accept QJson library path passed in ${qjson_LIBRARIES}

BUG: 288679

M  +1    -0    libkvkontakte/CMakeLists.txt

http://commits.kde.org/libkvkontakte/42b3652322b63fa6e11b426e40351f7138f51292
Comment 6 Raphael Kubo da Costa 2011-12-12 01:05:40 UTC
FWIW, this should not be needed anymore. The "qjson" prefix existed only in qjson's git master for a few months, and it was reverted back to "QJSON" a few days ago.