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

Removed unused template function. #9287

Merged
merged 1 commit into from
Dec 21, 2019

Conversation

ortymd
Copy link
Contributor

@ortymd ortymd commented Dec 11, 2019

Hi all,

I'm using vcpkg repo to get myself familiar with MSCS 2019 (after moving from Linux dev environment to Windos). Just by chance bumped into this function an got very interested where/how it is used, but unfortunately I failed to find where it is used in the project. Thus I removed it, recompiled and thought it is worth removing it.

@dan-shaw
Copy link
Contributor

If it builds for all platforms, it LGTM.

@dan-shaw dan-shaw added info:reviewed Pull Request changes follow basic guidelines and removed needs-further-review labels Dec 14, 2019
@ortymd
Copy link
Contributor Author

ortymd commented Dec 15, 2019

It's my first time working with Azure Piplelines.
Can someone please comment on vcpkg-windows-PR-test failure.
I failed to find anything relevant in the logs...

I checked build and runtime manually on Win 10 a Ubunu19. Found no issues.
Unfortunately I have no access to MacOS, so have nothing to say here so far.

@JackBoosY
Copy link
Contributor

/azp run

@ortymd
Copy link
Contributor Author

ortymd commented Dec 16, 2019

I see that the pipeline was restated for the pr and now windows job is ok. osx job seems not to be running for any of the active PRs...

@JackBoosY
Copy link
Contributor

@ortymd We are fixing OSX CI pipeline, see #9216.

@ras0219-msft ras0219-msft merged commit 37265f7 into microsoft:master Dec 21, 2019
@ras0219-msft
Copy link
Contributor

Thanks for the PR!

strega-nil pushed a commit to strega-nil/vcpkg that referenced this pull request May 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants