Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a way to query for plugin directory #19

Closed
pacho2 opened this issue Sep 9, 2014 · 2 comments
Closed

Add a way to query for plugin directory #19

pacho2 opened this issue Sep 9, 2014 · 2 comments

Comments

@pacho2
Copy link

pacho2 commented Sep 9, 2014

As discussed at:
https://bugs.gentoo.org/show_bug.cgi?id=519986

NetworkManager needs to get ppp plugin dir configured at build time with --with-pppd-plugin-dir=$(ppp_version). I have checked various distributions (debian, opensuse, fedora, arch...) and every one has a different workaround for this (some specify only a version, other checks for plugin dir running "ls" to know the one existing at that time (!)...)

Maybe would be interesting to provide a way to check for plugin dir used by built ppp that could allow other software to rely on it for building in proper place

Thanks a lot

@vincele
Copy link

vincele commented Jan 6, 2021

maybe pkg-config as in #47 could be used for that purpose.

enaess added a commit to enaess/ppp that referenced this issue Mar 23, 2021
This adds pppd.pc into $(INSTROOT)/$(LIBDIR)/pkgconfig. On some distributions
this would be /lib/pkgconfig, or /usr/lib/pkgconfig, but other distributions
may consider specifying --libdir=/usr/lib/x86_x64-linux-gnu/ and the pkgconfig
directory would be under that.

Manually cherry picking parts of two commits by @lkundrak from:
    https://github.com/NetworkManager/ppp/tree/lr/pkgconfig

Mainly, the difference between the original commit is not to replace DESTDIR
with "prefix". Leave DESTDIR alone, and add needed pkgconfig (pppd.pc.in)
as a part of the linux distribution (previously in pppd/plugin directory).

Signed-off-by: Eivind Naess <eivnaes@yahoo.com>
enaess added a commit to enaess/ppp that referenced this issue Mar 23, 2021
This adds pppd.pc into $(INSTROOT)/$(LIBDIR)/pkgconfig. On some distributions
this would be /lib/pkgconfig, or /usr/lib/pkgconfig, but other distributions
may consider specifying --libdir=/usr/lib/x86_x64-linux-gnu/ and the pkgconfig
directory would be under that.

Manually cherry picking parts of two commits by @lkundrak from:
    https://github.com/NetworkManager/ppp/tree/lr/pkgconfig

Mainly, the difference between the original commit is not to replace DESTDIR
with "prefix". Leave DESTDIR alone, and add needed pkgconfig (pppd.pc.in)
as a part of the linux distribution (previously in pppd/plugin directory).

Signed-off-by: Eivind Naess <eivnaes@yahoo.com>
enaess added a commit to enaess/ppp that referenced this issue Apr 15, 2021
This adds pppd.pc into $(INSTROOT)/$(LIBDIR)/pkgconfig. On some distributions
this would be /lib/pkgconfig, or /usr/lib/pkgconfig, but other distributions
may consider specifying --libdir=/usr/lib/x86_x64-linux-gnu/ and the pkgconfig
directory would be under that. Allowing --libdir to be specified at configure
time fixes ppp-project#223, providing pkgconfig support fixes ppp-project#19 and allows third party
packages pickup the plugin directory.

Manually cherry picking parts of two commits by @lkundrak from:
    https://github.com/NetworkManager/ppp/tree/lr/pkgconfig

Mainly, the difference between the original commit is not to replace DESTDIR
with "prefix". Leave DESTDIR alone, and add needed pkgconfig (pppd.pc.in)
as a part of the linux distribution (previously in pppd/plugin directory).

This fixes GitHub issues: ppp-project#19 ppp-project#47 and ppp-project#223

Signed-off-by: Eivind Naess <eivnaes@yahoo.com>
@Neustradamus
Copy link
Member

PR: #270

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

3 participants