00:14 < cfields> sipa: mmm, i've built all that before when working on toolchain stuff. If you're interested i can clean that up and push it somewhere

00:14 < cfields> (i've promised it to warren anyway)

00:17 < cfields> gmaxwell: that test doesn't need a separate libstdc++. Just some defines. There's a doc that describes the gcc magic that makes that work.

03:59 < gmaxwell> wtf importmulti gives you no way to just import a private key with a date and a bounded rescan horizon? :(

04:14 < sipa> gmaxwell: ?

04:16 < gmaxwell> importmulti can't actually import a bare private key, requires a redeemscript.

04:16 < sipa> or an address ,yes

04:19 < luke-jr> O.o

04:21 < sipa> because we shouldn't just infer what addresses credit us based on what keys

04:21 < sipa> i think that's the biggest mistake in the design of the wallet

04:21 < sipa> that ownership of addresses and possession of orivate keys are not separated things

05:48 < bitcoin-git> [bitcoin] da2x opened pull request #9690: Change 'Clear' button string to 'Reset' (master...patch-2) https://github.com/bitcoin/bitcoin/pull/9690

14:19 < luke-jr> qt/test/paymentservertests.cpp:65:6: warning: stack protector not protecting local variables: variable length buffer [-Wstack-protector]

14:19 < luke-jr> ^ are we trying to change these?

14:22 < gmaxwell> thats a dumb warning.

14:23 < gmaxwell> otoh we probably should avoid vararrays.

14:24 < gmaxwell> why is our configure scritpt ignoring CXXFLAGS ? :(

14:24 < gmaxwell> ah nevermind because it's failing.

16:53 < sipa> my bitcoind crashed due to a buffer overflow

16:59 < paveljanik> sipa, what commit hash?

16:59 < paveljanik> and how?

17:39 < bitcoin-git> [bitcoin] gmaxwell opened pull request #9691: Init ECC context for test_bitcoin_fuzzy. (master...fuzzy_initecc) https://github.com/bitcoin/bitcoin/pull/9691

18:10 < sipa> paveljanik: unclesr

18:51 < jl2012> with master, I find that all tests with setup_clean_chain = False failed on my MAC OS 10.12.2

18:52 < jl2012> same error: Error loading block database.

18:57 < sipa> hmm

20:08 < bitcoin-git> [bitcoin] sipa pushed 2 new commits to master: https://github.com/bitcoin/bitcoin/compare/7821db30e161...923dc447eaa8

20:08 < bitcoin-git> bitcoin/master 003cd60 Matt Corallo: Add missing mempool lock for CalculateMemPoolAncestors

20:08 < bitcoin-git> bitcoin/master 923dc44 Pieter Wuille: Merge #9578: Add missing mempool lock for CalculateMemPoolAncestors...

20:08 < bitcoin-git> [bitcoin] sipa closed pull request #9578: Add missing mempool lock for CalculateMemPoolAncestors (master...2017-01-fix-missing-wallet-mempool-lock) https://github.com/bitcoin/bitcoin/pull/9578

21:39 < MarcoFalke_> jl2012: Try deleting the qa cache dir

23:26 < luke-jr> shoot.

23:26 < luke-jr> wumpus: we should have taken advantage of the named arguments to switch amounts to satoshis..

23:27 < * luke-jr> wonders if we can get broad agreement that this would be okay to throw in now, considering if we don't, we'll be stuck supporting BTC values longer..