Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed "Access is denied." running Mainnet+Testnet #654

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

SkybuckFlying
Copy link

@SkybuckFlying SkybuckFlying commented Aug 27, 2024

What kind of change does this PR introduce? (check at least one)

  • Improvement
  • Bugfix
  • Feature
  • Refactor
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • [x ] No

Other information:

Fixes "Access is denied." when trying to run Testnet on the same Windows 11 machine as Mainnet. (Two go-vite executables).

The cause of the problem was both executables try to create the same IPC pipe with same name.

The solution was to make sure each executable has it's own pipe name. This is now done by adding a underscore and NetSelect string at the back of the pipe name.

This should now allow running mainnet and testnet side by side on same machine.

Unfortunately I cannot test it any further, because the testnet seems to be done ? Or at least the seed nodes/json/server is error 404.

I hope this gets fixed so I can run testnet as well, as well as others.

I am not sure what the effect of this could be on other operating systems, not sure if this code is supposed to be cross-platform, this should be tested by others first before applieing this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant