Skip to content

Conversation

jonatack
Copy link
Member

@jonatack jonatack commented Sep 4, 2020

Seen while reviewing #19572.

@fanquake fanquake added the Docs label Sep 4, 2020
@jonatack jonatack changed the title doc: update PyZMQ installation instructions, ZeroMQ link script, doc: update PyZMQ install instructions, fix zmq_sub.py file permissions Sep 4, 2020
@jonatack
Copy link
Member Author

jonatack commented Sep 4, 2020

If I'm not mistaken, contrib/zmq/zmq_sub.py has the wrong file permissions; changed them to 755.

Copy link
Contributor

@theStack theStack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 062e669 🧷
Checked that the changed/introduced URLs are visitable and correct and that the PyZMQ install instruction via pip works (tested in Ubuntu Xenial). Also ran the zmq_sub.py script directly to double-check that the execution flag is set now.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 062e669

@fanquake fanquake changed the title script, doc: update PyZMQ install instructions, fix zmq_sub.py file permissions doc: update PyZMQ install instructions, fix zmq_sub.py file permissions Sep 11, 2020
@fanquake fanquake merged commit dffefda into bitcoin:master Sep 11, 2020
@jonatack jonatack deleted the zmq-doc-fix branch September 11, 2020 06:20
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 11, 2020
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 27, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 28, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jun 29, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 1, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 15, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
PastaPastaPasta pushed a commit to PastaPastaPasta/dash that referenced this pull request Jul 16, 2021
…sub.py file permissions

062e669 script: fix zmq_sub.py file permissions (Jon Atack)
36f8e0c doc: update PyZMQ installation instructions, ZeroMQ link (Jon Atack)

Pull request description:

  Seen while reviewing bitcoin#19572.

ACKs for top commit:
  theStack:
    ACK 062e669 🧷
  fanquake:
    ACK 062e669

Tree-SHA512: 2210d92385377d066984d0a83882c3ece9f0f41c901b7eb375af9cdb57296f50f227c68193ccf35926073c2b788d58976442791a9fce2fc0f76452804d5cee6a
deadalnix pushed a commit to Bitcoin-ABC/bitcoin-abc that referenced this pull request Oct 1, 2021
Summary: This is a backport of [[bitcoin/bitcoin#19870 | core#19870]]

Test Plan: Tested link and `pyzmq` installation instruction

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Differential Revision: https://reviews.bitcoinabc.org/D10232
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants