Skip to content

[vcpkg baseline] Fix CI test#10508

Merged
dan-shaw merged 13 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/fix_ci_20200322
Mar 26, 2020
Merged

[vcpkg baseline] Fix CI test#10508
dan-shaw merged 13 commits intomicrosoft:masterfrom
JackBoosY:dev/jack/fix_ci_20200322

Conversation

@JackBoosY
Copy link
Contributor

No description provided.

@PhoebeHui PhoebeHui changed the title Fix CI test [vcpkg baseline] Fix CI test Mar 23, 2020
@dan-shaw
Copy link
Contributor

/azp run

1 similar comment
@dan-shaw
Copy link
Contributor

/azp run

@JackBoosY
Copy link
Contributor Author

JackBoosY commented Mar 23, 2020

Now, in most cases, the linux and osx pipelines exit by error:
[error]PowerShell exited with code '1'.
@dan-shaw Can you see the specific cause in detail on the ci machine?

@dan-shaw
Copy link
Contributor

For kealib, the repository has been deleted and moved.

@JackBoosY
Copy link
Contributor Author

@dan-shaw Already fixed on my machine, wait for other ports test results.

@dan-shaw
Copy link
Contributor

intel-ipsec requires NASM, I've just added it to the Linux CI, so we may need to rerun the pipeline

@dan-shaw
Copy link
Contributor

For mongo-cxx-driver failure (on master), it looks like the ICU library is being auto-detected on mongo-c-driver sometimes, which may lead to issues as described here: #7338

@JackBoosY
Copy link
Contributor Author

@dan-shaw After checking mongo-c-driver's CMakeLists.txt, I found that ICU is an option, so I will set icu to mongo-c-driver feature.

@JackBoosY
Copy link
Contributor Author

Monkeys-audio failed because the source could not be downloaded.

@JackBoosY
Copy link
Contributor Author

Now the last failing item is llvm.

@dan-shaw
Copy link
Contributor

I'm OK with failing llvm on the baseline. We are going to update llvm soon #10295

@JackBoosY
Copy link
Contributor Author

@dan-shaw The llvm failure is:

Creating llvm-ranlib.exe
Creating llvm-lib.exe
Creating llvm-dlltool.exe
CMake Error at tools/llvm-lto/cmake_install.cmake:36 (file):
  file INSTALL cannot copy file
  "C:/vsts/_work/4/s/buildtrees/llvm/x64-windows-rel/bin/llvm-lto.exe" to
  "C:/vsts/_work/4/s/packages/llvm_x64-windows/tools/llvm/llvm-lto.exe".
Call Stack (most recent call first):
  tools/cmake_install.cmake:41 (include)
  cmake_install.cmake:63 (include)

I've seen this many times and don't know what went wrong. Maybe a permission issue? Or is it a disk space issue?

@dan-shaw
Copy link
Contributor

@JackBoosY I'm not sure. I would need to investigate.

@PhoebeHui
Copy link
Contributor

PhoebeHui commented Mar 23, 2020

Similar issue offen happens to llvm, see #10051 (comment), rerun the PR on windows.

Update: after rerun, this issue still exist.

@JackBoosY
Copy link
Contributor Author

Trying to remove default feature tools to resolve this issue completely.

@JackBoosY
Copy link
Contributor Author

LINK : fatal error LNK1201: error writing to program database 'C:\vsts\_work\4\s\buildtrees\llvm\x64-windows-static-rel\bin\c-index-test.pdb'; check for insufficient disk space, invalid path, or insufficient privilege

@PhoebeHui
Copy link
Contributor

@dan-shaw, could you help have a look?

This was referenced Mar 25, 2020
@PhoebeHui
Copy link
Contributor

@JackBoosY, could you disable llvm in baseline temporarily for unblocking other PRs?

@JackBoosY JackBoosY marked this pull request as ready for review March 26, 2020 03:16
@JackBoosY JackBoosY requested a review from PhoebeHui March 26, 2020 03:16
@dan-shaw dan-shaw merged commit d98696d into microsoft:master Mar 26, 2020
@JackBoosY JackBoosY deleted the dev/jack/fix_ci_20200322 branch March 26, 2020 04:00
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