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

Update windows VC++ nmake makefiles #732

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

viruscamp
Copy link

@viruscamp viruscamp commented Jun 14, 2022

Update windows VC++ nmake makefiles, add saslPASSDSS. Now there are 14 plugins in 'plugins/NTMakefile'.
Update plugins/makeinit.sh to generate all 16 '*_init.c' files at once, using sh makeinit.sh.
Update plugins/passdss.c add #include <stdint.h> to make it compiled in vc-win32.

…4 plugins in 'plugins/NTMakefile'.

Update plugins/makeinit.sh to generate all 16 '*_init.c' files at once, using `sh makeinit.sh`.
Update plugins/passdss.c add `#include <stdint.h>` to make it compiled in vc-win32.

Signed-off-by: viruscamp <[email protected]>
@viruscamp
Copy link
Author

Tested with vs2019-x64, db-5.3.28, sqlite-3380500 and openssl-1.1.1o.
Tested with vs2019-x86, db-5.3.28, sqlite-3380500 and openssl-1.0.2u.

There are 14 plugins in 'plugins/NTMakefile', 12 of them were built success.
LDAP and GSSAPI have not been tested.
Plugins of kerberos4.c and gs2.c are not in 'plugins/NTMakefile'.

@quanah
Copy link
Contributor

quanah commented Jun 22, 2022

Some plugins are going to be deleted, such as kerberos4. May want to check the open tickets on that as well.

@Neustradamus
Copy link
Contributor

@Neustradamus
Copy link
Contributor

@hyc: Have you seen this PR?

@quanah
Copy link
Contributor

quanah commented Jul 23, 2024

May be better to use CMAKE if that support is added, see #842

@Neustradamus
Copy link
Contributor

@quanah: It is possible to have several possibilities...

@hyc
Copy link
Contributor

hyc commented Jul 23, 2024

@quanah: It is possible to have several possibilities...

Irrelevant. We don't have the time or resources to support redundant possibilities.

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.

4 participants