Skip to content

Conversation

smatthewenglish
Copy link
Contributor

keypoololdest is seconds since Unix epoch, not "GMT epoch" as was previously written

Resources:
https://bitcointalk.org/index.php?topic=4451.0
http://bitcoin.stackexchange.com/questions/35913/what-does-the-keypoololdest-field-mean

@maflcko
Copy link
Member

maflcko commented Oct 30, 2016

@s-matthew-english Did you know you can use $ git grep keypoololdest to see if there are other affected rpcs, so all of them can be fixed in one go?

@smatthewenglish
Copy link
Contributor Author

ahh, yeah I mean, I knew of 'grep' but I didn't use it before really. at the moment I'm tied up, but definetly later tonight I can and will do that, but- if you don't mind, could I try to do it myself so I can get some experience with that?

@smatthewenglish
Copy link
Contributor Author

alright, great!

I've grepped for 'GMT epoch' and I spotted another one in "src/wallet/rpcwallet.cpp", so now I've changed that one too.

I commited everything according to that discussion we had last time so- I think I did it right. Is it so?

@maflcko
Copy link
Member

maflcko commented Oct 31, 2016

utACK 7f61b49, I think both describe the same thing, but it can't hurt...

@sipa
Copy link
Member

sipa commented Oct 31, 2016

They do mean the same thing, but I think that Unix epoch is less ambiguous.

@maflcko maflcko merged commit 7f61b49 into bitcoin:master Oct 31, 2016
maflcko pushed a commit that referenced this pull request Oct 31, 2016
7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
luke-jr pushed a commit to luke-jr/bitcoin that referenced this pull request Dec 2, 2016
codablock pushed a commit to codablock/dash that referenced this pull request Jan 13, 2018
7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
CryptoCentric pushed a commit to absolute-community/absolute that referenced this pull request Feb 15, 2019
7f61b49 Change all instance of 'GMT epoch' to 'Unix epoch' (matthias)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants