Pull request to fix 'Makefile:127: recipe for target 'all' failed' (d…

…ashpay#11) * add 'mkdir build' in build instructions in README.md * change 'std::set<const BLSPublicKey>' => 'std::set<BLSPublicKey>' to fix std:set overloaded error
kwvg · Aug 20, 2018 · 5ad4919 · 5ad4919
1 parent 5e43d3c
commit 5ad4919
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/README.md b/README.md
@@ -198,6 +198,7 @@ Cmake and a c++ compiler are required for building.
 ```bash
 git submodule init
 git submodule update
+mkdir build
 cd build
 cmake ../
 cmake --build . -- -j 6

diff --git a/src/aggregationinfo.cpp b/src/aggregationinfo.cpp
@@ -286,7 +286,7 @@ void AggregationInfo::SortIntoVectors(std::vector<uint8_t*> &ms,
 // Simple merging, no exponentiation is performed
 AggregationInfo AggregationInfo::SimpleMergeInfos(
         std::vector<AggregationInfo> const &infos) {
-    std::set<const BLSPublicKey> pubKeysDedup;
+    std::set<BLSPublicKey> pubKeysDedup;
 
     AggregationTree newTree;
     for (const AggregationInfo &info : infos) {