From fd54a946ca6d42b53df8d9040263fa43f6e426e9 Mon Sep 17 00:00:00 2001
From: Profpatsch <mail@profpatsch.de>
Date: Mon, 2 Apr 2018 21:00:47 +0200
Subject: lib/debug: deprecate `addErrorContextToAttrs`
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The function isn’t used anywhere and `addErrorContext` is an undocumented
builtin.
The builtin is explicitely qualified at its two uses in the module system.
---
 lib/debug.nix | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'lib/debug.nix')

diff --git a/lib/debug.nix b/lib/debug.nix
index 16e8710168c..a792806fa79 100644
--- a/lib/debug.nix
+++ b/lib/debug.nix
@@ -11,10 +11,6 @@ in
 
 rec {
 
-  inherit (builtins) addErrorContext;
-
-  addErrorContextToAttrs = lib.mapAttrs (a: v: lib.addErrorContext "while evaluating ${a}" v);
-
   traceIf = p: msg: x: if p then trace msg x else x;
 
   traceValFn = f: x: trace (f x) x;
@@ -98,6 +94,12 @@ rec {
           + "and will be removed in the next release." )
     (if c x then true else traceSeq (showVal x) false);
 
+
+  addErrorContextToAttrs = attrs:
+    trace ( "Warning: `addErrorContextToAttrs` is deprecated "
+          + "and will be removed in the next release." )
+    (lib.mapAttrs (a: v: lib.addErrorContext "while evaluating ${a}" v) attrs);
+
   /* Evaluate a set of tests.  A test is an attribute set {expr,
      expected}, denoting an expression and its expected result.  The
      result is a list of failed tests, each represented as {name,
-- 
cgit 1.4.1