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

Error in installation documentation #7797

Closed
howardlowndes opened this issue Dec 9, 2023 · 8 comments
Closed

Error in installation documentation #7797

howardlowndes opened this issue Dec 9, 2023 · 8 comments
Labels
type:bug Bug that will be worked on as resources permit

Comments

@howardlowndes
Copy link

Version

Version 1.11.4

What browsers are you seeing the problem on?

This problem isn't browser related

What happened?

In the link https://ledgersmb.org/content/installing-ledgersmb-111, when dealing with setting up systemd, there is the line sed -e "s#WORKING_DIR#$PWD#" conf/systemd/ledgersmb_starman.service \ | sudo tee /etc/systemd/system/ledgersmb-starman.service
Executing this produces the error sed: can't read conf/systemd/ledgersmb_starman.service: No such file or directory

What should have happened?

The conf/ directory is now doc/conf/
On line documentation needs correction

@howardlowndes
Copy link
Author

Additionally starman is installed in /usr/local/bin contrary to the systemd doco, which has it in /usr/bin

@ehuelsmann
Copy link
Member

The conf/ directory is now doc/conf/
On line documentation needs correction

Thanks for the report! I've fixed the 1.11 documentation online and checked - and changed where necessary - all versions of the document back to 1.7 (when the move of conf/ to doc/conf/ was first released).

@ehuelsmann
Copy link
Member

Additionally starman is installed in /usr/local/bin contrary to the systemd doco, which has it in /usr/bin

Can you point me a bit more specifically where to look? I can't find it in the 1.11 installation document (the one I just changed with respect the doc/conf/ path).

By the way, where Starman can be found depends on whether it's installed from CPAN or by the system package manager. In the latter case, you will find it in /usr/bin; from what I understood from your installation procedure, you've installed from CPAN, so yours is in /usr/local/bin indeed. I think this clarification should be added to the documentation. I'll do it when you point me where to look :-).

@howardlowndes
Copy link
Author

howardlowndes commented Dec 10, 2023 via email

@neilt neilt added the type:bug Bug that will be worked on as resources permit label Dec 10, 2023
@ehuelsmann
Copy link
Member

Thanks for your quick reply! Regarding this conversation:

Additionally starman is installed in /usr/local/bin contrary to the systemd doco, which has it in /usr/bin

Can you point me a bit more specifically where to look? I can't find it in the 1.11 installation document (the one I just changed with respect the doc/conf/ path).

I meant to ask "can you point me a bit more specifically to the point in the docs where it says that starman is in /usr/bin?" because I'm having trouble finding the point in the docs where it does. I suspect I'm looking in the wrong place though...

@howardlowndes
Copy link
Author

howardlowndes commented Dec 11, 2023 via email

@ehuelsmann
Copy link
Member

I don't think the systemd commands are executed with a PATH sein. (Not sure though). Anybody using starman from their package repository (the majority?) will have it in /use/bin. Our Debian package also uses this file with minimal changes. I prefer to keep things simple for packages. Those installing by themselves without using Docker I expect they know what they are doing.

That leaves all actions in this issue addressed. Closing.

@howardlowndes
Copy link
Author

howardlowndes commented Dec 17, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:bug Bug that will be worked on as resources permit
Projects
None yet
Development

No branches or pull requests

3 participants