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

gherkin-c does not build with MSVC #1198

Closed
DewJunkie opened this issue Sep 29, 2020 · 3 comments
Closed

gherkin-c does not build with MSVC #1198

DewJunkie opened this issue Sep 29, 2020 · 3 comments
Labels
⌛ stale Will soon be closed by stalebot unless there is activity language: c library: gherkin 🔧 build Related to build / release process

Comments

@DewJunkie
Copy link
Contributor

DewJunkie commented Sep 29, 2020

Acceptance Criteria:

  • gherkin-c can build with msvc

Summary

MSVC includes the library m.lib in the default CRT. Explicitly linking causes the build to fail.

Expected Behavior

gerkin-c builds successfully by following the build instructions in readme.md

Current Behavior

C:\dev\extern\cucumber\gherkin\c\out\build\x64-Debug (default)\LINK : fatal error LNK1104: cannot open file 'm.lib'

Possible Solution

Add check for existance of m.lib and if exists, then link it in

include(CheckLibraryExists)
CHECK_LIBRARY_EXISTS(m log10 "" HAVE_LIB_M)   
if (HAVE_LIB_M)  
    target_link_libraries(gherkin PRIVATE m)
endif (HAVE_LIB_M)

Context & Motivation

Support all the platforms

Your Environment

MSVC 16.8.442.31521

@DewJunkie
Copy link
Contributor Author

Added PR 1199, maybe proper procedure is just to create a PR?

@aslakhellesoy aslakhellesoy added language: c library: gherkin 🔧 build Related to build / release process labels Sep 29, 2020
@stale
Copy link

stale bot commented Dec 15, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs.

@stale stale bot added the ⌛ stale Will soon be closed by stalebot unless there is activity label Dec 15, 2020
@stale
Copy link

stale bot commented Dec 25, 2020

This issue has been automatically closed because of inactivity. You can support the Cucumber core team on opencollective.

@stale stale bot closed this as completed Dec 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⌛ stale Will soon be closed by stalebot unless there is activity language: c library: gherkin 🔧 build Related to build / release process
Projects
None yet
Development

No branches or pull requests

2 participants