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

small improvements #174

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pages/docs/user-docs/docs-apps.md
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,7 @@ path nonsense? Don't worry, you don't. You can just use environment variables in
singularity exec --app foo foobar.simg env | grep foo
```

Let's talk about the output of the above in sections, you will notice some interesting things! First, notice that the app's `bin` has been added to the path, and it's `lib` added to the `LD_LIBRARY_PATH`. This means that anything you drop in either will automatically be added. You don't need to make these folders either, they are created for you.
Let's talk about the output of the above in sections, you will notice some interesting things! First, notice that the app's `bin` has been added to the path, and its `lib` added to the `LD_LIBRARY_PATH`. This means that anything you drop in either will be automatically added. You don't have to create those folders neither, they'll be created for you.

Copy link
Member

Choose a reason for hiding this comment

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

This would be a double negative, "don't have to" plus "neither" would mean you have to create them. I think it should be "you don't have to create those folders either"

```
LD_LIBRARY_PATH=/scif/apps/foo/lib::/.singularity.d/libs
Expand Down