Open
Conversation
MakisH
approved these changes
Apr 14, 2026
|
|
||
| ```conf | ||
| # set this to your selected installation prefix | ||
| PRECICE_PREFIX="${HOME}/software/prefix" |
| This method is discouraged as file layouts are fundamentally different, and we make **no** guarantees on keeping them consistent across releases. | ||
| Hence, the only reliable methods for using preCICE from the binary directory require `pkg-config` or `CMake`. | ||
| If your adapter or solver isn't using one of these methods, then we strongly suggest to install preCICE to a prefix or port the adapter to `pkg-config`. | ||
| If your adapter or solver isn't using one of these methods, we strongly suggest installing preCICE to a prefix or porting the adapter to `pkg-config` or `CMake`. |
Member
There was a problem hiding this comment.
Suggested change
| If your adapter or solver isn't using one of these methods, we strongly suggest installing preCICE to a prefix or porting the adapter to `pkg-config` or `CMake`. | |
| If your adapter or solver isn't using one of these methods, we strongly suggest installing preCICE to a prefix or porting the adapter to pkg-config or CMake. |
At least for CMake, strange to have the name as a command. This is actually the same in the complete page.
|
|
||
| ```conf | ||
| # Only for using preCICE directly from the build directory | ||
| # (as of preCICE v3.3.0 - later versions might differ). |
Member
There was a problem hiding this comment.
Could also bump the version here, since this PR is touching the page and the assumption still holds:
Suggested change
| # (as of preCICE v3.3.0 - later versions might differ). | |
| # (as of preCICE v3.4.0 - later versions might differ). |
Comment on lines
+100
to
+106
| After adding the file, logout and login again. Opening a new terminal will **not** be sufficient. | ||
| Verify the changes by listing the loaded environment in a new terminal: | ||
|
|
||
| ```terminal | ||
| systemctl show-environment | ||
| ``` | ||
|
|
Member
There was a problem hiding this comment.
Was this intentional repetition?
Suggested change
| After adding the file, logout and login again. Opening a new terminal will **not** be sufficient. | |
| Verify the changes by listing the loaded environment in a new terminal: | |
| ```terminal | |
| systemctl show-environment | |
| ``` |
| If the library `libprecice.so` cannot be found, then `LD_LIBRARY_PATH` is incorrect. | ||
| If the displayed version differs from the version you expect, then you have multiple conflicting preCICE installations on your system. | ||
|
|
||
| Next run: |
Member
There was a problem hiding this comment.
Suggested change
| Next run: | |
| Next, run: |
| @@ -32,12 +32,12 @@ Then add the following to your `.profile` (for bash) or `.zshrc` (for zsh). | |||
| ```bash | |||
| # set this to your selected installation prefix | |||
| PRECICE_PREFIX="~/software/prefix" | |||
Member
There was a problem hiding this comment.
shellcheck warning:
PRECICE_PREFIX="~/software/prefix"
^---------------^ SC2088 (warning): Tilde does not expand in quotes. Use $HOME.
Suggested change
| PRECICE_PREFIX="~/software/prefix" | |
| PRECICE_PREFIX="${HOME}/software/prefix" |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR cleans up the finding section.
${CPATH:+:$CPATH}, meaning if CPATH exists, evaluate to:$CPATH. Empty includes due to tailing:break compilers in super strange ways.environment.dchanges.