Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Remove nano-pow-server from nano-node git repo and docker #4042

Closed
dsiganos opened this issue Jan 12, 2023 · 1 comment · Fixed by #4043
Closed

Remove nano-pow-server from nano-node git repo and docker #4042

dsiganos opened this issue Jan 12, 2023 · 1 comment · Fixed by #4043
Assignees
@dsiganos
Labels
docker updates and changes to the docker build image documentation This item indicates the need for or supplies updated or expanded documentation removal Indicates functionality is being removed
Milestone
Todo

Comments

@dsiganos
Copy link
Contributor

dsiganos commented Jan 12, 2023 •
edited
Loading

The nano-node project brings in C++ nano-pow-server as a git submodule and, during the release process, it builds it and adds it to the nano-node docker.
https://github.com/nanocurrency/nano-pow-server.git

The nano-pow-server is both an independent project and a submodule of nano-node.
That is a complexity that seems unnecessary.

Additionally, there is an alternative pow server called nano-work-server and written in Rust, which seems to have superseded nano-pow-server.
https://github.com/nanocurrency/nano-pow-server.git

In the interests of reducing unnecessary complexity, we will be removing nano-pow-server from nano GitHub and from nano docker releases.
@dsiganos dsiganos added docker updates and changes to the docker build image removal Indicates functionality is being removed labels Jan 12, 2023
@dsiganos dsiganos added this to the Todo milestone Jan 12, 2023
@dsiganos dsiganos self-assigned this Jan 12, 2023
@dsiganos dsiganos mentioned this issue Jan 13, 2023
Merged
@dsiganos dsiganos added the documentation This item indicates the need for or supplies updated or expanded documentation label Jan 13, 2023
@dsiganos
Copy link
Contributor Author

nano.org has references to nano-pow-server that also need to removed.

# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@dsiganos dsiganos

Labels
docker updates and changes to the docker build image documentation This item indicates the need for or supplies updated or expanded documentation removal Indicates functionality is being removed
Projects
None yet
Milestone
Todo
Development

Successfully merging a pull request may close this issue.

Remove nano pow server dsiganos/nano-node
1 participant
@dsiganos