Skip to content
New issue

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

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

Already on GitHub? # to your account

Win32 fixes #500

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

Conversation

jon-turney
Copy link
Contributor

fb69d751 Fix Windows build with both ENABLED_SHARED and ENABLE_STATIC

Fix #375 when both shared and static libraries are built.

4b453061 Fix casing of "BaseTsd.h" include

Fix #436 for the case where cross-compiling to Windows.

Microsoft isn't consistent about the casing of header filenames in the
Windows SDK and documentation (e.g. "Windows.h" vs. "windows.h"), but
this doesn't matter on a case-insensitive filesystem.

If you are cross-compiling for Windows (e.g. using gcc), then the case
matters, and is practically controlled by the w32api headers provided by
MinGW64 project.
Define SOLV_STATIC_LIB for libsolv_static to avoid an error when
building it for Windows:

/work/libsolv/win32/getopt.c:11:14: error: variable ‘optind’ definition is marked dllimport
/work/libsolv/win32/getopt.c:11:24: error: variable ‘opterr’ definition is marked dllimport
/work/libsolv/win32/getopt.c:11:52: error: variable ‘__optreset’ definition is marked dllimport
v2:
Also run tests on Windows
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant