diff options
| author | Albin <albin@mullvad.net> | 2023-04-24 15:40:53 +0200 |
|---|---|---|
| committer | Albin <albin@mullvad.net> | 2023-04-24 15:54:46 +0200 |
| commit | f35ef1254b6b52542873d1020df80e3a25e08063 (patch) | |
| tree | 53507d4a59df90863584189c2972bf37da08f386 /android | |
| parent | c826091de45ae9af387472d24ecebb7ad5600fad (diff) | |
| download | mullvadvpn-f35ef1254b6b52542873d1020df80e3a25e08063.tar.xz mullvadvpn-f35ef1254b6b52542873d1020df80e3a25e08063.zip | |
Restructure tidy formatting script
The script can now be executed rather than having to be sourced.
Diffstat (limited to 'android')
| -rwxr-xr-x[-rw-r--r--] | android/scripts/ci-android-xml.sh | 44 |
1 files changed, 31 insertions, 13 deletions
diff --git a/android/scripts/ci-android-xml.sh b/android/scripts/ci-android-xml.sh index 0fd552b3a6..8e0546046b 100644..100755 --- a/android/scripts/ci-android-xml.sh +++ b/android/scripts/ci-android-xml.sh @@ -3,8 +3,27 @@ # CI/Developer script to format # Relies on Tidy - https://github.com/htacg/tidy-html5 +set -eu + +SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )" +cd "$SCRIPT_DIR" + +function main { + case ${1:-""} in + format) format;; + formatAndCheckDiff) format && checkDiff;; + "") + echo "Available subcommands: format, formatAndCheckDiff" + ;; + *) + echo "Unknown parameter: $1" + exit 1 + ;; + esac +} + # Autoformats Android XML files -function tidy-up-android-xml { +function format { tidy -xml \ -m \ -i \ @@ -14,25 +33,24 @@ function tidy-up-android-xml { --indent-attributes yes \ --indent-spaces 4 \ --literal-attributes yes \ - android/app/src/main/AndroidManifest.xml \ - android/app/src/main/res/anim*/*.xml \ - android/app/src/main/res/drawable*/*.xml \ - android/app/src/main/res/layout*/*.xml \ - android/app/src/main/res/values/*.xml + ../app/src/main/AndroidManifest.xml \ + ../app/src/main/res/anim*/*.xml \ + ../app/src/main/res/drawable*/*.xml \ + ../app/src/main/res/layout*/*.xml \ + ../app/src/main/res/values/*.xml # FIXME - when tidy learns to not leave whitespace around, remove the line below - https://github.com/htacg/tidy-html5/issues/864 - find android/app/src/main/ -name '*.xml' -exec sed -i -e 's/[ \t]*$//' '{}' ';' + find ../app/src/main/ -name '*.xml' -exec sed -i -e 's/[ \t]*$//' '{}' ';' } -# Autoformats Android XML files and returns 0 if no files were actually changed, or 1 if files were changed -function tidy-verify-xml { - tidy-up-android-xml - - if git diff --exit-code -- android/app/src/main/AndroidManifest.xml android/app/src/main/res; then +function checkDiff { + if git diff --exit-code -- ../app/src/main/AndroidManifest.xml ../app/src/main/res; then echo "Android XML files are correctly formatted" return 0 else - echo "android/app/src/main contains files that were changed, XML is not formatted properly" + echo "Android XML files are NOT correctly formatted" return 1 fi } + +main "$@" |
