| 25 Feb 2024 |
 @drupol:matrix.org | I will have limited availability on the computer next week, I finally take some days off, to Sardinia! | 09:14:24 |
| @drupol:matrix.org | Ciao a tutti | 09:14:27 |
 tgerbet | Enjoy your days off 👋 | 10:33:31 |
 @patka_123:matrix.org | Have a nice holiday Pol ! | 10:57:08 |
| 29 Feb 2024 |
 w | Guys I need some help ☹️ | 20:04:53 |
| w | In reply to @wjjunyor:matrix.org
Guys I need some help ☹️ error: syntax error, unexpected '=', expecting ')'
at /etc/nixos/wordpress-configuration.nix:14:23:
13| nixpkgs.overlays = [ (self: super:
14| wordpressPackages = builtins.fetchGit {
| ^
15| url = "https://git.helsinki.tools/helsinki-systems/wp4nix";
| 20:05:23 |
| w | In reply to @wjjunyor:matrix.org
Guys I need some help ☹️ I just doesn't get this overlay thing proprly | 20:06:50 |
| @drupol:matrix.org | Can you post your configuration somewhere? | 21:52:08 |
| 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 |
| @drupol:matrix.org | @tgerbet can't test your PR but I can merge it if builds are passing | 11:08:14 |
| w | In reply to @tgerbet:matrix.org
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";
};
- }
- )];
+ })
+ ];
}
Worked, now goin for the new error msgs | 14:28:47 |
| w | Tks | 14:28:54 |
| w | Pol: once you asked me to try the new phpcomposer - but I lost track of the mesage. Could you send it to me again? | 14:31:06 |
| w | * Pol: once you asked me to try the new php composer - but I lost track of the mesage. Could you send it to me again? | 14:46:51 |
| @drupol:matrix.org | Can't do that now on the phone. I'll be back tomorrow on my laptop . Ping me tomorrow and I'll see what I can do | 15:23:04 |
| 5 Mar 2024 |
| @drupol:matrix.org | I just had a meeting with Seldaek (Jordi B, the Composer's father). | 11:47:19 |
| @drupol:matrix.org | He gave me a better idea for the Composer builder. | 11:47:31 |
