Review Request: Fix kde4workspace-processui linker error on gentoo linux

Andreas Pakulat apaku at gmx.de
Fri Jul 22 11:08:28 UTC 2011



> On July 22, 2011, 9:29 a.m., Andreas Pakulat wrote:
> > This is wrong in all ways possible. Gentoo should just stop trying to break up packages which where never meant to be broken into pieces.
> > 
> > Anyway, at least a find_library call is necessary for this so CMake knows the absolute path to the processui library.
> 
> Andreas Pakulat wrote:
>     An even better fix would be to make the usage of the process-ui library optional and use the old simple process widget instead when its missing.
> 
> Sven Brauch wrote:
>     I'm aware of the fact that this isn't a clean fix, but I think the problem should be adressed somehow, so I suggested this.
>     
>     Okay, so what would you suggest instead? Compiling the tests could just be skipped if that library isn't present, if you think that's a better solution.

If you want to address the problem, then get the Gentoo maintainers to fix the config-module that is installed for kde-workspace. Thats a tough job though, been there and failed ;)

Adding a workaround to kdevelop is fine, but then at least find_library needs to be used to find the library if the config module is broken and does not provide the variables, so cmake knows the actual library and can generate proper dependencies and linking does not fail if for some reason the processui library is not in /usr/lib.

The library is not only used for the tests but also for the gdb plugin itself, so making it optional means providing a suitable replacement. Before the dependency on processui was introduced we had our own widget to show the list of processes and select one to attach to, that should still be findable in the history and hence could be resurrected.


- Andreas


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/102045/#review4961
-----------------------------------------------------------


On July 22, 2011, 8:34 a.m., Sven Brauch wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/102045/
> -----------------------------------------------------------
> 
> (Updated July 22, 2011, 8:34 a.m.)
> 
> 
> Review request for KDevelop.
> 
> 
> Summary
> -------
> 
> This fixes a linker error when compiling kdevelop from git on gentoo linux, described here, for example: http://forums.gentoo.org/viewtopic-t-831304-start-0.html
> 
> 
> Diffs
> -----
> 
>   debuggers/gdb/CMakeLists.txt 55aa261 
> 
> Diff: http://git.reviewboard.kde.org/r/102045/diff
> 
> 
> Testing
> -------
> 
> Well, it compiles.
> 
> 
> Thanks,
> 
> Sven
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20110722/e64a0f8e/attachment.html>


More information about the KDevelop-devel mailing list