!jngDrdMgndWibPCYsR:nixos.org

Nix PHP

77 Members
A room for PHP developers running on Nix22 Servers

Load older messages

SenderMessageTime
25 Feb 2024
@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.nuetuI 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.nuetuSo 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.nuetu
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
5 Mar 2024
@drupol:matrix.orgPolI just had a meeting with Seldaek (Jordi B, the Composer's father).11:47:19
@drupol:matrix.orgPolHe gave me a better idea for the Composer builder.11:47:31
@drupol:matrix.orgPol

That idea is great because:

  1. It simplifies the composer builder
  2. It works with ALL the versions of composer including >= 2.7
11:48:11
@drupol:matrix.orgPolNow I have to work on the composer plugin, make a new release, update the complete documentation... 11:50:52
@drupol:matrix.orgPolSo, composer 2.7.1 is not yet to be included in Nix this week.11:51:06

Show newer messages

Back to Room ListRoom Version: 6