libstore: remove Goal::key

this was a debugging aid from day one that should not have any impact on
build semantics, and if it *does* have an impact on build semantics then
build semantics are seriously broken. keeping the order imposed by these
keys will be impossible once we let a real event loop schedule our jobs.

Change-Id: I5c313324e1f213ab6453d82f41ae5e59de809a5b

1 files changed, 1 insertions, 7 deletions

diff --git a/src/libstore/build/goal.hh b/src/libstore/build/goal.hh
index f808be160..4436e44b1 100644
--- a/src/libstore/build/goal.hh
+++ b/src/libstore/build/goal.hh
@@ -22,14 +22,10 @@ class Worker;
  */
 typedef std::shared_ptr<Goal> GoalPtr;
 
-struct CompareGoalPtrs {
-    bool operator() (const GoalPtr & a, const GoalPtr & b) const;
-};
-
 /**
  * Set of goals.
  */
-typedef std::set<GoalPtr, CompareGoalPtrs> Goals;
+typedef std::set<GoalPtr> Goals;
 
 /**
  * Used as a hint to the worker on how to schedule a particular goal. For example,
@@ -167,8 +163,6 @@ public:
         return name;
     }
 
-    virtual std::string key() = 0;
-
     virtual void cleanup() { }
 
     /**