Danny McClanahan

Results 64 comments of Danny McClanahan

I believe a lot of this exists verbatim in the native backend? Is there an ideal format in which I can draw out how this could be hooked up? All...

> There is another important aspect to this ticket, which is how the "Platform-specific-Subsystem" instances would be constructed. We had discussed having `@rule`s to construct those instances by consuming a...

Also note that there is specific precedent for setting env vars specifically for subprocesses via subsystem/v2 ruleset in the native backend, starting hereish: https://github.com/pantsbuild/pants/blob/46c22fa9917dda2b0cf2c1d5880b8223454ae11d/src/python/pants/backend/native/config/environment.py#L173-L176

#10870 describes a method for making mutable caches remote-friendly. In that issue, it **currently** describes placing a `digest_hint` file to avoid having to re-snapshot the entire cache each time. However,...

> I am wondering whether we should drop fast-deps handling, given that: In general, I support this, since I didn't spend the time a year+ ago to follow up on...

i hid my previous comment since it wasn't feasible, but wanted to restate that I think removing fast-deps handling until it can be shown to improve performance is probably a...

> A huge % of wheels will have the entire `.dist-info` directory in the last 8-10 kb of the zip archive. I might be misunderstanding you, but this exact realization...

@jrose-signal I've chosen to tag each commit on `docs-effort` with the pull request it's from: see the collected changes at https://github.com/signalapp/libsignal/compare/main...cosmicexplorer:libsignal-client:docs-effort?expand=1. This means that you can quickly verify that each...

That's really weird. I know helm declares async as a dependency -- did you have `async` and `helm` already installed when you ran `use-package`? What versions are they if so?...

Ah, I see #399 (mentioned directly above `parse_iter()`) covers exactly this issue, not a different one. I'll see if I can page into that.