| 21 Nov 2024 |
 Tomodachi94 (they/them) | Update: tried to do that, I get strange errors relating to XML parsing | 18:27:37 |
| Tomodachi94 (they/them) | (Very specific /s, org/xml/sax/SAXParseException) | 18:28:55 |
| Tomodachi94 (they/them) | (Very specific /s, org/xml/sax/SAXParseException with no further information) | 20:35:56 |
| Tomodachi94 (they/them) | More information about my plans to rip out ant-contrib from the main ant package: https://github.com/NixOS/nixpkgs/issues/357758 | 22:02:41 |
| Tomodachi94 (they/them) | (Ideally this would go into the same staging cycle as the Ant setup hook PR for reasons listed in that issue and the PR) | 22:03:17 |
| Tomodachi94 (they/them) | * (Ideally this would go into the same staging cycle as the Ant setup hook PR) | 22:03:26 |
 emily | probably a week or two out from staging-next FWIW | 22:06:22 |
| Tomodachi94 (they/them) | Thanks ❤️ reviews are appreciated as always | 22:07:39 |
| Tomodachi94 (they/them) | Most of the complexity in the PR comes from allowing consumers to set flags (I'd say at least half of the SLOC is that) | 22:08:29 |
| emily | yeah, poke me in a few days (if everything is ready) and I'll try to look. trying to take it easy for a few days (we'll see if I manage 🙃) | 22:08:32 |
| Tomodachi94 (they/them) | In reply to@emilazy:matrix.org
yeah, poke me in a few days (if everything is ready) and I'll try to look. trying to take it easy for a few days (we'll see if I manage 🙃) For sure ❤️ hope you can find time to rest | 22:11:27 |
| 22 Nov 2024 |
| Tomodachi94 (they/them) | I'm wondering if we should move the ANT_HOME to $out/share/ant instead of $out/lib/ant before we hard-code that path in the findExternalAntTasks of the setup hook | 16:49:54 |
| Tomodachi94 (they/them) | I'm gonna do that, in a separate PR though | 17:02:42 |
| Tomodachi94 (they/them) | share/ant is the convention used by Debian and possibly others | 17:04:41 |
| Tomodachi94 (they/them) | ...and tons more too, see PR description: https://github.com/NixOS/nixpkgs/pull/358204 | 17:31:00 |
| Tomodachi94 (they/them) | * ...and tons of other distros too, see PR description: https://github.com/NixOS/nixpkgs/pull/358204 | 17:36:09 |
|  Morgan (@numinit) joined the room. | 17:50:01 |
| Tomodachi94 (they/them) | Welcome :) | 18:06:14 |
| Morgan (@numinit) | howdy :-) | 18:12:53 |
| Morgan (@numinit) | gradle2nix kind of rules my day job in certain ways, noticed that this exists and figured I'd poke my head in 😛 | 18:15:46 |
| 23 Nov 2024 |
| Tomodachi94 (they/them) | I'm wondering if we should make a temporary branch in Nixpkgs for all of my Ant refactors devel-ant, PRing from my fork into the branch. From there, merge devel-ant' into staging`, then delete it once it's done | 00:51:10 |
| Tomodachi94 (they/them) | I'm wondering if we should make a temporary branch in Nixpkgs for all of my Ant refactors devel-ant, PRing from my fork into the branch. From there, merge devel-ant into staging, then delete it once it's done | 00:52:54 |
| Tomodachi94 (they/them) | I'm wondering if we should make a temporary branch in Nixpkgs for all of my Ant refactors devel-ant, PRing from my fork into the branch. From there, merge devel-ant into staging, then delete it once it's merged there | 00:53:08 |
| Tomodachi94 (they/them) | It would save rebuilds and ensure the refactors all hit at the same time, but I'm not sure it's standard | 00:53:39 |
| emily | how many commits are we talking here in expectation? | 00:55:34 |
| emily | we did a separate branch to PR into for https://github.com/NixOS/nixpkgs/pull/354107, though that was a ~2-person collaborative project that took a couple weeks | 00:56:09 |
| emily | rebuilds aren't an issue – staging isn't built | 00:56:19 |
| emily | (that's why we have staging-next) | 00:56:24 |
| Tomodachi94 (they/them) | In reply to@emilazy:matrix.org
how many commits are we talking here in expectation? Unsure. That depends on how many dependencies on ant-contrib there are | 01:06:26 |
| emily | my intuition would be that you can get away with just having a PR | 01:10:19 |
