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

Add message about ldconfig #1050

Closed
wants to merge 2 commits into from

Conversation

TwilightDuck
Copy link

@TwilightDuck TwilightDuck commented May 4, 2023

Checklist

  • I have read the How to Contribute.
  • I have run yarn build after adding my changes without getting any errors.

Updating documentation or Bugfix

I wanted to run a preview node using systemd. I followed the exact instructions, but the systemd error output showed an error. After research I found this can be remedied by simply running ldconfig after setup.

@@ -515,6 +515,16 @@ cp -p "$(./scripts/bin-path.sh cardano-node)" $HOME/.local/bin/
cp -p "$(./scripts/bin-path.sh cardano-cli)" $HOME/.local/bin/
```

In order to run the cardano-node as a service using systemd you need to run the following command:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Since there are heaps of Linux flavours around:

Simply running ldconfig may not be enough (neither absence of it will guarantee the error, if LD_LIBRARY_PATH already points to local lib folder).

If using ldconfig, it would be an essential step to ensure /usr/local/lib - or the path where you install the lib to - is part of /etc/ld.so.config (or mentioned in a file in /etc/ld.so.conf.d).

Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

I was also pondering this when #1050 (comment) came in: the statement is literally true for appropriate flavours of Linux, but without background information also makes it sound like this is the only circumstance where you might use ldconfig and/or adjust LD_LIBRARY_PATH.

You might need it the first time running cardano-node from the command line or after adding dependent libraries. I think the addition would require a more complete explanation of why this is necessary to run cardano-node in general and not just for systemd while also addressing the issue of what to expect on different platforms.

@fill-the-fill fill-the-fill added the documentation Improvements or additions to documentation label May 5, 2023
@katomm
Copy link
Member

katomm commented Aug 1, 2023

@NickJorens are you finalizing this?

@TwilightDuck
Copy link
Author

@NickJorens are you finalizing this?

I'm sorry, but I wasn't planning on following up on this.

@rphair
Copy link
Collaborator

rphair commented Aug 1, 2023

OK then @katomm we have to drop this, because the advice offered is only intermittently applicable & not without some explanatory context.

@rphair rphair closed this Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants