Austin Schuh

Results 25 comments of Austin Schuh

Somehow I had persuaded myself that 9) doesn't need a format change and that it could be done by just building from the other end of the buffer and allocating...

> > * I would almost prefer an inversion of the current required/optional field status, having fields set to be required by default unless annotated to be optional. > >...

> @AustinSchuh > > > The offset for the scalar fields in the vtable is 0 if they aren't populated > > Yes, and that means that the value is...

Ran 5.3 RC1 on our build against our remote execution cluster and it fails to run tests. https://github.com/bazelbuild/bazel/issues/16003 is the bug I filed. This is a regression from 5.0, which...

We are using rawrtc in https://github.com/frc971/971-Robot-Code (a hard RT pubsub framework for robotics applications) for all our plotting and debug webpages. The dependencies make it a lot easier to work...

Aw, I was hopeful. I fixed some merge conflicts (looks like "../".replace() -> Strings.replace("../", ...) and I still get the same failure. ``` ==================== Test output for @aos//aos:condition_test: /var/lib/worker/work/3/exec/bazel-out/k8-opt/bin/external/aos/aos/condition_test.runfiles/repo/../aos/aos/condition_test: error...

It does! Great work, thanks for the prompt response and prompt resolution. I really appreciate it. FYI, there's the same merge conflict with Strings.replace. Not a hard thing to fix...

Aw, @fmeum , looks like this fixed almost all of the tests except one which expects $(location) and RPATH to agree on the path to the shared libraries. https://github.com/bazelbuild/bazel/issues/16108 is...

5.3.1 works for me. I'm hitting https://github.com/bazelbuild/bazel/issues/16108 but that feels separate.

Oooh, thanks @CasperN ! That looks very helpful. It looks like it only supports 1 path, I might need to teach it to support multiple paths. But that's easier than...