Skip to content
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

Hiawatha: Init at 10.5 #22916

Closed
wants to merge 5 commits into from
Closed

Hiawatha: Init at 10.5 #22916

wants to merge 5 commits into from

Conversation

ndowens
Copy link
Contributor

@ndowens ndowens commented Feb 17, 2017

Motivation for this change

Add hiawatha to repo

Things done
  • [ x] Tested using sandboxing
    (nix.useSandbox on NixOS,
    or option build-use-sandbox in nix.conf
    on non-NixOS)
  • Built on platform(s)
    • [ x] NixOS
    • macOS
    • Linux
  • [ x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • [x ] Tested execution of all binary files (usually in ./result/bin/)
  • [ x] Fits CONTRIBUTING.md.

Hiawatha: Fixed missing pkg tag

	new file:   pkgs/servers/http/hiawatha/default.nix

hiawatha: 10.5 module

	modified:   nixos/modules/misc/ids.nix
	modified:   pkgs/servers/http/hiawatha/default.nix

	modified:   nixos/modules/misc/ids.nix
	modified:   nixos/modules/module-list.nix
	new file:   nixos/modules/services/web-servers/hiawatha.nix
	new file:   pkgs/servers/http/hiawatha/default.nix
	modified:   pkgs/top-level/all-packages.nix
@@ -13738,6 +13738,8 @@ with pkgs;

hexedit = callPackage ../applications/editors/hexedit { };

hiawatha = callPackage ../servers/http/hiawatha { };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will once I get home

meta = with stdenv.lib; {
description = "An advanced and secure webserver";
license = licenses.gpl2;
homepage = "https://www.hiawatha-webserver.org";
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you add yourself as a maintainer? You need to include your details in /lib/maintainers.nix too.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, I see you're already in maintainers.nix :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I am in the list already 😀 Surprised I left out that line.

@ndowens
Copy link
Contributor Author

ndowens commented Feb 21, 2017

Gonna close it and open another; messed up with git

@ndowens ndowens closed this Feb 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants