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 NMake Makefiles for Visual Studio builds #99

Merged
merged 1 commit into from Apr 26, 2018

Conversation

fanc999
Copy link
Collaborator

@fanc999 fanc999 commented Apr 26, 2018

Hi,

This adds a set of NMake Makefiles and associated items for building libpsl using Visual Studio's NMake. This also pulls out the source listings of libpsl into a separate Makefile snippet so that it can be included by both the autotools Makefile.am and the NMake Makefiles for the build.

This attempts to make things modularized and this reuses items that have been used in the past for building items such as HarfBuzz (which I maintained the NMake Makefiles for it before it switched to CMake), due to the differences between GNUish Make and NMake. Due to the complexity of building libidn/libidn2 with libunistring using Visual Studio, currently only builds using libicu as the runtime/builtin, or builds without runtime/builtin usage are supported with this set of NMake Makefiles.

With blessings, thank you!

This adds a set of Visual Studio NMake Makefiles that can be used to
build libpsl, either as a DLL or as a static .lib.  Building of the psl
tool and the tests in tests/ (but not fuzz, since those tests use
fmemopen(), that is not supported on Windows) are also supported.

A simplified "install" target is provided that copies the build
results to a default or set prefix with the pkg-config file so that
other packages (such as libsoup) with build systems that depend on this
.pc file can be built.

These files make use of autotools stuff so that they have the up-to-date
version info upon a release.

Currently, only builds using the ICU runtime/builtin or with no
runtime/builtin are supported with Visual Studio builds.
@coveralls
Copy link

Coverage Status

Coverage remained the same at 93.648% when pulling 6a5da68 on fanc999:build.win into 724fbd0 on rockdaboot:master.

@rockdaboot rockdaboot merged commit c93175c into rockdaboot:master Apr 26, 2018
@rockdaboot
Copy link
Owner

Great, thank you !

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.

None yet

4 participants