| 19 Aug 2025 |
 emily | position doesn't tell you what overrides might apply, etc. | 18:49:39 |
| emily | the correct solution is to do it by .drv, which is how nixpkgs-review --depth would work if someone felt like implementing it | 18:49:52 |
 jade_ | oh sure, i just want to know attr names that come from a given file name | 18:49:53 |
| emily | you can just arrange the graph of changed .drvs such that you only test things that aren't changing only because something upstream changed, and tier those so that --depth 1 builds directly changed packages, --depth 2 builds packages that use directly changed packages, --depth 3 builds packages that use those, etc. | 18:50:47 |
| jade_ | https://github.com/Mic92/nixpkgs-review/issues/554 filed a bug so that it's at least tracked where someone might find it if they're bored | 18:52:25 |
| jade_ | how does it match drvs? trivially by name or by similarity metric plus name or? | 18:52:46 |
| emily | I had a more written up design somewhere but not to hand. I think you do not actually need to match .drvs at all | 18:53:15 |
| emily | like it's something approximating "cut off things that mention the .drvs you are rebuilding in this tier". and the lowest depth is, rebuild packages that do not mention any .drvs that are present in the before eval | 18:54:32 |
| emily | though that would punt things you directly change but that depend on other things you directly change to the next depth | 18:54:46 |
| emily | to avoid that you'd need to match up .drvs yeah | 18:54:58 |
| jade_ | mmm | 18:55:41 |
| jade_ | the lack of ability to get metadata into drvs is rather irritating | 18:55:51 |
 K900 | I have like 80% of this in nix-drv-index tbh | 18:56:42 |
 aloisw | In reply to @raitobezarius:matrix.org
oh lol vcunat intervening in that commit in the comment area I remember a staging-next breaking due to the introduction of incorrect floating-point behaviour in unrelated code. | 19:50:50 |
| jade_ | (
(source_code
expression: (function_expression
formals: (formals
formal: (formal name: (identifier) @ident)
)
)
)
(#any-of? @ident "pytestCheckHook" "pytest")
)
well that didn't require much thinking
| 20:58:01 |
 raitobezarius | WeetHet https://git.lix.systems/lix-project/lix/issues/966#issuecomment-14125 sadness | 20:59:19 |
| raitobezarius | also darwin CI still not my friend | 21:00:10 |
| raitobezarius | https://gerrit.lix.systems/c/lix/+/4004 | 21:00:14 |
| jade_ | https://bytes.zone/posts/tree-grepper/ wow this is cool. sorta off topic. i am going to probably write a nixpkgs package. | 21:19:13 |
 piegames | Woow | 21:40:04 |
| piegames | Need | 21:40:05 |
| emily | we have https://github.com/ast-grep/ast-grep iirc | 21:41:13 |
| emily | I believe people have used it for Nixpkgs work | 21:41:19 |
| emily | probably more Production Ready™ | 21:41:47 |
| jade_ | it, very frustratingly, does not take tree sitter queries in s-expr form | 21:49:10 |
| jade_ | i have tried it before, it has a bunch of opinions which annoy me a varying amount depending on the day | 21:49:34 |
| jade_ | mostly just that repos should be configured for it (which is totally fair and defensible, but someone might whine about it in nixpkgs) | 22:00:14 |
| jade_ | https://github.com/NixOS/nixpkgs/pull/435115 have! (with review request attached :P) | 22:27:22 |
| 20 Aug 2025 |
|  Federico Damián Schonborn (he/they) changed their profile picture. | 01:06:55 |
| raitobezarius | In reply to @raitobezarius:matrix.org
https://gerrit.lix.systems/c/lix/+/4004 that revert might not make sense in the end | 10:20:22 |
