| 23 Sep 2025 |
Wolfgang Walther | exactly | 12:54:43 |
Vladimír Čunát | So, when are we switching to pijul? 😆 | 12:58:38 |
Fabián Heredia | Hi, for cmake 4 I want to pull a patch from sourceforge but I can't seem to find an easy way to do that and seems like other maintainers have opted to inline / commit the patch into a file in nikpkgs. wsid?
https://sourceforge.net/p/panotools/libpano13/ci/698e20b4d296c1dbde9d010c3fb8d54050e56ddb/ | 15:03:10 |
Fabián Heredia | * Hi, for cmake 4 I want to pull a patch from sourceforge but I can't seem to find an easy way to do that and seems like other maintainers have opted to inline / commit the patch into a file in nikpkgs. What should I do?
https://sourceforge.net/p/panotools/libpano13/ci/698e20b4d296c1dbde9d010c3fb8d54050e56ddb/ | 15:03:21 |
K900 | IMO just vendor it | 15:06:49 |
K900 | It's three lines who gives a shit | 15:06:52 |
Vladimír Čunát | Or even substituteInPlace 😄 | 15:39:51 |
emily | (I would be fine with substituteInPlace) | 15:47:58 |
Lun | docker-tini has a merged CMake 4 patch, doesn't apply on top of the newest tagged release
do I substitute or bump to an unreleased rev that has the fix? | 15:59:47 |
emily | how old is the release? | 16:00:03 |
Lun | diff from last tag to HEAD https://github.com/krallin/tini/compare/v0.19.0...369448a167e8b3da4ca5bca0b3307500c3371828 | 16:00:16 |
emily | 5 years and benign fixes, including one that might avoid a warning disable flag or such? I'd bump yeah | 16:03:55 |
Lun | lixPackageSets.git is failing some toml tests with overflow errors
do we just need to bump it past https://git.lix.systems/lix-project/lix/commit/4de09b6b5493db4bd7f6348255a1fdcb38b9ed2f ? | 16:06:31 |
hexa | cc aloisw | 16:07:27 |
aloisw | Current version has that toml11 4 patch reverted due to incompatibility with the toml11 3 in unstable (and 25.05). https://github.com/NixOS/nixpkgs/pull/444623 has a newer version of the patch that supports both toml11 versions. | 16:08:42 |
emily | we already have the toml11 bump in -next | 16:09:52 |
emily | is there a reason not to drop the patch and just merge it into there? | 16:10:08 |
Grimmauld (any/all) | https://github.com/NixOS/nixpkgs/pull/445081 i am wondering, if cmake is broken, is just not building with cmake a fix we would want? I am a bit worried, because things downstream may break if they expect the cmake files to be present | 16:10:17 |
Lun | I can hit that with a review run and see if the deps get upset if you want | 16:11:12 |
Lun | seems likely to cause problems but it depends how dependents find it - they might already be shipping a FindX.cmake that works for this case or something | 16:11:45 |
Grimmauld (any/all) | thats what i would have done in the next step too, but i first wanted to check in whether the approach is even worth considering | 16:11:49 |
aloisw | 25.05 basically, if you prefer I can also prepare a drop of the patch against staging-next. | 16:11:52 |
Lun | substituting the cmake version seems preferable to me assuming it's workable and there isn't a whole bunch of cruft that broke and needs a proper patch in it | 16:12:36 |
Grimmauld (any/all) | there is also an upstream patch | 16:12:49 |
Grimmauld (any/all) | imo fetching that is the way | 16:12:59 |
Grimmauld (any/all) | Actually should be fine | 16:15:05 |
Grimmauld (any/all) | the cmake files aren't being installed anyways lol | 16:15:12 |
Grimmauld (any/all) | (checked against current unstable) | 16:15:26 |
Grimmauld (any/all) | i'll diffoscope to make sure i am not missing anything, and then just merge tbh | 16:19:36 |
Grimmauld (any/all) | Ah okay it does break | 16:20:38 |