Skip to content

Conversation

theStack
Copy link
Contributor

The file permissions for src/net_processing.cpp have been changed in #21713, as discovered by fanquake (#21713 (comment)). This PR removes the executable flag again.

@kiminuo
Copy link
Contributor

kiminuo commented Apr 19, 2021

ACK f2f2541 :)

Changes the file permissions back to the original.

@jnewbery
Copy link
Contributor

ACK f2f2541

@promag
Copy link
Contributor

promag commented Apr 19, 2021

ACK f2f2541.

@rebroad
Copy link
Contributor

rebroad commented Apr 19, 2021

darn it.. sorry... something in my setup makes this keep happening... a bug in linux for windows perhaps.

@fanquake fanquake merged commit 67a3593 into bitcoin:master Apr 20, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Apr 20, 2021
f2f2541 remove executable flag for src/net_processing.cpp (Sebastian Falbesoner)

Pull request description:

  The file permissions for `src/net_processing.cpp` have been changed in bitcoin#21713, as discovered by fanquake (bitcoin#21713 (comment)). This PR removes the executable flag again.

ACKs for top commit:
  kiminuo:
    ACK f2f2541 :)
  jnewbery:
    ACK f2f2541
  promag:
    ACK f2f2541.

Tree-SHA512: 1d5a62afb1152029e69fccea2ae53dcb262a91724a5c03dfc4de8c409b280814d0c211c2f9a71f1a6e927f4ed571ba4ac311de9de8ebb797eaf1051674241bdb
Copy link

@78051301012 78051301012 left a comment

Choose a reason for hiding this comment

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

ok

@bitcoin bitcoin locked and limited conversation to collaborators Apr 20, 2021
@theStack theStack deleted the 2021-remove-exec-flag-from-net_processing branch July 31, 2021 20:08
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.

8 participants