opencensus-cpp icon indicating copy to clipboard operation
opencensus-cpp copied to clipboard

Add windows testing

Open bogdandrutu opened this issue 7 years ago • 8 comments

Probably use appveyor for that.

bogdandrutu avatar Dec 13 '17 03:12 bogdandrutu

I've started work on this.

g-easy avatar Feb 23 '18 05:02 g-easy

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 avatar Feb 23 '18 07:02 g-easy

I'm running into Cannot open compiler generated file: '': Invalid argument - this is https://github.com/bazelbuild/bazel/issues/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

g-easy avatar Feb 26 '18 12:02 g-easy

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 avatar Mar 01 '18 22:03 isturdy

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

isturdy avatar Mar 01 '18 22:03 isturdy

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

bogdandrutu avatar Sep 24 '18 22:09 bogdandrutu

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 avatar Sep 25 '18 22:09 g-easy

#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.

g-easy avatar Mar 20 '19 03:03 g-easy