Skip to content

NetBSD support#36

Open
manu0401 wants to merge 2 commits intoarthurdejong:masterfrom
manu0401:netbsd
Open

NetBSD support#36
manu0401 wants to merge 2 commits intoarthurdejong:masterfrom
manu0401:netbsd

Conversation

@manu0401
Copy link

No description provided.

manu0401 added 2 commits July 18, 2019 18:23
File missing in original commit
@manu0401
Copy link
Author

The CI failed because it was denied install of /lib/libnss_ldap.so.2 but it is not obvious to me that this is related to my change. Is the case?

@@ -0,0 +1,542 @@
/*
* This file is derived from NetBSD stub for PADL nss_ldap
* http://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/databases/nss_ldap/files/netbsd.c
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you provide some more information on the copyright holders and license of the original file?

@arthurdejong
Copy link
Owner

Thanks for providing the NetBSD support!

The CI failed because it was denied install of /lib/libnss_ldap.so.2 but it is not obvious to me that this is related to my change. Is the case?

This is related to the --with-nss-libdir change. Previously it would use the value of --libdir which is /usr/lib by default. Can NetBSD NSS modules only be installed in /lib?

On FreeBSD configure is commonly called with:

./configure --enable-warnings --enable-pynslcd --enable-utils \
  --with-pam-seclib-dir=/usr/local/lib --prefix=/usr/local \
  --mandir=/usr/local/man --infodir=/usr/local/info/ \
  CC=cc CFLAGS='-O2 -pipe -fno-strict-aliasing' \
  LDFLAGS='-L/usr/local/lib' \
  CPP=cpp CPPFLAGS=-I/usr/local/include

So you could also pass the appropriate flags to the configure script when installing on NetBSD.

I don't have a NetBSD VM at the moment to test this on but I'll try to set one up.

@manu0401
Copy link
Author

manu0401 commented Sep 29, 2019 via email

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants