Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Add windows testing #6

Open
bogdandrutu opened this issue Dec 13, 2017 · 8 comments
Open

Add windows testing #6

bogdandrutu opened this issue Dec 13, 2017 · 8 comments
Assignees

Comments

@bogdandrutu
Copy link

Probably use appveyor for that.

@g-easy
Copy link
Contributor

g-easy commented Feb 23, 2018

I've started work on this.

@g-easy
Copy link
Contributor

g-easy commented Feb 23, 2018

This is what I've got so far: https://github.com/g-easy/opencensus-cpp/tree/appveyor

It gets surprisingly far but build is broken on the proto targets with underscores that we filter out on Travis, and test is broken somewhere in protobuf. I'll try to resume work on this next week.

Example output: https://ci.appveyor.com/project/g-easy/opencensus-cpp

@g-easy
Copy link
Contributor

g-easy commented Feb 26, 2018

I'm running into Cannot open compiler generated file: '': Invalid argument - this is bazelbuild/bazel#4149 (cl.exe can't handle long paths and gives a confusing error)

My workaround for now is:

mkdir c:\t
bazel --output_user_root=c:/t/ build //targets

@isturdy
Copy link
Contributor

isturdy commented Mar 1, 2018

Worked a bit on this in #81--presently blocked on boringssl selecting posix copts rather than windows. I am not certain what the cause is; they seem to be using the same setup that I successfully used for google benchmark.

@isturdy
Copy link
Contributor

isturdy commented Mar 1, 2018

Apparently gRPC pinned their boringssl dependency on a who-knows-how-old version with broken Windows builds under bazel. (Trying to fix with grpc/grpc#14566)

@bogdandrutu
Copy link
Author

I think this is fixed. Please reopen if work is still left here.

@g-easy
Copy link
Contributor

g-easy commented Sep 25, 2018

We don't build everything and we don't test anything.

It looks like isturdy was working on (thanks!) getting the gRPC example / integration to build but ran into a grpc issue.

@g-easy g-easy reopened this Sep 25, 2018
@g-easy
Copy link
Contributor

g-easy commented Mar 20, 2019

#292 replaced Appveyor Bazel build with a CMake one. I'd like to see Bazel come back, and also for everything to be built and tested like it is on Travis/Linux.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants