-
Notifications
You must be signed in to change notification settings - Fork 70
Open
Labels
Devicetree-bandDT-band specific issues/enhancements/questions.DT-band specific issues/enhancements/questions.reporting_relatedissues/enhancements/questions related to logs processingissues/enhancements/questions related to logs processing
Description
- When switching to use the Python script, some control characters are redirected to
ethtool_test.log, which causes parsing issues in the generated report.
- The script link in this page needs to be updated: Test-installation-of-Linux-distributions.
- Option A: Update the link to point to the correct script.
- Option B: Remove the script link from the webpage and instead directly link to the GitHub README section.
The original issue:
### Description:
When running ethtool-test.sh on a system where /bin/sh is not bash (e.g. Debian/Ubuntu, where /bin/sh is linked to dash), the script fails immediately with a syntax error:
./ethtool-test.sh: 30: Syntax error: "(" unexpected
This happens because the script uses Bash-specific features (arrays, [[ ]], =~, for (( )), etc.) but the shebang is #!/bin/sh. In POSIX sh, these constructs are not supported.
Update #!/bin/sh to be #!/bin/bash then issue is gone.
### Environment:
Debian 13
Metadata
Metadata
Assignees
Labels
Devicetree-bandDT-band specific issues/enhancements/questions.DT-band specific issues/enhancements/questions.reporting_relatedissues/enhancements/questions related to logs processingissues/enhancements/questions related to logs processing