Dylan Baker
Dylan Baker
I've adde a bunch of stuff to make working with run_project_tests more pleasant, as I had to run a few tests over and over on windows to figure some of...
I need to find a chance to rebase it and fix the failing tests, just not right at the top of queue right now
Well, since I use the canonical two space indent (:D), I'd expect this: ```meson if (cc.links( ''' bool func(uint foo) { return foo < MAX_INT; } ''', )) cargs +=...
My principle problem is that whenever I see: ``` if ( ... ) ... ``` This is super confusing, because `^\)` It makes me think "This if block is done,...
I suspect we'll want an option for this, because from my experience in python people either really love the: ```meson if ( condition ) body endif ``` style, or they...
@dnicolodi: I've added an explicit method for checking the choices, so all of that is hidden inside the options module, which I think is a bit more elegant than having...
@dnicolodi fixed the typo, and dropped the range patch. I played around with getting better error messages, and just felt like it wasn't worth the amount of code involved.
I think this one, and then following that: https://github.com/mesonbuild/meson/pull/13737. Which would give us a really solid base of typing guarantees to build the option refactor on top of so that...
From my perspective I've addressed all of the review feedback, with the exception of the question about `__post_init__` vs `init=False`. I have a preference but if the wider consensus is...
I can reproduce this on Linux, so it doesn't appear to be Windows specific. I haven't looked into it further, but it seems from the updated error message in the...