Merge ~fginther/+git/raspi-rt:fginther/bionic-install-deps into ~juergh/+git/raspi-rt-tests:main
Proposed by
Francis Ginther
Status: | Needs review |
---|---|
Proposed branch: | ~fginther/+git/raspi-rt:fginther/bionic-install-deps |
Merge into: | ~juergh/+git/raspi-rt-tests:main |
Diff against target: |
12 lines (+0/-1) 1 file modified
lib/run-test (+0/-1) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juerg Haefliger | Pending | ||
Review via email: mp+412643@code.launchpad.net |
Commit message
install-deps: Don't capitalize RT_OS_CODENAME
Doing so will result in any codename checks failing if they also don't
use a capitalized string.
Signed-off-by: Francis Ginther <email address hidden>
Description of the change
The following will fail to skip bionic due to the capitalization:
# Add dependencies required by this script
if [ "${RT_OS_CODENAME}" != "bionic" ] ; then
fi
I fine it easier to keep these string as returned to avoid having issues the next time a codename specific condition is needed.
To post a comment you must log in.