Skip to content

Commit 858f45b

Browse files
author
Richard Top
committed
updated the text to fit PR as a variable
1 parent 75a1cd4 commit 858f45b

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

docs/adding_software/debugging_failed_builds.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,9 @@ The Automatic build and deploy [bot](../bot.md) installs software by executing t
144144

145145
Note that you could also combine approaches: first build everything using the `EESSI-install-software.sh` script, until you reproduce the failure. Then, start making modifications (e.g. changes to the EasyConfig, patches, etc) and trying to rebuild that package individually to test your changes.
146146

147-
To build software using the `EESSI-install-software.sh` script, you'll first need to get the diff file for the PR. This is used by the `EESSI-install-software.sh` script to see what is changed in this PR - and thus what needs to be build for this PR. To download the diff for PR 360, we would e.g. do
147+
To build software using the EESSI-install-software.sh script, you first need to download the diff file for the relevant pull request. The script uses this diff to determine which changes are included in the PR and therefore which software needs to be built.
148+
149+
We now define the PR number through a ${PR} variable and use it throughout the commands. For example:
148150
```
149151
PR=123456789 # change this to correct the PR number
150152
wget https://github.com/EESSI/software-layer/pull/${PR}.diff

0 commit comments

Comments
 (0)