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

[feature] Included ITM functionality for building with MSVC #1080

Merged
merged 2 commits into from Dec 20, 2020

Conversation

slyshykO
Copy link
Collaborator

No description provided.

@slyshykO slyshykO marked this pull request as draft December 19, 2020 12:45
@slyshykO slyshykO marked this pull request as ready for review December 19, 2020 13:55
@simplerobot
Copy link
Collaborator

Thanks for fixing this. The fix looks fine to me.

I notice the windows compilation is commented out in the .travis.yml file. Should we create an issue to add windows support to the pipeline checks?

@Nightwalker-87
Copy link
Member

@simplerobot: I would have added it already, if it was working properly.

@Nightwalker-87 Nightwalker-87 changed the title fix msvc build Included ITM functionality for building with MSVC Dec 20, 2020
@Nightwalker-87 Nightwalker-87 added this to In progress in Release v1.7.0 via automation Dec 20, 2020
@Nightwalker-87 Nightwalker-87 added this to the v1.6.2 milestone Dec 20, 2020
@Nightwalker-87 Nightwalker-87 changed the title Included ITM functionality for building with MSVC [feature] Included ITM functionality for building with MSVC Dec 20, 2020
@Nightwalker-87 Nightwalker-87 moved this from In progress to Reviewer approved in Release v1.7.0 Dec 20, 2020
@Nightwalker-87 Nightwalker-87 merged commit 4918223 into stlink-org:develop Dec 20, 2020
Release v1.7.0 automation moved this from Reviewer approved to Done Dec 20, 2020
@slyshykO slyshykO deleted the fix-msvc-build branch December 20, 2020 13:08
@stlink-org stlink-org locked as resolved and limited conversation to collaborators Mar 20, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

3 participants