-
-
Notifications
You must be signed in to change notification settings - Fork 17.7k
n8n: 1.123.5 -> 2.1.4 #472698
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
n8n: 1.123.5 -> 2.1.4 #472698
Conversation
|
NixOS test passes locally (it only fails in CI because it's unfree). I'd like if someone else could test this on their setup, feel free to ask how to do it if you're unsure. Since I'm not actually using this package (just willing to fix build failures), I don't want to merge a major version bump. |
|
As a data point I tested this PR on my local install and it appears to work fine. Thanks for updating it. |
|
I have noticed that the JS code tasks no longer seem to be working when using the internal runner. I'm not sure if it's upstream related yet though, as I do see some upstream changes around it. |
|
Maybe related upstream issue: n8n-io/n8n#23553 |
|
This patch got the JS runners working: --- package.nix 2025-12-23 22:42:57.740471347 -0400
+++ package.nix 2025-12-23 23:07:16.909240631 -0400
@@ -51,6 +51,7 @@
libkrb5
libmongocrypt
libpq
+ python3
];
buildPhase = ''
@@ -90,7 +91,9 @@
mv {packages,node_modules} $out/lib/n8n
makeWrapper $out/lib/n8n/packages/cli/bin/n8n $out/bin/n8n \
- --set N8N_RELEASE_TYPE "stable"
+ --set N8N_RELEASE_TYPE "stable" \
+ --prefix PATH : ${lib.makeBinPath [ nodejs python3 ]}
+
runHook postInstall
'';The python running still has a warning: |
|
@KenMacD, I think it's because the runner expects We probably need to install |
ed55a92 to
f548458
Compare
|
Thanks for all of your research! For now I added nodejs and python. |
f548458 to
a2f4c1a
Compare
a2f4c1a to
b25cb54
Compare
|
Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript. meta.description for n8n is: Free and source-available fair-code licensed workflow automation tool meta.homepage for n8n is: https://n8n.io meta.changelog for n8n is: https://github.com/n8n-io/n8n/releases/tag/[email protected] Updates performed
To inspect upstream changesImpactChecks done
Caution A test defined in
Rebuild report (if merged into master) (click to expand)Instructions to test this update (click to expand)Either download from the cache: (The nixpkgs-update cache is only trusted for this store-path realization.) Or, build yourself: Or: After you've downloaded or built it, look at the files and if there are any, run the binaries: Pre-merge build resultsWe have automatically built all packages that will get rebuilt due to This gives evidence on whether the upgrade will break dependent packages.
|
Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for n8n is: Free and source-available fair-code licensed workflow automation tool
meta.homepage for n8n is: https://n8n.io
meta.changelog for n8n is: https://github.com/n8n-io/n8n/releases/tag/[email protected]
Updates performed
To inspect upstream changes
Impact
Checks done
Caution
A test defined in
passthru.testsdid not pass.Rebuild report (if merged into master) (click to expand)
Instructions to test this update (click to expand)
Either download from the cache:
(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the
trusted-userslist or you can usesudosince root is effectively trusted.Or, build yourself:
Or:
After you've downloaded or built it, look at the files and if there are any, run the binaries:
Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-reviewresultGenerated using
nixpkgs-review.Command:
nixpkgs-review --extra-nixpkgs-config '{ allowBroken = false; }'Commit:
f1d8bc76db24483cdc7b6d53bb459a915f86a971x86_64-linux✅ 1 package built:
Maintainer pings
cc @gepbird @AdrienLemaire for testing.
Tip
As a maintainer, if your package is located under
pkgs/by-name/*, you can comment@NixOS/nixpkgs-merge-bot mergeto automatically merge this update using thenixpkgs-merge-bot.Add a 👍 reaction to pull requests you find important.