[KDE/Mac] Review Request 122394: Fix OSX library names in kdeinit5.app

[René J.V. Bertin]

> On Feb. 2, 2015, 10:56 p.m., René J.V. Bertin wrote:
> > src/kdeinit/kinit.cpp, lines 90-92
> > <https://git.reviewboard.kde.org/r/122394/diff/1/?file=346478#file346478line90>
> >
> >     These are true shared libraries that are also used for "-l" style linking with ld?
> 
> Jeremy Whiting wrote:
>     I'm not sure I understand the question, is there some other type of library on OSX besides "true shared libraries that are also used for -l style linking with ld"? file says they are Mach-O 64-bit dynamically linked shared library x86_64 if that helps answer your question.

What I mean is if there are applications (or other libraries) that link in those libraries using `-lKF5KIOCore`, `-lKF5Parts` or `-lKF5Plasma`. In that case, the .dylib extension is obligatory. In all other cases, the extension can in fact be anything. Thus, plugins and modules usually have the .so extension on OS X, just like on Linux.

Your modification is correct in itself (evidently, if you not longer get complaints about libraries not being found). But if those libraries are only ever loaded dynamically you could also modify the CMake file so that they are created with a .so extension (and leave the C++ code alone).


- René J.V.


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122394/#review75242
-----------------------------------------------------------


On Feb. 2, 2015, 9:51 p.m., Jeremy Whiting wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122394/
> -----------------------------------------------------------
> 
> (Updated Feb. 2, 2015, 9:51 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X, kdelibs, David Faure, and Ian Wadham.
> 
> 
> Bugs: 343707
>     https://bugs.kde.org/show_bug.cgi?id=343707
> 
> 
> Repository: kinit
> 
> 
> Description
> -------
> 
> OSX Doesn't have .so libraries, so use OSX names in kdeinit5.app to load the correct libraries needed.
> 
> 
> Diffs
> -----
> 
>   src/kdeinit/kinit.cpp 3c3c913 
> 
> Diff: https://git.reviewboard.kde.org/r/122394/diff/
> 
> 
> Testing
> -------
> 
> kdeinit5.app no longer complains about the missing .so libraries.
> 
> 
> Thanks,
> 
> Jeremy Whiting
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-mac/attachments/20150202/d1ffa5d1/attachment.html>