-
-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
Draft: discourse 3.1.0 backport to nixos-23.05 #262450
Conversation
The update.py script would error out if the spec had a trailing new line. The split that follows it would try to split it into two, and error out as it can't.
This was fixed globally in NixOS#240000 and thus is not needed here anymore. # Conflicts: # pkgs/servers/web-apps/discourse/default.nix
Suggested-by: talyz <kim.lindberger@gmail.com> # Conflicts: # pkgs/servers/web-apps/discourse/default.nix
I think #240000 also needs to be applied? |
Thanks @TheNeikos, I applied those commits and I think that gets me further. However now the tests freeze on the "building discourse-assets-3.1.0" step like this:
Is this another obvious problem, or is there a way to get more debugging output from these commands? |
You could try running with -L, that should output the log as it’s coming in. And otherwise, you’d have to read what the code does at the position it reports… |
The -L gives more info, still not sure what it means since there are various errors and yet it keeps running:
|
I'm giving up on this for now but will leave this up in case anyone else wants to try to pick it up from here. |
Closing since NixOS 23.05 is EOL. |
I'm trying to apply #250735 or #257294 to nixos-23.05 and keep running into this "Failed to open TCP connection to github.com:443" error when I run the tests:
I think that's related to removing the sass-embedded patch in 8e8a6ed. But I don't understand it or know what to do to get it to work. @TheNeikos @MTRNord do you have any suggestions?