refactor(blockservice, merkledag, namesys) deprecate u.ErrNotFound

blockservice/blockservice.go

@@ -1,6 +1,7 @@
 package blockservice
 
 import (
+	"errors"
 	"fmt"
 
 	context "github.com/jbenet/go-ipfs/Godeps/_workspace/src/code.google.com/p/go.net/context"
@@ -13,6 +14,7 @@ import (
 )
 
 var log = u.Logger("blockservice")
+var ErrNotFound = errors.New("blockservice: key not found")
 
 // BlockService is a block datastore.
 // It uses an internal `datastore.Datastore` instance to store values.
@@ -73,7 +75,7 @@ func (s *BlockService) GetBlock(ctx context.Context, k u.Key) (*blocks.Block, er
 		return blk, nil
 	} else {
 		log.Debug("Blockservice GetBlock: Not found.")
-		return nil, u.ErrNotFound
+		return nil, ErrNotFound
 	}
 }
 

merkledag/merkledag.go

@@ -103,7 +103,7 @@ func (n *Node) RemoveNodeLink(name string) error {
 			return nil
 		}
 	}
-	return u.ErrNotFound
+	return fmt.Errorf("merkledag: %s not found", name)
 }
 
 // Copy returns a copy of the node.

namesys/dns.go

@@ -1,13 +1,12 @@
 package namesys
 
 import (
+	"fmt"
 	"net"
 
 	b58 "github.com/jbenet/go-ipfs/Godeps/_workspace/src/github.com/jbenet/go-base58"
 	isd "github.com/jbenet/go-ipfs/Godeps/_workspace/src/github.com/jbenet/go-is-domain"
 	mh "github.com/jbenet/go-ipfs/Godeps/_workspace/src/github.com/jbenet/go-multihash"
-
-	u "github.com/jbenet/go-ipfs/util"
 )
 
 // DNSResolver implements a Resolver on DNS domains
@@ -44,5 +43,5 @@ func (r *DNSResolver) Resolve(name string) (string, error) {
 		return t, nil
 	}
 
-	return "", u.ErrNotFound
+	return "", fmt.Errorf("namesys: %v not found", name)
 }