zjturner
zjturner
I don’t remember if deps attribute allows expressions, but perhaps it allows query functions
I posted this elsewhere in another issue, but since this is relevant. Try [this bxl](https://gist.github.com/zjturner/0a9c99d8e4fa50dbd7a74d2c1637cc63). It should be quite a bit faster because it is based on aquery and doesn't...
I eventually realized I didn’t need to parse the args file to build the args, I could just have the compile_commands reference the argsfile as is. The majority of the...
I’ll do it as soon as I’m in the office next week
Ok, I made a PR. First time, there doesn't seem to be a way to explicitly add reviewers, so hopefully it just gets seen naturally. Not sure how the process...
> "platform" and "configuration" mean more or less the same thing, from the documentation: I am not sure this is true (but as you point out, some things are unclear,...
> The way we do it at Meta is put all the C++ compilers on a shared drive and then in the toolchain point at that shared drive. I wouldn't...
Oh man, it looks like this works 🤣 ``` metadata = json.decode(str(target.attrs_eager().metadata)) ```
If you can’t read it from bxl, then where _are_ you supposed to read it from? Regardless of whether the restriction is lifted, it feels like I'm missing the point,...
Perhaps targets need a metatdata attribute directly from bxl. But still, the json decode will work around it from within a rule anyway it seems