Skip to content

Conversation

JackBoosY
Copy link
Contributor

@JackBoosY JackBoosY commented Feb 13, 2020

The current CMakeLists.txt does not support building in UNIX, so add support.

Realted: #9784.

Note: this port doesn't contain any features.

@PhoebeHui
Copy link
Contributor

/azp run

1 similar comment
@PhoebeHui
Copy link
Contributor

/azp run

@JackBoosY JackBoosY marked this pull request as ready for review February 20, 2020 02:33
@dan-shaw dan-shaw added info:reviewed Pull Request changes follow basic guidelines and removed needs-further-review labels Mar 20, 2020
@dan-shaw
Copy link
Contributor

/azp run

@dan-shaw dan-shaw added waiting for response and removed info:reviewed Pull Request changes follow basic guidelines labels Mar 20, 2020
@dan-shaw
Copy link
Contributor

@JackBoosY CI fails on Linux

@JackBoosY
Copy link
Contributor Author

@dan-shaw Re-run and the test was successful.

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Mar 26, 2020
@dan-shaw
Copy link
Contributor

Should the CI baseline be updated and remove linux=fail?

@dan-shaw dan-shaw added waiting for response and removed info:reviewed Pull Request changes follow basic guidelines labels Mar 28, 2020
@JackBoosY
Copy link
Contributor Author

/azp run

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label Apr 2, 2020
@dan-shaw
Copy link
Contributor

dan-shaw commented Apr 6, 2020

/azp run

@JackBoosY
Copy link
Contributor Author

@dan-shaw Ready to merge now.

@ras0219-msft ras0219-msft merged commit 981daa4 into microsoft:master Apr 27, 2020
@JackBoosY JackBoosY deleted the dev/jack/9784 branch April 28, 2020 02:12
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