15 Sep 2025 |
raitobezarius | basically, we need flakes-compat to support the follows system | 18:40:49 |
raitobezarius | and then you wouldn't have to deal with this | 18:40:55 |
raitobezarius | but you'd lose all the flakes CLI | 18:41:03 |
raitobezarius | let's just fix that bug I'd say | 18:41:08 |
laurents | so git+file:// instead of path:// fixes it :) | 18:41:26 |
raitobezarius | great :) | 19:02:39 |
raitobezarius | I still think some warning to recommend git+file:// is relevant | 19:03:21 |
laurents | I'll see if I can work out all the layers of stuff I have to learn to fix that piece of code | 19:03:37 |
laurents | but yes, I agree that a nice warning would be better | 19:04:00 |
raitobezarius | perhaps if (newRef.input.scheme != "git") { warn("You are using blablablabla"); } as an addition would be sufficient | 19:09:16 |
raitobezarius | (and a test) | 19:09:20 |
laurents | dumb question (again), but I'm trying to redo what you did above, replacing the github: ref with a git+https:// and I keep getting git errors, did you change anything else to make it work? | 19:26:08 |
raitobezarius | i had to modify manually the flake.lock | 20:04:51 |
raitobezarius | because if you go from the one you had, it's already broken | 20:04:57 |
raitobezarius | so i nuked all the entries regarding your input | 20:05:04 |
raitobezarius | and re-inserted it | 20:05:06 |
16 Sep 2025 |
| olivia [she/it] 🏳️⚧️ changed their profile picture. | 06:59:57 |
| olivia [she/it] 🏳️⚧️ changed their profile picture. | 09:04:49 |
laurents | well well, it looks like using a git+https://github.com/some/repo input does not work if said repo is a fork. If I clone the repo, and create a new one on github that is not a fork, say git+https://github.com/some/repo_that_is_not_a_fork , then lix is happy to use it as an input. | 12:27:03 |
raitobezarius | absolutely unhinged bug | 12:57:28 |
laurents | ok, apologies, ignore me, somehow the nix cache got into an invalid state. once I removed it, the fork version worked again. I'm not sure how it got into that broken state, but I think I will leave it at that for now. Sorry for the noise. | 13:27:00 |
raitobezarius | don't worry, it's called flakes for a reason | 13:30:57 |
aloisw | How are the fallback code owners decided? https://gerrit.lix.systems/c/lix/+/4135 has approval from someone in the core team but still doesn't have the code owner check satisfied. | 15:29:09 |
jade_ | aloisw: it is a file in the refs/meta/config branch that has me, raito and pennae in it. that could of course get changed/fixed though. | 17:33:56 |
jade_ | (the intent of it was to not have it apply to anything, but that has evidently been shown to be wrong sometimes) | 17:47:14 |
jade_ | https://gerrit.lix.systems/c/lix/+/4164 let's fix those ones cuz they are definitely wrong | 17:50:44 |
aloisw | perl also seems to have no owners. | 18:01:30 |
jade_ | aloisw: fixed | 20:30:59 |
17 Sep 2025 |
just1602 | I wish it was also supporting gerrit OWNER file: https://github.com/vim/vim/pull/18299/files | 00:11:28 |
emily | put up a new version of the build-dir stuff with a fix for the regression (and a test that hopefully works) | 01:29:06 |