Add I2P to -onlynet docs and make all release notes present tense
-onlynet doc cleanup; present tense release-notes #1736
pull freewil wants to merge 1 commits into bitcoin:master from freewil:release-notes-onlynet changing 2 files +10 −10-
freewil commented at 1:54 PM on August 28, 2012: contributor
-
df15f7958e
-onlynet doc cleanup; present tense release-notes
Add I2P to -onlynet docs and make all release notes present tense
-
jgarzik commented at 2:06 PM on August 28, 2012: contributor
Please separate into two separate commits. We do not put different logical changes (grammar changes + onlynet docs) in the same commit.
In this case, your grammar changes are NAK'd and your other changes are ACK'd.
-
sipa commented at 2:25 PM on August 28, 2012: member
Please don't add I2P to the documentation. There is only support for it at the level of parsing them, and mapping the garlicat IPv6 range to it. For all other purposes, it is not functional.
Furthermore, I've recently learned that this way of implementation is not viable at all, as I2P needs longer addresses to be globally routable. The correct solution is removing it entirely from the source, and later re-add it correctly, I'm afraid. That will be for after 0.7, though.
-
-
jgarzik commented at 5:42 PM on August 28, 2012: contributor
closing, superceded
- jgarzik closed this on Aug 28, 2012
- owlhooter referenced this in commit f4502099a4 on Oct 10, 2018
- DrahtBot locked this on Sep 8, 2021
