!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

76 Members
A room for PHP developers running on Nix22 Servers

Load older messages

SenderMessageTime
25 Feb 2024
@drupol:matrix.orgPol tgerbet: I've made the most recent bumps in the PR, removed the features we discussed, everything is clean now. 09:13:20
@drupol:matrix.orgPol * tgerbet: I've made new commits to bump to the most recent version in the PR, removed the features we discussed (it's actually not needed now), everything is clean now, ready for review. 09:13:50
@drupol:matrix.orgPolI will have limited availability on the computer next week, I finally take some days off, to Sardinia!09:14:24
@drupol:matrix.orgPolCiao a tutti09:14:27
@tgerbet:matrix.orgtgerbetEnjoy your days off 👋10:33:31
@patka_123:matrix.org@patka_123:matrix.org Have a nice holiday Pol ! 10:57:08
29 Feb 2024
@wjjunyor:matrix.orgwGuys I need some help ☹️20:04:53
@wjjunyor:matrix.orgw
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
@wjjunyor:matrix.orgw
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.orgPolCan you post your configuration somewhere?21:52:08
1 Mar 2024
@wjjunyor:matrix.orgw
In reply to @drupol:matrix.org
Can you post your configuration somewhere?
https://github.com/wjjunyor/pegasus 		02:52:19
@etu:failar.nu@etu:failar.nuI 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@etu:failar.nuSo it could be marked as broken in the meta attribute in the derivation to catch it earlier :)08:45:15
@tgerbet:matrix.orgtgerbet Hum maybe we should the require-dev section in the composer.json in composerInstallBuildHook when composerNoDev is true 08:55:26
@tgerbet:matrix.orgtgerbet * Hum maybe we should remove the require-dev section in the composer.json in composerInstallBuildHook when composerNoDev is true 08:55:50
@tgerbet:matrix.orgtgerbet

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:matrix.orgtgerbetRedacted or Malformed Event09:37:35
@tgerbet:matrix.orgtgerbet
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:matrix.orgtgerbet
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@patka_123:matrix.orgIn the PR description it says fixes the build, but on the bottom it says failed to build codesniffer in the nixpkgs-review output09:50:32
@tgerbet:matrix.orgtgerbet php-cs-fixer !== php-codesniffer :) 09:51:31
@patka_123:matrix.org@patka_123:matrix.orgJesus, ****** christ. I am soooo blind...09:52:07
@etu:failar.nu@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:matrix.orgtgerbet

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.orgPol @tgerbet can't test your PR but I can merge it if builds are passing  11:08:14
@wjjunyor:matrix.orgw
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
@wjjunyor:matrix.orgwTks 14:28:54
@wjjunyor:matrix.orgw 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
@wjjunyor:matrix.orgw * 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.orgPol 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

Show newer messages

Back to Room ListRoom Version: 6