| 3 Nov 2023 |
 Janne Heß | There's https://github.com/tpwrules/nixos-apple-silicon | 15:33:01 |
| 6 Nov 2023 |
|  adamcstephens 🐝 joined the room. | 16:00:47 |
| adamcstephens 🐝 | 👋 | 16:01:18 |
 stigo | hey adamcstephens 🐝 :) | 16:27:48 |
| stigo | adamcstephens 🐝: I'm happy to help out with fixing slimserver, prefering to use packages from perlPackages of course. I suggest that we wait a bit for staging-next to be merged into master is it contains a lot of changes to perlPackages, and a clang update that affects darwin packages in particular. Ref: https://github.com/NixOS/nixpkgs/pull/265556#issuecomment-1795211682 | 16:33:51 |
| stigo | Another concern would be the dependency on perl 5.36, as it will be removed from nixpkgs when 5.40 is released. It could be a dependency that wasn't compatible with 5.38, or ofc something in slimserver itself. We should also look at what the problem was there and see if it can be easily patched. | 16:39:49 |
| adamcstephens 🐝 | i'm unfamiliar with the perl lifecycle, but it looks like that gives us until 2025 for 5.36? | 17:06:09 |
| adamcstephens 🐝 |
To the best of our ability, we will provide "critical" security patches / releases for any major version of Perl whose 5.x.0 release was within the past three years
| 17:07:47 |
| adamcstephens 🐝 | and perl 5.36 was released May 28th 2022 | 17:08:01 |
| adamcstephens 🐝 | or do we drop releases earlier than upstream stops security patching? | 17:08:25 |
| stigo | A new version of Perl5 is released roughly every year or so, and in nixpkgs we keep the two currently maintained versions. | 17:09:03 |
| stigo | Which today are 5.36 and 5.38 | 17:11:27 |
| adamcstephens 🐝 | so 5.36 will be good until roughly may-july next year (depending on release date) | 17:12:09 |
| stigo | Yeah basically | 17:13:09 |
| stigo | Usually backwards compat is not a big issue in perl5, but there are of course some issues :) | 17:14:27 |
| stigo | I've proposed to also only keep one version of perl5 in nixpkgs to make maintenance a bit easier, maybe two if/when perl7 ships. | 17:15:09 |
| adamcstephens 🐝 | i suspect that may be fine with slimserver, if we can get it using our perl packages instead of its own | 17:16:00 |
| adamcstephens 🐝 | so to try and move forward, am i correct in assuming i should be removing this directory and replacing it with buildInputs ? https://github.com/Logitech/slimserver/tree/public/8.3/CPAN | 17:17:19 |
| adamcstephens 🐝 | and if so, any tips to save me the time of manually tracking down all the associated packages? :) | 17:18:05 |
| stigo | In reply to @adam:robins.wtf
so to try and move forward, am i correct in assuming i should be removing this directory and replacing it with buildInputs ? https://github.com/Logitech/slimserver/tree/public/8.3/CPAN I'd guess so, unless something needs to be patched for it to not use the CPAN directory. First I'd check what kind of error messages you get from just using the CPAN directory, likely there are some modules that are not compatible with 5.38 or maybe som nix specific patches that are missing. | 17:19:59 |
| adamcstephens 🐝 | it's currently failing with the error in https://github.com/NixOS/nixpkgs/issues/265488 | 17:21:00 |
| stigo | But I think we should wait for staging-next before trying, as it has over 600 commits to perlPackages :) I hope it will be merged into master very soon. | 17:21:08 |
| adamcstephens 🐝 | in that CPAN directory is an arch directory which has pre-compiled modules in it | 17:21:32 |
| adamcstephens 🐝 | (and doesn't have 5.38 obviously) | 17:22:31 |
| stigo | In reply to @adam:robins.wtf
it's currently failing with the error in https://github.com/NixOS/nixpkgs/issues/265488 Those messages do not seems to be perl specific, but come from some tooling inside slimserver | 17:22:26 |
| adamcstephens 🐝 | right. | 17:24:11 |
| adamcstephens 🐝 | In reply to @stigo:matrix.org
Those messages do not seems to be perl specific, but come from some tooling inside slimserver so to fix these failures, i've added perlPackages to buildInputs for the packages they provide in their pre-built vendoring. this causes version mismatch errors though. e.g.
machine # [ 5.747365] slimserver.pl[848]: Parser.pm and Expat.pm versions don't match at /nix/store/7s6gzxci51sf9ypabsn091pgdcxcaan5-perl5.36.1-slimserver-8.3.1/CPAN/XML/Parser.pm line 20.
| 17:37:22 |
| stigo | They still seem to load modules from the vendored CPAN directory according to the error message above | 17:40:43 |
| stigo | ...slimserver-8.3.1/CPAN/XML/Parser.pm line 20
| 17:41:15 |
| adamcstephens 🐝 | Ok, so I’ll remove any that we’re providing and see where that takes me | 17:41:17 |
