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

Update README.md #51

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update README.md #51

wants to merge 3 commits into from

Conversation

masiv1001
Copy link

Update for installation procedure.
Added distribution specific instructions for installing node using NodeSource binaries, leaving source compilation only for those who need it.

Update for installation procedure.
Added distribution specific instructions for installing node using NodeSource binaries, leaving source compilation only for those who need it.
Remove inconsistencies, fix typos, and add git as a dependency.
@masiv1001
Copy link
Author

There should be a way to standardize ArRPC as a service, using systemd.

Something like:

[Unit]
Description=ArRPC

[Service]
ExecStart=/var/www/arrpc/src
Restart=always
User=nobody
# Note Debian/Ubuntu uses 'nogroup', RHEL/Fedora uses 'nobody'
Group=nogroup
Environment=PATH=/usr/bin:/usr/local/bin
Environment=NODE_ENV=production
WorkingDirectory=/var/www/arrpc

[Install]
WantedBy=multi-user.target

Not sure yet how it works, I still have to do a little research but I suspect arrpc should not be downloaded directly to the current directory via git, it should be instead copied to /var/www/arrpc
Above code is only preliminary, a template taken from another place.

Small inconsistency (dnf and yum) fix.
@ariankordi
Copy link
Contributor

It may help if you make it into a user service rather than a system service by putting it in ~/.config/systemd/user, then use systemctl --user like systemctl --user status arrpc.service etc.

@masiv1001
Copy link
Author

Thanks for your advise, I'm new with these kinds of things. I will research about it, try to implement it and see how it goes :)

@bagusnl
Copy link

bagusnl commented Oct 30, 2023

We could also add guide to install it on Windows using winget as package manager to install both NodeJS (OpenJS.NodeJS) and NSSM (NSSM.NSSM) to make it as a service that runs on Windows start and does not give any console window.
Here is the commands to be used to add the service using NSSM:

nssm install arRPC "C:\Program Files\nodejs\node.exe"
nssm set arRPC AppParameters src
nssm set arRPC AppDirectory F:\GitHub\arRPC
nssm set arRPC AppExit Default Exit
nssm set arRPC AppPriority BELOW_NORMAL_PRIORITY_CLASS
nssm set arRPC AppStdout F:\GitHub\arRPC.log
nssm set arRPC AppStdoutCreationDisposition 2
nssm set arRPC AppStderr F:\GitHub\arRPC.log
nssm set arRPC AppStderrCreationDisposition 2
nssm set arRPC DisplayName arRPC
nssm set arRPC ObjectName LocalSystem
nssm set arRPC Start SERVICE_DELAYED_AUTO_START
nssm set arRPC Type SERVICE_INTERACTIVE_PROCESS

@masiv1001
Copy link
Author

I think standardizing the way arrpc initializes using a user-service or similar is out of the scope of this commit, therefore I request the merging to the main branch for the time being for the sake of user experience.

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.

3 participants