I found it really helpful when debugging stuff to not need to delete the debug.log everytime I re-start the client.
- changes ShrinkDebugFile() function to use nice MiB and KiB boundaries
add -cleardebugfile switch to clear debug.log on startup #1811
pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:add_cleardebugfile changing 3 files +18 −10-
Diapolo commented at 6:44 PM on September 9, 2012: none
-
0de15d3f38
add -cleardebugfile switch to clear debug.log on startup
I found it really helpful when debugging stuff to not need to delete the debug.log everytime I re-start the client. - changes ShrinkDebugFile() function to use nice MiB and KiB boundaries
-
luke-jr commented at 8:56 PM on September 9, 2012: member
As much as I prefer using historically-standard KB/MB units in general, this client has always used SI-standard kB/MB units consistently. So ACK changing it (ideally everywhere), and ACK leaving it alone too ;)
On the new commandline option, I feel we are getting a bit too much code to configure a mere debug log. In this case, it should be just as easy to use your OS's rm/del command immediately before launching bitcoind. Is there any reason to have it builtin?
-
laanwj commented at 2:08 AM on September 10, 2012: member
Removing debug logs is one of these actions that can easily be done in the script that launches bitcoin(d). It's not worth adding yet another command-line flag for.
Let's keep command line options for bitcoin/network-related settings.
-
gavinandresen commented at 3:19 PM on September 10, 2012: contributor
NACK. I agree with @laanwj
-
Diapolo commented at 7:11 PM on September 10, 2012: none
Closing for getting too much NACKs ^^.
- Diapolo closed this on Sep 10, 2012
-
BitcoinPullTester commented at 2:47 AM on September 11, 2012: none
Automatic sanity-testing: FAILED MERGE, see http://jenkins.bluematt.me/pull-tester/0de15d3f386852d8afb30be85c0f53a3274897f6 for test log.
This pull does not merge cleanly onto current master
- DrahtBot locked this on Sep 8, 2021
