Use system univalue by default #7485

1. 
   luke-jr commented at 9:57 pm on February 8, 2016: member

   For future merging
2. Use system univalue by default f587fd51f1
3. 
   laanwj commented at 10:39 pm on February 8, 2016: member

   Previous discussion was in #7349, where it was decided not to automatically use an installed univalue in favor of the included one.
4. 
   luke-jr commented at 11:01 pm on February 8, 2016: member

   …for now.
5. 
   dcousens commented at 3:44 am on February 9, 2016: contributor

   Why would you opt for system default? Out of interest.
6. 
   luke-jr commented at 3:48 am on February 9, 2016: member

   Standard best practices. Libraries shouldn’t be bundled at all. We have a reasonable case for bundling consensus-critical ones like LevelDB and libsecp256k1, but Univalue is not consensus-critical at all.
7. laanwj added the label Build system on Feb 10, 2016
8. 
   laanwj commented at 3:54 pm on April 1, 2016: member

   We’re not going to do this (at least for the forseeable future), so closing for now.
9. laanwj closed this on Apr 1, 2016

   ---

10. DrahtBot locked this on Sep 8, 2021

Contributors
luke-jr laanwj dcousens

Labels
Build system