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
tengine: init init at v2.2.2 #44146
tengine: init init at v2.2.2 #44146
Conversation
, gd, geoip | ||
, withDebug ? false | ||
, withMail ? false | ||
, withIPv6 ? false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why disable IPv6? I believe that any server application should support IPv6 by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, fixed.
pkgs/top-level/all-packages.nix
Outdated
@@ -19954,6 +19954,11 @@ with pkgs; | |||
|
|||
teeworlds = callPackage ../games/teeworlds { }; | |||
|
|||
tengine = callPackage ../servers/http/tengine { | |||
stdenv = overrideCC stdenv gcc6; | |||
modules = [ nginxModules.rtmp nginxModules.dav nginxModules.moreheaders nginxModules.modsecurity-nginx]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
with nginxModules; [ rtmp dav ... ]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tengine support build with nginx modules
pkgs/top-level/all-packages.nix
Outdated
@@ -19954,6 +19954,11 @@ with pkgs; | |||
|
|||
teeworlds = callPackage ../games/teeworlds { }; | |||
|
|||
tengine = callPackage ../servers/http/tengine { | |||
stdenv = overrideCC stdenv gcc6; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this necessary? Did you try to build with default gcc7 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Error build with gcc7.
@Izorkin i added a commit to fix the build with gcc7 so we do not need to use gcc6. Please test. |
Thanks. Works. |
@GrahamcOfBorg build tengine |
Success on x86_64-linux (full log) Attempted: tengine Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: tengine Partial log (click to expand)
|
Success on x86_64-darwin (full log) Attempted: tengine Partial log (click to expand)
|
aarch64 failure can be fixed later by someone familiar with the platform. |
Motivation for this change
Add web server tengine.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)