| 16 Nov 2024 |
 oak 🏳️🌈♥️ | Yes | 02:18:33 |
| oak 🏳️🌈♥️ | Ah it's so awful to work with all these Android tools sometimes, maybe I'll try to check the latest merged commits from the lineageos gerrit | 02:19:06 |
 Atemu | Okay so we'd expect the same error then if this isn't device-specific | 02:19:39 |
| Atemu | Though honestly at this point I'd recommend you to just take that version of LOS19 | 02:20:01 |
| Atemu | It updates to 20 just fine | 02:20:08 |
| oak 🏳️🌈♥️ | but yes cheeseburger also provides same error | 02:21:20 |
| Atemu | Could you stash and check whether that error is present on the commit I gave you too? Just a sanity check | 02:22:02 |
| oak 🏳️🌈♥️ | You mean this commit? https://github.com/nix-community/robotnix/commit/f4d5a275a8e7381d08460a633e94d29147f61760 | 02:23:03 |
| Atemu | Yup | 02:23:15 |
| oak 🏳️🌈♥️ | I already said it built the instantnoodlep version without any issues if I don't modify it, but if I run the update scripts then this issue of it trying to run repo appears | 02:23:40 |
| Atemu | Ah right | 02:23:48 |
| Atemu | Anything interesting in the diff? | 02:23:59 |
| oak 🏳️🌈♥️ | So I would assume the cheeseburger version also builds fine or at least doesn't have this issue | 02:24:03 |
| Atemu | Any removed entries in the device dirs for instance? | 02:24:31 |
| Atemu | You could basically also diff the source.dirs | 02:25:02 |
| Atemu | Or if you want to use nix-diff then the debug-enter-env drvs | 02:25:17 |
| oak 🏳️🌈♥️ | Well I don't want to spend too much time figuring this out, but if it would have been something like reverting one commit from one of the repos to make it work again I'd had done that level of effort | 02:27:21 |
| oak 🏳️🌈♥️ | What could be the repository where they have added this manifest generation into the build? | 02:27:57 |
| Atemu | I'd guess https://github.com/LineageOS/android_vendor_lineage | 02:28:42 |
| oak 🏳️🌈♥️ | https://review.lineageos.org/c/LineageOS/android_vendor_lineage/+/356656 I probably found it | 02:28:52 |
| oak 🏳️🌈♥️ | Haven't they also done this for 20 and 21? | 02:29:56 |
| Atemu | Not sure | 02:30:24 |
| Atemu | I can't remember seeing this error but I'd have to check the diff for the LOS 20 merge | 02:30:44 |
| Atemu | Ah | 02:31:41 |
| Atemu | e1b99ec7a9bf57d4e13e157ed289a1943d1e4849 | 02:31:42 |
| oak 🏳️🌈♥️ | https://review.lineageos.org/c/LineageOS/android_vendor_lineage/+/356657 I see this for 20 | 02:32:08 |
| Atemu | Wait where's the patch | 02:33:00 |
| Atemu | Ah, I was on a super old branch on this machine | 02:33:33 |
| Atemu | 11e02c0ce2452b3822a2f8695ac600b6e245ac9a | 02:34:00 |
| oak 🏳️🌈♥️ | Am I seeing correctly that this commit actually adds a patch and fix for the issue, but it's not used anywhere | 02:37:43 |
