Skip to content

Commit

Permalink
Strip double quotes from umask check (#2632)
Browse files Browse the repository at this point in the history
  • Loading branch information
pkishino authored Jun 15, 2023
1 parent 1103172 commit fad0ad1
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion transmission/userSetup.sh
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ if [ -n "$PUID" ] && [ ! "$(id -u root)" -eq "$PUID" ]; then
if [ -z "$TRANSMISSION_UMASK" ] ; then
# fetch from settings.json if not defined in environment
# because updateSettings.py is called after this script is run
TRANSMISSION_UMASK=$(jq .umask ${TRANSMISSION_HOME}/settings.json)
TRANSMISSION_UMASK=$(jq .umask ${TRANSMISSION_HOME}/settings.json | tr -d \" )

This comment has been minimized.

Copy link
@richard087

richard087 Sep 5, 2023

could be simpler
TRANSMISSION_UMASK=$(jq -r .umask ${TRANSMISSION_HOME}/settings.json )

I came to make this same fix. Thanks for your hard work.

fi

TRANSMISSION_UMASK_OCTAL=$( printf "%o\n" "${TRANSMISSION_UMASK}" )
Expand Down

0 comments on commit fad0ad1

Please sign in to comment.