From 44487cc7f1493836b7c8bcffb34eadea19d45751 Mon Sep 17 00:00:00 2001 From: Sean Chittenden Date: Sun, 10 Jul 2016 13:55:06 -0700 Subject: [PATCH] Build the Cgroup fingerprinter on only Linux. Change the logic from `!linux` to an empty build tag so that *if* another platform picks up Cgroups support they can add themselves to the necessary build tags for this fingerprinter and be on their way. Because this technology isn't inherently Linux-specific and isn't mutually exclusive of other resource isolation containers, resist the urge to rename the Cgroup fingerprinter to something generic like the ResourceContainerFingerprinter. --- client/fingerprint/cgroup.go | 2 ++ client/fingerprint/cgroup_default.go | 2 +- client/fingerprint/cgroup_test.go | 2 ++ 3 files changed, 5 insertions(+), 1 deletion(-) diff --git a/client/fingerprint/cgroup.go b/client/fingerprint/cgroup.go index c0ec7c674..1ec8d8793 100644 --- a/client/fingerprint/cgroup.go +++ b/client/fingerprint/cgroup.go @@ -1,3 +1,5 @@ +// +build linux + package fingerprint import ( diff --git a/client/fingerprint/cgroup_default.go b/client/fingerprint/cgroup_default.go index 43c55fd85..51e3bf58c 100644 --- a/client/fingerprint/cgroup_default.go +++ b/client/fingerprint/cgroup_default.go @@ -1,4 +1,4 @@ -// +build !linux +// +build package fingerprint diff --git a/client/fingerprint/cgroup_test.go b/client/fingerprint/cgroup_test.go index be5d251e0..f12379405 100644 --- a/client/fingerprint/cgroup_test.go +++ b/client/fingerprint/cgroup_test.go @@ -1,3 +1,5 @@ +// +build linux + package fingerprint import (