| 1 Mar 2024 |
 w | In reply to @drupol:matrix.org
Can you post your configuration somewhere? https://github.com/wjjunyor/pegasus | 02:52:19 |
 @etu:failar.nu | I just wanted to report that the new php-codesniffer package doesn't build on PHP 8.1, some dependency says that it doesn't work there :) | 08:44:37 |
| @etu:failar.nu | So it could be marked as broken in the meta attribute in the derivation to catch it earlier :) | 08:45:15 |
 tgerbet | Hum maybe we should the require-dev section in the composer.json in composerInstallBuildHook when composerNoDev is true | 08:55:26 |
| tgerbet | * Hum maybe we should remove the require-dev section in the composer.json in composerInstallBuildHook when composerNoDev is true | 08:55:50 |
| tgerbet | Dropping the require-dev in the composer.json and setting packages-dev to [] in the composer.lock seems to work
Running nixpkgs-review on it and submitting it to review
| 09:21:25 |
| tgerbet | Redacted or Malformed Event | 09:37:35 |
| tgerbet | In reply to @wjjunyor:matrix.org
https://github.com/wjjunyor/pegasus Try with something looking like this
diff --git a/wordpress-configuration.nix b/wordpress-configuration.nix
index 04805bc..54eceb1 100644
--- a/wordpress-configuration.nix
+++ b/wordpress-configuration.nix
@@ -10,13 +10,13 @@
environment.systemPackages = with pkgs; [ wp4nix php ];
environment.variables.WP_VERSION = "6.4";
- nixpkgs.overlays = [ (self: super:
- wordpressPackages {
- src = builtins.fetchGit {
+ nixpkgs.overlays = [
+ (self: super: {
+ wordpressPackages = builtins.fetchGit {
url = "https://git.helsinki.tools/helsinki-systems/wp4nix";
ref = "master";
};
- }
- )];
+ })
+ ];
}
| 09:38:03 |
| tgerbet | In reply to @tgerbet:matrix.org
Dropping the require-dev in the composer.json and setting packages-dev to [] in the composer.lock seems to work
Running nixpkgs-review on it and submitting it to review
https://github.com/NixOS/nixpkgs/pull/292508 | 09:44:31 |
 @patka_123:matrix.org | In the PR description it says fixes the build, but on the bottom it says failed to build codesniffer in the nixpkgs-review output | 09:50:32 |
| tgerbet | php-cs-fixer !== php-codesniffer :) | 09:51:31 |
| @patka_123:matrix.org | Jesus, ****** christ. I am soooo blind... | 09:52:07 |
| @etu:failar.nu | In reply to @tgerbet:matrix.org
https://github.com/NixOS/nixpkgs/pull/292508 Oh, nice :) I've noticed this problem because I'm still using 8.1 :p | 10:07:15 |
| tgerbet | And fixing php81Packages.php-cs-fixer requires this patch and a patch like https://github.com/NixOS/nixpkgs/pull/292511
I will deal with it when it becomes possible
| 10:09:35 |
