| 16 Nov 2024 |
 atemu12 | Could you try building cheeseburger or FP4? Just to make sure there's nothing funky with those specific devices | 02:16:56 |
 oak 🏳️🌈♥️ | Maybe they added some commit that they want to include the repo manifest as part of the build | 02:16:57 |
| oak 🏳️🌈♥️ | Testing with cheeseburger now | 02:17:29 |
| atemu12 | I highly doubt they'd do that this far into LOS19's life (or death, really) | 02:17:34 |
| atemu12 | Did you update its device and vendor dirs too? | 02:17:47 |
| 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 |
| atemu12 | Okay so we'd expect the same error then if this isn't device-specific | 02:19:39 |
| atemu12 | Though honestly at this point I'd recommend you to just take that version of LOS19 | 02:20:01 |
| atemu12 | It updates to 20 just fine | 02:20:08 |
| oak 🏳️🌈♥️ | but yes cheeseburger also provides same error | 02:21:20 |
| atemu12 | 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 |
| atemu12 | 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 |
| atemu12 | Ah right | 02:23:48 |
| atemu12 | 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 |
| atemu12 | Any removed entries in the device dirs for instance? | 02:24:31 |
| atemu12 | You could basically also diff the source.dirs | 02:25:02 |
| atemu12 | 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 |
| atemu12 | 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 |
| atemu12 | Not sure | 02:30:24 |
| atemu12 | I can't remember seeing this error but I'd have to check the diff for the LOS 20 merge | 02:30:44 |
| atemu12 | Ah | 02:31:41 |
| atemu12 | e1b99ec7a9bf57d4e13e157ed289a1943d1e4849 | 02:31:42 |
