Skip to content

Conversation

PhoebeHui
Copy link
Contributor

@PhoebeHui PhoebeHui commented May 21, 2020

Related to #11427

Fix the The following issues in CI baseline

  1. Building boost-* with arm-uwp requires the x86-windows boost-* which breaks CI testing, Ignore them
    boost-atomic:arm-uwp=ignore
    boost-signals:arm-uwp=ignore
    boost-exception:arm-uwp=ignore
    boost-regex:arm-uwp=ignore
    boost-nowide:arm-uwp=ignore
    boost-system:arm-uwp=ignore
    boost-container:arm-uwp=ignore

  2. Resolve conflicts bettwen microsoft-signalr and signalrclient, they are different repo, and they don't depend on each other.

Failures:
Installing package microsoft-signalr[core,cpprestsdk,default-features]:x86-windows...
The following files are already installed in F:/vcpkg/boost_173/vcpkg/installed/x86-windows and are in conflict with microsoft-signalr:x86-windows

Installed by signalrclient:x86-windows
bin/signalrclient.dll
debug/bin/signalrclient.dll
debug/lib/signalrclient.lib
include/signalrclient/_exports.h
include/signalrclient/connection.h
include/signalrclient/connection_state.h
include/signalrclient/hub_connection.h
include/signalrclient/hub_exception.h
include/signalrclient/log_writer.h
include/signalrclient/trace_level.h
include/signalrclient/transport_type.h
include/signalrclient/web_exception.h
lib/signalrclient.lib

  1. Remove itk:x64-osx=fail in CI baseline file.
    PASSING, REMOVE FROM FAIL LIST: itk:x64-osx (/Users/vagrant/Data/work/1/s/scripts/ci.baseline.txt)

@PhoebeHui PhoebeHui changed the title [baseline] Fix boost-*:arm-uwp failure and resolve conflicts in CI [Baseline] Fix boost-*:arm-uwp failure and resolve conflicts in CI May 21, 2020
@PhoebeHui
Copy link
Contributor Author

@strega-nil, could you help merge this PR?

@JackBoosY JackBoosY added the info:reviewed Pull Request changes follow basic guidelines label May 22, 2020
@BillyONeal
Copy link
Member

Sorry, I broke this with #11511

@strega-nil
Copy link
Contributor

Thanks @PhoebeHui :)

Hopefully @BillyONeal can refix this next week :3

@strega-nil strega-nil merged commit 2ac2b5f into microsoft:master May 23, 2020
@BillyONeal
Copy link
Member

@strega-nil #11545 should fix :)

@PhoebeHui PhoebeHui deleted the dev/Phoebe/FixBaseline branch May 26, 2020 06:48
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