Skip to content

CMake // Fixed issue with static build for MSVC2010#74

Closed
snikulov wants to merge 1 commit intocurl:masterfrom
snikulov:cmake_vc2010_static_update
Closed

CMake // Fixed issue with static build for MSVC2010#74
snikulov wants to merge 1 commit intocurl:masterfrom
snikulov:cmake_vc2010_static_update

Conversation

@snikulov
Copy link
Contributor

@snikulov snikulov commented Jul 9, 2013

Fixed issue with static build for MSVC2010.
The issue was raised in following thread http://curl.haxx.se/mail/lib-2013-07/0046.html

After some investigation I've discovered known issue http://public.kitware.com/Bug/view.php?id=11240
When .rc file is linked to static lib it fails with following linker error

LINK : warning LNK4068: /MACHINE not specified; defaulting to X86
file.obj : fatal error LNK1112: module machine type 'x64' conflicts with target machine type 'X86'

Fix add target property /MACHINE: for MSVC generation.

Also removed old workarounds - it caused errors during msvc build.

@bagder
Copy link
Member

bagder commented Jul 11, 2013

On Tue, 9 Jul 2013, Sergei Nikulov wrote:

Please send patches to the curl-library mailing list so that all interested
hackers can read and comment them.

/ daniel.haxx.se

@snikulov
Copy link
Contributor Author

I already done this http://curl.haxx.se/mail/lib-2013-07/0070.html
Should I re-send it again?

@bagder
Copy link
Member

bagder commented Jul 16, 2013

Thanks, merged and pushed with an edited commit message with details from here!

@bagder bagder closed this Jul 16, 2013
@lock lock bot locked as resolved and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants