Skip to content

winbuild: Free name $(CC) Makefile#950

Merged
bagder merged 1 commit into
curl:masterfrom
webmaster128:free-name-cc
Aug 9, 2016
Merged

winbuild: Free name $(CC) Makefile#950
bagder merged 1 commit into
curl:masterfrom
webmaster128:free-name-cc

Conversation

@webmaster128
Copy link
Copy Markdown
Contributor

@webmaster128 webmaster128 commented Aug 9, 2016

In the old line number 290, CC and CURL_CC had the same value. After
that, /DCURL_STATICLIB was added to CC but not CURL_CC (intended?).

This gets rid of the CC variable entirely. It is a first step to make it
possible to manually set a CC variable in order to be able to change the
compiler.


Before, tool_hugehelp.c, nonblock.c, rawstr.c, strtoofft.c, warnless.c have been compiled without /DCURL_STATICLIB.

In the old line number 290, CC and CURL_CC had the same value. After
that, /DCURL_STATICLIB was added to CC but not CURL_CC (intended?).

This gets rid of the CC variable entirely. It is a first step to make it
possible to manualyl set a CC variable in order to be able to change the
compiler.
@mention-bot
Copy link
Copy Markdown

@webmaster128, thanks for your PR! By analyzing the annotation information on this pull request, we identified @yangtse, @captain-caveman2k and @mback2k to be potential reviewers

@bagder bagder merged commit b2ac016 into curl:master Aug 9, 2016
@bagder
Copy link
Copy Markdown
Member

bagder commented Aug 9, 2016

thanks!

@lock lock Bot locked as resolved and limited conversation to collaborators Jan 19, 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.

3 participants