Skip to content

Cmake before 3.11 requires a source argument#145

Merged
devhawk merged 1 commit into
microsoft:masterfrom
CaptainZippy:patch-1
Jan 25, 2019
Merged

Cmake before 3.11 requires a source argument#145
devhawk merged 1 commit into
microsoft:masterfrom
CaptainZippy:patch-1

Conversation

@CaptainZippy

Copy link
Copy Markdown
Contributor

No description provided.

@devhawk

devhawk commented Jan 25, 2019

Copy link
Copy Markdown
Contributor

Had to manually kick off PR builds due to mis-configuration of xlang's AzDO pipelines. PR validation from external forks should kick off automatically now.

UPDATE manual PR build failed as I don't apparently know how to trigger a PR validation build from an external repo. Still digging

@devhawk

devhawk commented Jan 25, 2019

Copy link
Copy Markdown
Contributor

@CaptainZippy can you push a trivial whitespace change to this PR in an attempt to kick off the PR validation?

@devhawk

devhawk commented Jan 25, 2019

Copy link
Copy Markdown
Contributor

I'm going to go ahead and merge this w/o validation builds. I setup my own xlang fork and submitted a test PR to ensure forked repo PR builds were working. I made the same change @CaptainZippy made, and it built fine.

@devhawk devhawk merged commit 4ba51d3 into microsoft:master Jan 25, 2019
@devhawk

devhawk commented Jan 25, 2019

Copy link
Copy Markdown
Contributor

THanks for your PR @CaptainZippy !

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.

3 participants