summary refs log tree commit diff
path: root/pkgs/servers/consul/default.nix
diff options
context:
space:
mode:
authorNiklas Hambüchen <mail@nh2.me>2018-10-26 18:07:56 +0200
committerNiklas Hambüchen <mail@nh2.me>2018-11-03 18:39:46 +0100
commit2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0 (patch)
tree85d6d24513e1f5f696ac60448134c9cbfc99a31c /pkgs/servers/consul/default.nix
parentdf28b4b6e5a996a4971a0b32a98e654e4c44a54a (diff)
downloadnixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar.gz
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar.bz2
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar.lz
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar.xz
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.tar.zst
nixpkgs-2cb7f5fb1e502fe5502d37fce0a9b7a16ea24fd0.zip
consul: 0.9.3 -> 1.3.0.
Removes the old UI build tooling; it is no longer necessary
because as of 1.2.0 it's bundled into the server binary.
It doesn't even need to have JS built, because it's bundled into
the release commit's source tree (see #48714).

The UI is enabled by default, so the NixOS service is
updated to directly use `ui = webUi;` now.

Fixes #48714.
Fixes #44192.
Fixes #41243.
Fixes #35602.

Signed-off-by: Niklas Hambüchen <mail@nh2.me>
Diffstat (limited to 'pkgs/servers/consul/default.nix')
-rw-r--r--pkgs/servers/consul/default.nix19
1 files changed, 12 insertions, 7 deletions
diff --git a/pkgs/servers/consul/default.nix b/pkgs/servers/consul/default.nix
index 5cf19279100..d1dcd78667f 100644
--- a/pkgs/servers/consul/default.nix
+++ b/pkgs/servers/consul/default.nix
@@ -1,22 +1,27 @@
-{ stdenv, buildGoPackage, consul-ui, fetchFromGitHub }:
+{ stdenv, buildGoPackage, fetchFromGitHub }:
 
 buildGoPackage rec {
   name = "consul-${version}";
-  version = "0.9.3";
+  version = "1.3.0";
   rev = "v${version}";
 
   goPackagePath = "github.com/hashicorp/consul";
 
+  # Note: Currently only release tags are supported, because they have the Consul UI
+  # vendored. See
+  #   https://github.com/NixOS/nixpkgs/pull/48714#issuecomment-433454834
+  # If you want to use a non-release commit as `src`, you probably want to improve
+  # this derivation so that it can build the UI's JavaScript from source.
+  # See https://github.com/NixOS/nixpkgs/pull/49082 for something like that.
+  # Or, if you want to patch something that doesn't touch the UI, you may want
+  # to apply your changes as patches on top of a release commit.
   src = fetchFromGitHub {
     owner = "hashicorp";
     repo = "consul";
     inherit rev;
-    sha256 = "1176frp7kimpycsmz9wrbizf46jgxr8jq7hz5w4q1x90lswvrxv3";
+    sha256 = "1zv84snvrjm74w3v3rr27linsbxj00m73xd047sb78a4766xs2h0";
   };
 
-  # Keep consul.ui for backward compatability
-  passthru.ui = consul-ui;
-
   preBuild = ''
     buildFlagsArray+=("-ldflags" "-X github.com/hashicorp/consul/version.GitDescribe=v${version} -X github.com/hashicorp/consul/version.Version=${version} -X github.com/hashicorp/consul/version.VersionPrerelease=")
   '';
@@ -26,6 +31,6 @@ buildGoPackage rec {
     homepage = https://www.consul.io/;
     platforms = platforms.linux ++ platforms.darwin;
     license = licenses.mpl20;
-    maintainers = with maintainers; [ pradeepchhetri ];
+    maintainers = with maintainers; [ pradeepchhetri vdemeester nh2 ];
   };
 }