Project

General

Profile

Actions

Bug #350

open

pkg-config file libraries results in overlinking

Added by Matthew Krupcale over 4 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Start date:
08/13/2020
Due date:
% Done:

0%

Estimated time:

Description

The scales-ppm{,-core}.pc.in pkg-config files specifies several libraries in the Libs: portion which are actually internal, private library dependencies of scales-ppm. That is, they are not part of the public interface/API and are just implementation details. These should be specified instead in Libs.private: so that they are only added to the link when creating a static library.

Similarly, the MPI includes are part of the public scales-ppm-core API, so we need to include those, but we don't need parmetis or metis includes here (in any case, they should have been in scales-ppm.pc.in, not scales-ppm-core.pc.in, but they're not needed there either since they're again not part of the public API).

See attached patch and [1-2] for how libraries should be specified for linking.

[1] https://people.freedesktop.org/~dbn/pkg-config-guide.html
[2] https://cmake.org/pipermail/cmake/2016-May/063400.html


Files

scales-ppm-pkg-config-private-libraries.patch (1.69 KB) scales-ppm-pkg-config-private-libraries.patch Fix pkg-config file private libraries and includes Matthew Krupcale, 08/13/2020 08:23 PM
Actions #1

Updated by Thomas Jahns almost 3 years ago

Sorry for taking so long. This should be addressed with change commit:2167449bbd8637b698c19297c47db6d7a166fad9, part of the 1.0.8 release.

Actions #2

Updated by Thomas Jahns almost 3 years ago

  • Status changed from New to Resolved
Actions

Also available in: Atom PDF