contrib: Update location of seeds.txt #10495

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2017_06_seeds_source changing 1 files +1 −1

laanwj commented at 8:40 AM on June 1, 2017: member

Update the steps for updating the hardcoded seed nodes to point to the new filename on @sipa's server, and add command to decompress it.

Ref: #10163
laanwj added the label Docs and Output on Jun 1, 2017
jonasschnelli commented at 8:59 AM on June 1, 2017: contributor

Tested ACK ac9df130a507177ec77ef3f5f4fa43c0c5ef57c3
in contrib/seeds/README.md:11 in ac9df130a5 outdated
```
   7 | @@ -8,7 +8,7 @@ and remove old versions as necessary.
   8 |  
   9 |  The seeds compiled into the release are created from sipa's DNS seed data, like this:
  10 |  
  11 | -    curl -s http://bitcoin.sipa.be/seeds.txt > seeds_main.txt
  12 | +    curl -s http://bitcoin.sipa.be/seeds.txt.gz | gunzip > seeds_main.txt
```
practicalswift commented at 9:21 AM on June 1, 2017:

Nit: Isn't | gzip -dc preferred over | gunzip?

laanwj commented at 9:27 AM on June 1, 2017:

For what reason?

practicalswift commented at 9:40 AM on June 1, 2017:

@laanwj Good question! :-) Perhaps it is a legacy thing, but for some reason I have it hard wired that gzip -dc is used when operating on stdin/stdout and gunzip is for unpacking file.gz, removing the original file and leaving file. They might be totally equivalent nowadays. If so, ignore my minor nit :-)

practicalswift commented at 9:44 AM on June 1, 2017:

Found this thread: https://lists.gnu.org/archive/html/bug-automake/2009-10/msg00012.html
paveljanik commented at 9:43 AM on June 1, 2017: contributor

IIRC on some systems, gunzip symlink was not available in the default install.
laanwj commented at 9:51 AM on June 1, 2017: member

IIRC on some systems, gunzip symlink was not available in the default install.

I tried on FreeBSD, OpenBSD, Linux (Ubuntu) and gunzip worked out of the box at least... Also these are not general instructions for end-users, but for ourselves. I personally find gunzip much easier to remember. But meh I'm fine with changing it.

contrib: Update location of seeds.txt

Update the steps for updating the hardcoded seed nodes to point to the
new filename on @sipa's server, and add command to decompress it.

Ref: #10163

ac9cd953d9

laanwj force-pushed on Jun 1, 2017
fanquake commented at 10:46 AM on June 1, 2017: member

utACK https://github.com/bitcoin/bitcoin/pull/10495/commits/ac9cd953d91e2448d24851b2f4973c7be8f4c3cd
practicalswift commented at 10:49 AM on June 1, 2017: contributor

ACK ac9cd95
laanwj merged this on Jun 1, 2017
laanwj closed this on Jun 1, 2017
laanwj referenced this in commit 6a38b79de4 on Jun 1, 2017
luke-jr referenced this in commit 67014a0f21 on Jun 3, 2017
luke-jr referenced this in commit 5784efe7dc on Jun 3, 2017
luke-jr referenced this in commit ad0b0e75c9 on Jun 3, 2017
luke-jr referenced this in commit b6a17892b4 on Jun 3, 2017
luke-jr referenced this in commit 4e02c454b2 on Jun 5, 2017
luke-jr referenced this in commit 7794ef540c on Jun 5, 2017
luke-jr referenced this in commit 76f9cf9ac9 on Jun 5, 2017
nomnombtc referenced this in commit 0e1fe8cc4e on Jul 17, 2017
codablock referenced this in commit e9976d20f1 on Jan 26, 2018
andvgal referenced this in commit b806c90bf7 on Jan 6, 2019
CryptoCentric referenced this in commit dcddd69d29 on Feb 27, 2019
DrahtBot locked this on Sep 8, 2021

Contributors

Labels

Linked (view graph)

#10163 Downloading sipa's seeds.txt yields a 404 error