12017-07-28T00:01:21 *** coredump_ has joined #bitcoin-core-dev
22017-07-28T00:03:16 *** Murch has quit IRC
32017-07-28T00:13:24 *** Murch has joined #bitcoin-core-dev
42017-07-28T00:15:16 *** jamesob has joined #bitcoin-core-dev
52017-07-28T00:21:49 *** praxeology has quit IRC
62017-07-28T00:22:36 *** sam_c has quit IRC
72017-07-28T00:27:14 *** sam_c has joined #bitcoin-core-dev
82017-07-28T00:30:20 *** sam_c has quit IRC
92017-07-28T00:41:07 *** sam_c has joined #bitcoin-core-dev
102017-07-28T00:46:21 *** AaronvanW has quit IRC
112017-07-28T00:51:45 *** Ylbam has quit IRC
122017-07-28T01:09:20 *** chjj has quit IRC
132017-07-28T01:13:23 <bitcoin-git> [bitcoin] gmaxwell opened pull request #10945: Update defaultAssumeValid according to release-process.md. (master...201707-update-assumevalid) https://github.com/bitcoin/bitcoin/pull/10945
142017-07-28T01:22:00 *** Murch has quit IRC
152017-07-28T01:24:18 *** Murch has joined #bitcoin-core-dev
162017-07-28T01:24:47 *** Murch has quit IRC
172017-07-28T01:36:34 *** dabura667 has joined #bitcoin-core-dev
182017-07-28T01:41:28 *** coredump_ has quit IRC
192017-07-28T01:50:07 *** jtimon has quit IRC
202017-07-28T01:52:28 *** Orion3k has quit IRC
212017-07-28T01:57:36 *** snkey has quit IRC
222017-07-28T02:04:59 *** jamesob has quit IRC
232017-07-28T02:06:11 *** ivan has joined #bitcoin-core-dev
242017-07-28T02:07:09 *** Orion3k has joined #bitcoin-core-dev
252017-07-28T02:15:33 *** snq has joined #bitcoin-core-dev
262017-07-28T02:21:16 *** snq has quit IRC
272017-07-28T02:21:52 *** PRab has quit IRC
282017-07-28T02:30:47 *** d_t has joined #bitcoin-core-dev
292017-07-28T02:32:47 *** chjj has joined #bitcoin-core-dev
302017-07-28T02:42:31 *** snq has joined #bitcoin-core-dev
312017-07-28T02:45:49 *** praxeology has joined #bitcoin-core-dev
322017-07-28T02:46:50 *** str4d has quit IRC
332017-07-28T03:09:27 *** praxeology has quit IRC
342017-07-28T03:13:45 *** praxeology has joined #bitcoin-core-dev
352017-07-28T03:14:09 *** praxeology has joined #bitcoin-core-dev
362017-07-28T03:23:02 *** Orion3k has quit IRC
372017-07-28T03:26:23 *** EagleTM has joined #bitcoin-core-dev
382017-07-28T03:27:40 *** Eagle[TM] has quit IRC
392017-07-28T03:39:38 *** snq has quit IRC
402017-07-28T03:41:10 *** jamesob has joined #bitcoin-core-dev
412017-07-28T03:45:08 *** veleiro has joined #bitcoin-core-dev
422017-07-28T03:49:46 *** veleiro has quit IRC
432017-07-28T03:58:26 *** veleiro has joined #bitcoin-core-dev
442017-07-28T04:04:12 *** goatpig has quit IRC
452017-07-28T04:14:47 *** justan0theruser has joined #bitcoin-core-dev
462017-07-28T04:16:59 *** justanotheruser has quit IRC
472017-07-28T04:21:45 *** jeep-ss has quit IRC
482017-07-28T04:32:50 *** veleiro has quit IRC
492017-07-28T04:52:10 *** Dyaheon has quit IRC
502017-07-28T04:54:35 *** Dyaheon has joined #bitcoin-core-dev
512017-07-28T05:07:55 *** HeyRoll has joined #bitcoin-core-dev
522017-07-28T05:09:50 *** Chris_Stewart_5 has joined #bitcoin-core-dev
532017-07-28T05:19:02 <bitcoin-git> [bitcoin] sipa opened pull request #10946: Add chainwork to getchaintxstats (master...20170727_chainworkstats) https://github.com/bitcoin/bitcoin/pull/10946
542017-07-28T05:29:50 *** Chris_Stewart_5 has quit IRC
552017-07-28T05:30:32 *** snq has joined #bitcoin-core-dev
562017-07-28T05:32:14 *** snq has joined #bitcoin-core-dev
572017-07-28T05:33:22 <gmaxwell> sipa: should I add updating ChainTxData to the PR I have for the assumevalid?
582017-07-28T05:38:52 *** snq has quit IRC
592017-07-28T05:40:20 *** snq has joined #bitcoin-core-dev
602017-07-28T05:40:21 *** d_t has quit IRC
612017-07-28T05:42:01 *** chjj has quit IRC
622017-07-28T05:46:36 *** d_t has joined #bitcoin-core-dev
632017-07-28T05:50:24 <sipa> gmaxwell: ?
642017-07-28T05:51:01 <gmaxwell> sipa: I have a PR open to update the assumevalid, I could update chainTxData too, I was reminded by your above PR.
652017-07-28T05:55:54 <wumpus> profall: it's really useful for giving whole UTXOs away
662017-07-28T05:56:33 <wumpus> (or alternatively, sending them to some other wallet of yourself, but the idea is to not have to add anything extra for fee, resulting in change)
672017-07-28T05:57:04 <wumpus> this used to be a process of bisection
682017-07-28T05:59:10 <sipa> gmaxwell: ah, i see!
692017-07-28T05:59:22 <sipa> i just wrote thst PR to verify your nMinimumChainWork
702017-07-28T05:59:55 <gmaxwell> sipa: yes, I responded to that PR to point out that the info is already in getblockchaininfo, the release process docs tell you how to verify it. :)
712017-07-28T06:00:59 *** justan0theruser has quit IRC
722017-07-28T06:01:22 *** justanotheruser has joined #bitcoin-core-dev
732017-07-28T06:07:40 <wumpus> "The fee rate (in %s/kB) used to discard change (to fee) if it would be dust at this fee rate (default: %s) Note: We will always discard up to the dust relay fee and a discard fee above that is limited by the longest target fee estimate"
742017-07-28T06:07:44 <wumpus> this confuses translators
752017-07-28T06:08:28 <wumpus> - "the dust relay fee" -- this sounds like a fee applied to relaying dust; is it?
762017-07-28T06:08:37 <wumpus> - "a discard fee" -- this sounds like a fee applied when discarding something, which is probably isn't. Does it relate to the fee rate which seems to be the main topic of this string?
772017-07-28T06:08:49 <wumpus> - "discard change (to fee)" -- perhaps the explanation in the parentheses could be more precise. Does this suggest that the change is added to the fee in case it is otherwise considered dust? Then perhaps the parentheses may say something like "(by adding change to the fee instead)"?
782017-07-28T06:09:01 <wumpus> - "longest target fee estimate" -- which of the other words does "longest" relate to here? Could this be rewritten as "longest estimete of the target fee", or perhaps "fee estimate of the longest target", or something else?
792017-07-28T06:10:12 <wumpus> is this a user-servicable option at all? if not, might I propose moving it to debug-help wholesale and not translating it?
802017-07-28T06:12:36 *** HeyRoll has quit IRC
812017-07-28T06:15:28 <wumpus> it uses too many internal technical terms that appear nowhere else, which indicates that someone that doesn't know internal wallet logic has no business setting it
822017-07-28T06:16:36 <sipa> longest refers to long term estimate, i believe
832017-07-28T06:16:55 <sipa> i would agree with making it an expert/debug option
842017-07-28T06:19:54 <gmaxwell> I don't think the target thing should really be a user servisable option.
852017-07-28T06:20:02 <gmaxwell> The discard threshold is.
862017-07-28T06:20:51 <wumpus> this is about 'discardfee=<amt>'
872017-07-28T06:20:57 <wumpus> well if so, it needs better documentation
882017-07-28T06:22:26 <wumpus> this is from a person that has been translating bitcoin strings to Danish for a long time, I think it's a good indication that if he doesn't understand any of it, no other non-dev will
892017-07-28T06:22:42 <gmaxwell> Right!
902017-07-28T06:25:03 <wumpus> this again makes me doubt whether it makes sense to translate option helps at all. It's a bit like translating obscure error messages - having a variant of the technical terms in every language doesn't make it easier to look for
912017-07-28T06:25:22 <wumpus> especially if the translators don't understand it and just do a literal translation
922017-07-28T06:28:37 *** Ylbam has joined #bitcoin-core-dev
932017-07-28T06:37:20 *** jamesob has quit IRC
942017-07-28T06:41:09 *** J-wolf has joined #bitcoin-core-dev
952017-07-28T06:47:02 *** marcoagner has quit IRC
962017-07-28T06:50:30 *** J-wolf has quit IRC
972017-07-28T07:05:35 *** J-wolf has joined #bitcoin-core-dev
982017-07-28T07:26:20 *** Dyaheon has quit IRC
992017-07-28T07:29:15 *** Dyaheon has joined #bitcoin-core-dev
1002017-07-28T07:35:50 *** EagleTM has quit IRC
1012017-07-28T07:46:55 *** timothy has joined #bitcoin-core-dev
1022017-07-28T07:54:44 *** Yogaqueef has joined #bitcoin-core-dev
1032017-07-28T08:12:52 *** miknotauro has joined #bitcoin-core-dev
1042017-07-28T08:18:37 *** justanotheruser has quit IRC
1052017-07-28T08:22:29 *** Blisscan has joined #bitcoin-core-dev
1062017-07-28T08:26:36 *** Blisscan has quit IRC
1072017-07-28T08:31:35 *** vicenteH has joined #bitcoin-core-dev
1082017-07-28T08:40:19 *** cysm has quit IRC
1092017-07-28T08:48:08 *** cysm has joined #bitcoin-core-dev
1102017-07-28T08:51:05 *** sanada has quit IRC
1112017-07-28T08:52:45 *** dabura667 has quit IRC
1122017-07-28T09:10:49 *** miknotauro has quit IRC
1132017-07-28T09:21:57 *** jtimon has joined #bitcoin-core-dev
1142017-07-28T09:27:03 <bitcoin-git> [bitcoin] NicolasDorier opened pull request #10947: [Wallet] Bare segwit scriptPubKey are not considered change by the wallet (master...importaddresssegwit) https://github.com/bitcoin/bitcoin/pull/10947
1152017-07-28T09:33:19 *** Dyaheon has quit IRC
1162017-07-28T09:34:25 *** Dyaheon has joined #bitcoin-core-dev
1172017-07-28T09:42:05 *** Drabiv has joined #bitcoin-core-dev
1182017-07-28T10:00:03 *** J-wolf has quit IRC
1192017-07-28T10:00:37 *** J-wolf has joined #bitcoin-core-dev
1202017-07-28T10:01:05 *** ill has quit IRC
1212017-07-28T10:03:58 *** BashCo_ has joined #bitcoin-core-dev
1222017-07-28T10:05:14 *** J-wolf has quit IRC
1232017-07-28T10:05:57 *** BashCo has quit IRC
1242017-07-28T10:10:37 *** promag has joined #bitcoin-core-dev
1252017-07-28T10:12:06 *** jtimon has quit IRC
1262017-07-28T10:26:39 *** riemann has joined #bitcoin-core-dev
1272017-07-28T10:28:00 <promag> wumpus: I think #10885 is ready
1282017-07-28T10:28:02 <gribble> https://github.com/bitcoin/bitcoin/issues/10885 | Reject invalid wallets by promag · Pull Request #10885 · bitcoin/bitcoin · GitHub
1292017-07-28T10:34:57 *** SopaXorzTaker has joined #bitcoin-core-dev
1302017-07-28T10:39:58 <wumpus> promag: thanks
1312017-07-28T10:43:39 <bitcoin-git> [bitcoin] laanwj opened pull request #10948: p2p: Hardcoded seeds update pre-0.15 branch (master...2017_07_seeds_update) https://github.com/bitcoin/bitcoin/pull/10948
1322017-07-28T10:51:55 <sipa> wumpus: we have a number of obvious but invasive scripted refactors... what is your opinion on what a good time for those is?
1332017-07-28T10:52:45 <sipa> i was thinking (assuming they're acceptable, of course), that right before branching may be advisable, so as to not hurt backportability of fixes
1342017-07-28T10:53:45 <sipa> that does go against the idea of not having invasive changes after freeze
1352017-07-28T10:54:14 <wumpus> sipa: well as long as they're not features, straightforward, and low-risk, it could be ok
1362017-07-28T10:55:02 <gmaxwell> My view is that getting those kinds of safe refactors in before branching is good because it helps backports.
1372017-07-28T10:55:34 <bitcoin-git> [bitcoin] laanwj pushed 4 new commits to master: https://github.com/bitcoin/bitcoin/compare/0b11a0784875...70888a39c43a
1382017-07-28T10:55:35 <bitcoin-git> bitcoin/master 3ef77a0 João Barbosa: Reject duplicate wallet filenames
1392017-07-28T10:55:35 <bitcoin-git> bitcoin/master a6da027 João Barbosa: Reject invalid wallet files
1402017-07-28T10:55:36 <bitcoin-git> bitcoin/master d84e78e John Newbery: [wallet] Specify wallet name in wallet loading errors
1412017-07-28T10:56:09 <bitcoin-git> [bitcoin] laanwj closed pull request #10885: Reject invalid wallets (master...2017-07-prevent-duplicate-wallets) https://github.com/bitcoin/bitcoin/pull/10885
1422017-07-28T10:56:41 <wumpus> well it seems to be quite quiet this time before branch, so it seems we could get around to that
1432017-07-28T10:56:50 <wumpus> which ones is this about?
1442017-07-28T11:10:49 *** riemann has quit IRC
1452017-07-28T11:17:22 <sipa> wumpus: #10483 at elast
1462017-07-28T11:17:24 <gribble> https://github.com/bitcoin/bitcoin/issues/10483 | scripted-diff: Use the C++11 keyword nullptr to denote the pointer literal instead of the macro NULL by practicalswift · Pull Request #10483 · bitcoin/bitcoin · GitHub
1472017-07-28T11:18:10 <sipa> #10607
1482017-07-28T11:18:11 <gribble> https://github.com/bitcoin/bitcoin/issues/10607 | scripted-diff: stop using the gArgs wrappers by benma · Pull Request #10607 · bitcoin/bitcoin · GitHub
1492017-07-28T11:19:05 <sipa> i assumed there were more, but they're not scripted
1502017-07-28T11:20:30 *** riemann has joined #bitcoin-core-dev
1512017-07-28T11:26:27 <wumpus> tagged them for 0.15
1522017-07-28T11:32:19 *** promag has quit IRC
1532017-07-28T11:38:54 *** rafalcpp_ has quit IRC
1542017-07-28T11:39:15 *** rafalcpp has joined #bitcoin-core-dev
1552017-07-28T11:51:32 *** instagibbs has quit IRC
1562017-07-28T11:54:14 *** instagibbs has joined #bitcoin-core-dev
1572017-07-28T12:15:55 *** goatpig has joined #bitcoin-core-dev
1582017-07-28T12:27:35 *** phog has quit IRC
1592017-07-28T12:28:52 *** riemann has quit IRC
1602017-07-28T12:46:18 *** [b__b] has quit IRC
1612017-07-28T12:46:41 *** jannes has joined #bitcoin-core-dev
1622017-07-28T13:01:43 *** Mordan has joined #bitcoin-core-dev
1632017-07-28T13:11:20 *** justanotheruser has joined #bitcoin-core-dev
1642017-07-28T13:14:39 *** Victor_sueca has joined #bitcoin-core-dev
1652017-07-28T13:14:58 *** Victorsueca has quit IRC
1662017-07-28T13:15:00 *** Victor_sueca is now known as Victorsueca
1672017-07-28T13:15:24 *** jamesob_ has quit IRC
1682017-07-28T13:15:58 *** jamesob has joined #bitcoin-core-dev
1692017-07-28T13:21:51 *** justanotheruser has quit IRC
1702017-07-28T13:22:15 *** justanotheruser has joined #bitcoin-core-dev
1712017-07-28T13:24:17 <morcos> wumpus: re: -discardfee help
1722017-07-28T13:24:31 <morcos> I agree it is a slightly involved concept
1732017-07-28T13:24:59 <morcos> I originally had a shorter message but ryanofsky asked me to include the information about its interaction with dust and estimates
1742017-07-28T13:25:32 <morcos> It's almost like that more technical information should only be included if you have showDebug
1752017-07-28T13:25:50 *** riemann has joined #bitcoin-core-dev
1762017-07-28T13:26:08 *** jeep-ss has joined #bitcoin-core-dev
1772017-07-28T13:26:25 <wumpus> yes, the problem is not so much that there is extra information, but that it refers to all kinds of terms that are unknown to users
1782017-07-28T13:26:26 <morcos> I kind of agree with gmaxwell that this is kind of a user serviceable option. It's your tolerance for throwing money away to fee rather than creating a small output
1792017-07-28T13:26:45 <wumpus> maybe that's a better help message :)
1802017-07-28T13:27:02 <wumpus> those are terms everyone can relate to :-)
1812017-07-28T13:27:42 <morcos> The only issue is where the correct place is to document how it interacts with these other things... but perhaps the code is sufficient
1822017-07-28T13:27:48 <wumpus> but yeah the question is when someone would set it
1832017-07-28T13:28:10 <wumpus> heh it's almost as if having only option documentation isn't enough anymore
1842017-07-28T13:28:40 <wumpus> not that we have any kind of other documentation where this would be a good place for :/
1852017-07-28T13:28:43 <morcos> I'm guessing they would set it if they wonder why they are occasionally paying a higher fee and they are really trying to avoid that
1862017-07-28T13:28:55 <morcos> You know.. maybe we should just move it to be a debug option
1872017-07-28T13:29:32 <morcos> Well I'm wiling to change it whichever way we think is best
1882017-07-28T13:30:40 <wumpus> as I said, another option would be to never translate command line option help at all, I don't think descriptions like this will be improved by translation in most cases
1892017-07-28T13:31:13 <morcos> Yes or could we just not have them translate the "Note:" part
1902017-07-28T13:31:29 <wumpus> translation would be ideally used for things like UI messages that use normal english, and a set of defined common terms for the application
1912017-07-28T13:31:47 <wumpus> there's not really any win in translating concepts we've made up ourselves
1922017-07-28T13:32:29 <wumpus> yeah
1932017-07-28T13:33:02 <morcos> The fee rate (in %s/kB) that indicates your tolerance used for discarding change by adding it to the fee. (default: %s) Note: An output is discarded if it is dust at this rate, but we will always discard up to the dust relay fee and a discard fee above that is limited by the longest target fee estimate
1942017-07-28T13:33:06 <wumpus> what is funny, in practice the command-line options are only translated when you call --help through bitcoin-qt, not when using bitcoind
1952017-07-28T13:33:17 <wumpus> (the latter doesn't contain the translation data)
1962017-07-28T13:33:28 <morcos> s/used//
1972017-07-28T13:35:03 <morcos> 1.) Change to above 2) Change to above and delete note. A) Move to Debug option
1982017-07-28T13:35:05 <wumpus> morcos: better!
1992017-07-28T13:35:12 <morcos> I'm fine with 1,2,1A,2A or nothing
2002017-07-28T13:38:12 <wumpus> I'm fine with all as well, but 1 seems best
2012017-07-28T13:38:41 <morcos> will do
2022017-07-28T13:44:39 <sipa> morcos: it is user servicable? things will may go badly though if you set if sufficiently different from the rest of the network, no?
2032017-07-28T13:47:46 <morcos> sipa: how do you mean?
2042017-07-28T13:48:14 <morcos> no, it is only used to increase what would be already thrown away by the dustrelayfee
2052017-07-28T13:48:43 <morcos> which is also an option but i agree a non-user-servicable one that should not be changed
2062017-07-28T13:49:53 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2072017-07-28T13:50:40 <morcos> actually not sure if we are there yet, but ideally once we have this option, then it should be fine to lower the dustrelayfee to accomplish greg's goal of no longer having that concept, as long as your discardfee doesn't go below the OLD dustrelayfee.
2082017-07-28T13:51:02 <morcos> but there are some slight other issues there such as with fee estimation
2092017-07-28T13:53:51 *** Guyver2 has joined #bitcoin-core-dev
2102017-07-28T13:55:46 <bitcoin-git> [bitcoin] morcos opened pull request #10949: Clarify help message for -discardfee (master...betterhelp) https://github.com/bitcoin/bitcoin/pull/10949
2112017-07-28T14:04:13 *** riemann has quit IRC
2122017-07-28T14:10:22 *** cheese_ has joined #bitcoin-core-dev
2132017-07-28T14:10:30 *** J-wolf has joined #bitcoin-core-dev
2142017-07-28T14:12:47 *** dgenr8 has quit IRC
2152017-07-28T14:16:38 *** veleiro has joined #bitcoin-core-dev
2162017-07-28T14:16:50 *** dgenr8 has joined #bitcoin-core-dev
2172017-07-28T14:19:12 *** Deacyded has joined #bitcoin-core-dev
2182017-07-28T14:21:03 *** AaronvanW has joined #bitcoin-core-dev
2192017-07-28T14:22:01 *** Deacydal has quit IRC
2202017-07-28T14:34:06 *** veleiro has quit IRC
2212017-07-28T14:34:17 *** jeep-ss has quit IRC
2222017-07-28T14:37:47 *** Dyaheon has quit IRC
2232017-07-28T14:40:44 *** J-wolf has quit IRC
2242017-07-28T14:41:44 *** Dyaheon has joined #bitcoin-core-dev
2252017-07-28T14:44:19 *** J-wolf has joined #bitcoin-core-dev
2262017-07-28T14:44:57 *** Chris_Stewart_5 has quit IRC
2272017-07-28T14:56:05 *** J-wolf has quit IRC
2282017-07-28T14:59:12 *** mmgen has joined #bitcoin-core-dev
2292017-07-28T15:04:02 *** Guyver2 has quit IRC
2302017-07-28T15:04:21 *** Guyver2 has joined #bitcoin-core-dev
2312017-07-28T15:15:03 <Lightsword> for #10301 should I leave getentropy via sys/random.h as OSX only or try and generalize it?
2322017-07-28T15:15:04 <gribble> https://github.com/bitcoin/bitcoin/issues/10301 | Check if sys/random.h is required for getentropy. by jameshilliard · Pull Request #10301 · bitcoin/bitcoin · GitHub
2332017-07-28T15:20:26 *** stick`` is now known as stick`
2342017-07-28T15:21:23 *** J-wolf has joined #bitcoin-core-dev
2352017-07-28T15:21:48 *** J-wolf has quit IRC
2362017-07-28T15:22:34 *** jamesob has quit IRC
2372017-07-28T15:23:06 *** jamesob has joined #bitcoin-core-dev
2382017-07-28T15:23:19 *** J-wolf has joined #bitcoin-core-dev
2392017-07-28T15:26:49 *** J-wolf has quit IRC
2402017-07-28T15:27:52 *** jamesob has quit IRC
2412017-07-28T15:38:40 <jonasschnelli> Lightsword: I guess it makes sense on OSX 10.12(+) ..
2422017-07-28T15:42:57 <jonasschnelli> Lightsword: Do you expect other operating systems providing getentropy() via sys/random.h?
2432017-07-28T15:43:49 <Lightsword> jonasschnelli, well looks like itâs there on solaris but I donât really have a way to test against that, OSX also uses the weak import fork backwards compatibility
2442017-07-28T15:45:02 <jonasschnelli> Lightsword: By "generalize" you mean removing the && defined(MAC_OSX)?
2452017-07-28T15:45:08 <Lightsword> yeah
2462017-07-28T15:45:36 <jonasschnelli> I don't see a reason why not do to that.
2472017-07-28T15:46:23 <Lightsword> well there was a glibc issue right?
2482017-07-28T15:46:26 <jonasschnelli> But maybe ask cfields what he things
2492017-07-28T15:46:29 <jonasschnelli> thinks
2502017-07-28T15:46:44 *** J-wolf has joined #bitcoin-core-dev
2512017-07-28T15:49:54 *** SopaXorzTaker has quit IRC
2522017-07-28T15:51:21 *** SopaXorzTaker has joined #bitcoin-core-dev
2532017-07-28T15:53:44 *** ula has quit IRC
2542017-07-28T15:57:55 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2552017-07-28T15:58:49 *** instagibbs has quit IRC
2562017-07-28T16:00:09 *** instagibbs has joined #bitcoin-core-dev
2572017-07-28T16:04:19 *** vicenteH has quit IRC
2582017-07-28T16:12:10 *** str4d has joined #bitcoin-core-dev
2592017-07-28T16:13:06 <wumpus> Lightsword: I'd make it macosx specific
2602017-07-28T16:13:28 <wumpus> we should vet OSes where using getentropy is safe one by one
2612017-07-28T16:13:30 <Lightsword> wumpus, yeah I went that way since there was an openbsd specific version
2622017-07-28T16:13:35 <wumpus> right.
2632017-07-28T16:17:59 <wumpus> I initially made it too generalized, which caused a few issues (with linux, for example) in the first place
2642017-07-28T16:18:32 <wumpus> not serious issues with randomness, luckily, but build problems
2652017-07-28T16:22:18 *** jamesob has joined #bitcoin-core-dev
2662017-07-28T16:27:19 *** Chris_Stewart_5 has quit IRC
2672017-07-28T16:44:28 *** cheese_ has quit IRC
2682017-07-28T16:44:46 *** Dizzle has joined #bitcoin-core-dev
2692017-07-28T17:34:49 *** timothy has quit IRC
2702017-07-28T17:46:10 *** miknotauro has joined #bitcoin-core-dev
2712017-07-28T18:08:16 *** Chris_Stewart_5 has joined #bitcoin-core-dev
2722017-07-28T18:15:45 *** achow101 has quit IRC
2732017-07-28T18:23:05 *** vicenteH has joined #bitcoin-core-dev
2742017-07-28T18:23:32 *** Mordan has quit IRC
2752017-07-28T18:36:49 *** mmgen has quit IRC
2762017-07-28T18:38:24 *** snkey has joined #bitcoin-core-dev
2772017-07-28T18:41:34 *** snq has quit IRC
2782017-07-28T18:41:53 *** Chris_Stewart_5 has quit IRC
2792017-07-28T18:43:29 *** goatpig has quit IRC
2802017-07-28T18:43:53 *** luke-jr has quit IRC
2812017-07-28T18:51:32 *** miknotauro has quit IRC
2822017-07-28T18:53:55 *** miknotauro has joined #bitcoin-core-dev
2832017-07-28T18:55:13 *** jamesob_ has joined #bitcoin-core-dev
2842017-07-28T18:55:55 *** EagleTM has joined #bitcoin-core-dev
2852017-07-28T18:56:13 *** blznblzn2 has joined #bitcoin-core-dev
2862017-07-28T18:59:56 *** jamesob_ has quit IRC
2872017-07-28T19:00:59 *** jamesob_ has joined #bitcoin-core-dev
2882017-07-28T19:01:34 *** jamesob_ has joined #bitcoin-core-dev
2892017-07-28T19:02:06 *** eck has quit IRC
2902017-07-28T19:03:51 *** Dizzle has quit IRC
2912017-07-28T19:04:17 *** Drabiv has quit IRC
2922017-07-28T19:05:49 *** jamesob_ has quit IRC
2932017-07-28T19:07:39 *** Dizzle has joined #bitcoin-core-dev
2942017-07-28T19:08:54 *** Guyver2_ has joined #bitcoin-core-dev
2952017-07-28T19:11:57 *** Guyver2 has quit IRC
2962017-07-28T19:11:58 *** Guyver2_ is now known as Guyver2
2972017-07-28T19:12:23 *** snq has joined #bitcoin-core-dev
2982017-07-28T19:14:17 *** miknotauro has quit IRC
2992017-07-28T19:15:27 *** snkey has quit IRC
3002017-07-28T19:16:54 *** snkey has joined #bitcoin-core-dev
3012017-07-28T19:19:10 *** snq has quit IRC
3022017-07-28T19:24:53 *** snq has joined #bitcoin-core-dev
3032017-07-28T19:27:10 *** snkey has quit IRC
3042017-07-28T19:28:25 *** eck has joined #bitcoin-core-dev
3052017-07-28T19:40:38 *** achow101 has joined #bitcoin-core-dev
3062017-07-28T19:48:33 *** dermoth has quit IRC
3072017-07-28T19:48:48 *** Guyver2 has quit IRC
3082017-07-28T19:50:04 *** dermoth has joined #bitcoin-core-dev
3092017-07-28T19:53:39 *** whphhg has quit IRC
3102017-07-28T20:01:04 *** Dizzle has quit IRC
3112017-07-28T20:03:04 *** Drabiv has joined #bitcoin-core-dev
3122017-07-28T20:06:04 *** str4d has quit IRC
3132017-07-28T20:07:21 *** Drabiv has quit IRC
3142017-07-28T20:08:23 *** Drabiv has joined #bitcoin-core-dev
3152017-07-28T20:12:17 *** whphhg has joined #bitcoin-core-dev
3162017-07-28T20:12:34 *** Mordan has joined #bitcoin-core-dev
3172017-07-28T20:13:58 *** Drabiv has quit IRC
3182017-07-28T20:15:13 *** Drabiv has joined #bitcoin-core-dev
3192017-07-28T20:24:01 *** miknotauro has joined #bitcoin-core-dev
3202017-07-28T20:28:41 *** btcdrak has quit IRC
3212017-07-28T20:31:16 *** j_ybt has joined #bitcoin-core-dev
3222017-07-28T20:38:00 *** btcdrak has joined #bitcoin-core-dev
3232017-07-28T20:46:17 *** Murch has joined #bitcoin-core-dev
3242017-07-28T20:55:38 *** abpa has joined #bitcoin-core-dev
3252017-07-28T21:01:59 *** miknotauro has quit IRC
3262017-07-28T21:02:51 *** jamesob_ has joined #bitcoin-core-dev
3272017-07-28T21:06:57 *** jamesob_ has quit IRC
3282017-07-28T21:19:17 *** Drabiv has quit IRC
3292017-07-28T21:20:25 *** praxeology1 has joined #bitcoin-core-dev
3302017-07-28T21:21:10 *** praxeology1 has quit IRC
3312017-07-28T21:22:46 *** Giszmo has quit IRC
3322017-07-28T21:23:19 *** praxeology has quit IRC
3332017-07-28T21:41:07 *** Giszmo has joined #bitcoin-core-dev
3342017-07-28T21:44:14 <kanzure> sendtoaddress doesn't seem to do any chaining off of other sendtoaddress-constructed unconfirmed outputs. and getbalance doesn't have a flag to ignore amounts stuck in unconfirmed sendtoaddress-generated outputs. i think fundrawtransaction is my only option, is that right?
3352017-07-28T21:47:34 <kanzure> i guess i could use listunspent minconf=1 and check if the result is empty and/or take the sum of the amounts if any.
3362017-07-28T21:50:04 <kanzure> anyway, the getbalance RPC docstring could at least mention that it is a highly promiscuous definition of available, and the sendtoaddress docstring could mention the lack of chaining..
3372017-07-28T21:51:53 *** Mordan has quit IRC
3382017-07-28T21:54:44 *** johnpark_pj has quit IRC
3392017-07-28T21:57:49 *** johnpark_pj has joined #bitcoin-core-dev
3402017-07-28T22:00:43 <sipa> kanzure: well getbalance follows the mempool's rules for availability... and long chains of dependent transactions don't get into the mempool
3412017-07-28T22:01:45 <sipa> getbalance will aim to avoid spending low-confirmation count outputs, and avoid long-ubconfirmed-chain outputs if possible
3422017-07-28T22:03:09 <kanzure> my use case was very simple, i had mined some initial blocks, needed to spend them somewhere, was spending a bunch of tiny amounts, exhausted available utxos; my test was checking getbalance and mining a block if the balance was below some threshold. but the balance is reported as very high despite having no sendtoaddress-spendable coins, oops.
3432017-07-28T22:20:11 *** jeep-ss has joined #bitcoin-core-dev
3442017-07-28T22:20:21 *** praxeology has joined #bitcoin-core-dev
3452017-07-28T22:26:24 *** Yogaqueef has quit IRC
3462017-07-28T22:27:06 *** Mordan has joined #bitcoin-core-dev
3472017-07-28T22:30:07 *** J-wolf has quit IRC
3482017-07-28T22:38:27 *** Aaronvan_ has joined #bitcoin-core-dev
3492017-07-28T22:40:37 *** AaronvanW has quit IRC
3502017-07-28T22:49:22 <kanzure> also, how do i debug "insufficient fee" in regtest mode? the transaction is 3528 bytes paying 30 sat/byte (so 105840 sat).
3512017-07-28T22:49:24 *** Aaronvan_ is now known as AaronvanW
3522017-07-28T22:51:10 *** Giszmo has quit IRC
3532017-07-28T23:04:08 *** jamesob_ has joined #bitcoin-core-dev
3542017-07-28T23:08:40 *** jamesob_ has quit IRC
3552017-07-28T23:14:10 *** intcat has quit IRC
3562017-07-28T23:16:18 *** Aaronvan_ has joined #bitcoin-core-dev
3572017-07-28T23:17:04 *** Aaronva__ has joined #bitcoin-core-dev
3582017-07-28T23:18:27 *** str4d has joined #bitcoin-core-dev
3592017-07-28T23:19:52 *** AaronvanW has quit IRC
3602017-07-28T23:20:30 *** Aaronvan_ has quit IRC
3612017-07-28T23:21:21 *** intcat has joined #bitcoin-core-dev
3622017-07-28T23:23:46 *** quer has joined #bitcoin-core-dev
3632017-07-28T23:40:26 *** quer has quit IRC
3642017-07-28T23:42:24 *** snkey has joined #bitcoin-core-dev
3652017-07-28T23:43:10 *** Mordan has quit IRC
3662017-07-28T23:45:04 *** snq has quit IRC
3672017-07-28T23:48:23 *** snq has joined #bitcoin-core-dev
3682017-07-28T23:51:22 *** snkey has quit IRC
3692017-07-28T23:52:42 *** intcat has quit IRC
3702017-07-28T23:52:54 *** snkey has joined #bitcoin-core-dev
3712017-07-28T23:53:05 *** intcat has joined #bitcoin-core-dev
3722017-07-28T23:55:46 *** snq has quit IRC
3732017-07-28T23:59:24 *** snq has joined #bitcoin-core-dev
3742017-07-28T23:59:51 *** Aaronva__ is now known as AaronvanW