Skip to content

Conversation

JackBoosY
Copy link
Contributor

@JackBoosY JackBoosY commented Aug 24, 2020

Fixes #12501

See ***/blob/master/docs/maintainers/maintainer-guide.md#manifest for solution.

@JackBoosY JackBoosY added info:internal depends:vm-update PR contains changes to the VM provisioning scripts labels Aug 24, 2020
@JackBoosY
Copy link
Contributor Author

See github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#manifest for solution.

At least users can see this link now.

@JackBoosY JackBoosY marked this pull request as ready for review August 24, 2020 08:52
@BillyONeal BillyONeal removed the depends:vm-update PR contains changes to the VM provisioning scripts label Aug 24, 2020
@BillyONeal
Copy link
Member

Dropping depends:vm-update since there are no VM changes here.

I asked around for some Azure Pipelines folks to see if there's a way we can fix this without de-linkifying the results.

@BillyONeal
Copy link
Member

Filed: https://developercommunity.visualstudio.com/content/problem/1160967/azure-devops-is-stripping-uris-from-build-log-outp.html

I'd like to give the ADO team a few days to try to resolve this before landing this workaround, but the workaround looks correct to me.

@BillyONeal
Copy link
Member

They didn't have an immediate solution, so merging this for now. Thanks for your help!

@BillyONeal BillyONeal merged commit 582e30d into microsoft:master Aug 25, 2020
@JackBoosY JackBoosY deleted the dev/jack/12501 branch August 26, 2020 02:10
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.

[vcpkg|manifest] Improve error messages
3 participants