cpptrace icon indicating copy to clipboard operation
cpptrace copied to clipboard

Published 20 hours ago • verified publisher icon jeremy-rifkin

Enable Bazel build system support

Open WSUFan opened this issue 1 year ago • 4 comments

This pull request enables Bazel build support for cpptrace. Please note that currently, only Linux is supported. Additional platform support will be added if necessary.

To build with bazel, simply run

bazel build //...

To test with bazel, simply run

bazel test //...

WSUFan avatar Aug 09 '24 00:08 WSUFan

Can you approve Github workflows for this PR? Let's see how it goes. image

WSUFan avatar Aug 11 '24 18:08 WSUFan

I can

jeremy-rifkin avatar Aug 11 '24 22:08 jeremy-rifkin

Looks like there were a lot more test failures than I would have expected due to trying to build cmake in a folder called build. That's really unfortunate, I don't like case-insensitive filesystems. Calling it BUILD.bazel might be best?

jeremy-rifkin avatar Aug 11 '24 23:08 jeremy-rifkin

Looks like there were a lot more test failures than I would have expected due to trying to build cmake in a folder called build. That's really unfortunate, I don't like case-insensitive filesystems. Calling it BUILD.bazel might be best?

Yeah, I changed the filename. But to run Github workflows, still need another approval? :(

WSUFan avatar Aug 12 '24 01:08 WSUFan

It looks like CI is failing to build, happy to merge once it builds

jeremy-rifkin avatar Aug 18 '24 16:08 jeremy-rifkin

It looks like CI is failing to build, happy to merge once it builds

Yeah. This is strange because I tested it successfully in my env (using ubuntu 22.04 image). But I changed some build configs and let's see.

WSUFan avatar Aug 18 '24 18:08 WSUFan

It looks like that passed, sweet! Thanks again for taking the time to set this up!

jeremy-rifkin avatar Aug 19 '24 00:08 jeremy-rifkin