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

Conversation

FooBarQuaxx
Copy link
Contributor

No description provided.

@@ -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"

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.

2 participants