Skip to content

Conversation

kafeg
Copy link
Contributor

@kafeg kafeg commented Apr 25, 2020

Describe the pull request

  • Added support for build libvpx on Mac and Linux

  • x64-linux, x64-osx

@kafeg
Copy link
Contributor Author

kafeg commented Apr 25, 2020

Hello! I got error "There are no build logs for libvpx build. bla bla bla" for the "vcpkg-osx-PR-test"

But everything was built fine.

Can you please help me with understanding and fixing the problem?

@LilyWangL LilyWangL self-requested a review April 26, 2020 02:02
@LilyWangL LilyWangL self-assigned this Apr 26, 2020
@LilyWangL
Copy link
Contributor

Thanks for your PR. Could you please remove the following codes in scripts/ci.baseline.txt, it makes Linux and MACOS build failed.

libvpx:x64-linux=fail
libvpx:x64-osx=fail

@kafeg
Copy link
Contributor Author

kafeg commented Apr 26, 2020

Done! Thank you for your prompt!

@LilyWangL LilyWangL added info:reviewed Pull Request changes follow basic guidelines and removed waiting for response labels Apr 28, 2020
@strega-nil
Copy link
Contributor

@kafeg would you mind resolving the merge conflict? otherwise, this looks great, and I'd love to merge it :)

@kafeg
Copy link
Contributor Author

kafeg commented May 1, 2020

Hey @strega-nil ! Thank you for remind ;)
Conflicts fixed, locally everything builds fine, but there is again something wrong with CI...

@kafeg
Copy link
Contributor Author

kafeg commented May 6, 2020

Hello @strega-nil? Is everything ok? Could you merge this, please? 🙏

@strega-nil
Copy link
Contributor

Yep, I just somehow missed it in my notifications, sorry!

@strega-nil strega-nil merged commit 07f7b67 into microsoft:master May 6, 2020
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.

3 participants