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

windows: only look for ancient compilers #146

Merged
merged 1 commit into from
Sep 11, 2017

Conversation

daviddrysdale
Copy link
Member

Also drop the use of a versioned output directory; just use
.\msvc

Also drop the use of a versioned output directory; just use
.\msvc
Copy link
Member

@bradh352 bradh352 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, needing to list every version of MSVC always seemed like an odd choice.

Of course, I'd also approve a request to completely kill the MSVC build system and standardize on CMake for Windows ... but that's just me, I doubt there'd be wide support for that.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.08%) to 95.46% when pulling 725be06 on daviddrysdale:oldwin into cb68526 on c-ares:master.

Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, but I don't build on Windows and I have thus not tested this...

@daviddrysdale daviddrysdale merged commit 7e1a8fc into c-ares:master Sep 11, 2017
@daviddrysdale daviddrysdale deleted the oldwin branch September 11, 2017 18:39
dimbleby pushed a commit to dimbleby/c-ares that referenced this pull request Sep 11, 2017
Also drop the use of a versioned output directory; just use
.\msvc
DronRathore pushed a commit to DronRathore/c-ares that referenced this pull request Mar 11, 2020
Also drop the use of a versioned output directory; just use
.\msvc
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.

4 participants