[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2023-05-14 Thread Christoph Cullmann
https://bugs.kde.org/show_bug.cgi?id=442555

Christoph Cullmann  changed:

   What|Removed |Added

 Status|REPORTED|RESOLVED
  Latest Commit||https://invent.kde.org/fram
   ||eworks/extra-cmake-modules/
   ||commit/028d2afde1b5ec765c20
   ||6bf3db6574c85d200e5e
 Resolution|--- |FIXED

--- Comment #7 from Christoph Cullmann  ---
Git commit 028d2afde1b5ec765c206bf3db6574c85d200e5e by Christoph Cullmann, on
behalf of Dawid Wróbel.
Committed on 14/05/2023 at 19:59.
Pushed by cullmann into branch 'master'.

FindCanberra: link against target provided by pkgconfig

target_link_libraries() against the target generated by
pkg_check_modules() so that the additional linker options are not
lost, such as the additional libraries to link against when
building statically.

M  +4-1find-modules/FindCanberra.cmake

https://invent.kde.org/frameworks/extra-cmake-modules/commit/028d2afde1b5ec765c206bf3db6574c85d200e5e

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-11-22 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=442555

--- Comment #6 from Christophe Giboudeaux  ---
Static libraries are not supposed to have external link dependencies. If
linking fails if ltdl isn't also linked, that's a strong indication that it's a
public link target.

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-11-19 Thread Dawid Wróbel
https://bugs.kde.org/show_bug.cgi?id=442555

--- Comment #5 from Dawid Wróbel  ---
(In reply to Christophe Giboudeaux from comment #4)
> If it's a static lib, why would it have to link to anything else? sounds
> like something is broken is the canberra build system

Linking happens at build time of the static app, not of the library itself.
That information must be conveyed to the linker so it knows which libraries are
needed, including the indirect dependencies of the direct ones. Please refer to
https://people.freedesktop.org/~dbn/pkg-config-guide.html on the
Requires.private, Libs.private and Requires.private explanation.

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-11-19 Thread Christophe Giboudeaux
https://bugs.kde.org/show_bug.cgi?id=442555

--- Comment #4 from Christophe Giboudeaux  ---
If it's a static lib, why would it have to link to anything else? sounds like
something is broken is the canberra build system

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-11-19 Thread Dawid Wróbel
https://bugs.kde.org/show_bug.cgi?id=442555

--- Comment #3 from Dawid Wróbel  ---
(In reply to Nicolas Fella from comment #2)
> Could you show what the resulting linker invocation looks like and what you
> expect it to be?

I don't have it handy right now, but before making the proposed change, the
-lltdl was missing from the linker invocation, and is correctly applied after
the change.

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-09-16 Thread Nicolas Fella
https://bugs.kde.org/show_bug.cgi?id=442555

Nicolas Fella  changed:

   What|Removed |Added

 CC||nicolas.fe...@gmx.de

--- Comment #2 from Nicolas Fella  ---
Could you show what the resulting linker invocation looks like and what you
expect it to be?

-- 
You are receiving this mail because:
You are watching all bug changes.

[extra-cmake-modules] [Bug 442555] FindCanberra ignores any extra libs if provided by the libcanberra.pc

2021-09-16 Thread Dawid Wróbel
https://bugs.kde.org/show_bug.cgi?id=442555

--- Comment #1 from Dawid Wróbel  ---
https://invent.kde.org/frameworks/extra-cmake-modules/-/merge_requests/187

-- 
You are receiving this mail because:
You are watching all bug changes.