Bash Script to run a backup and upload to AWS S3 Failing - bash

Question

Here's the code:
#!/bin/bash
# -------------------------------------------------
# -------------------------------------------------
# Use a comma-delimited list of single-quoted
# strings of the usernames to batch
# if it's empty it will backup all user directories
# in /home
# -------------------------------------------------
# -------------------------------------------------
USER_ACCOUNT=();
# -------------------------------------------------
# -------------------------------------------------
# Make sure the aws script is installed on the
# server, and the bucket name to upload these too
# are exact... case-sensitive
# -------------------------------------------------
# -------------------------------------------------
S3BUCKET='Kevs-Bucket/Test';
# Loop through the user array
# If it's empty, then get all users in the /home
# directory, based on each folder
# do not include root user
if [ ${#USER_ACCOUNT[#]} -eq 0 ]; then
# turn off dotglob and nullglob
cd /home;
shopt -s dotglob;
shopt -s nullglob;
DIRARR=(*/);
# we have our directories, now loop them and grab the user
# once we have the user, skip the root user
for d in ${!DIRARR[#]}; do
# Assign an account variable
ACCT=stat -c '%U' ${DIRARR[$i]}; #NOT WORKING HERE
if [ "$ACCT" == "root" ]; then
echo "ROOT";
else
run_backup $ACCT $S3BUCKET;
fi;
done;
else
# we have our list, now loop through them all
for i in ${!USER_ACCOUNT[#]}; do
# Assign an account variable
ACCT=${USER_ACCOUNT[$i]};
run_backup $ACCT $S3BUCKET;
done;
fi;
# -------------------------------------------------
# -------------------------------------------------
# Run the actual backup
run_backup(){
LOGFILE=/batch-move.log
# Package the account
./scripts/pkgacct $1;
echo '##########################################' >> $LOGFILE;
echo "# Start: date +'%T'" >> $LOGFILE;
echo "# Backing Up: $1" >> $LOGFILE;
echo '##########################################' >> $LOGFILE;
# Upload it to S3
s3put $2/cpmove-$1.tar.gz /home/cpmove-$1.tar.gz;
echo '##########################################' >> $LOGFILE;
echo "# Uploading Backup: $1" >> $LOGFILE;
echo '##########################################' >> $LOGFILE;
# Remove the file from the server
/bin/rm -f /home/cpmove-$1.tar.gz;
echo '##########################################' >> $LOGFILE;
echo "# Removing Backup Up: $1" >> $LOGFILE;
echo "# Finish: date +'%T'" >> $LOGFILE;
echo '##########################################' >> $LOGFILE;
}
I'm getting an error here ACCT=stat -c '%U' ${DIRARR[$i]}; #NOT WORKING HERE and the error is stating that -c is not a valid option for stat on my CentOS server
I have verified through other means that stat -c does work, so I assume that my code attempting to get the folders owner into a variable is incorrect.
Can you help me figure it out?

Answer 1

The line that's not working (below) contains the variable $i which you have not defined and the $() notation is not present, see added code below "EDIT - Try this".
ACCT=stat -c '%U' ${DIRARR[$i]}; #NOT WORKING HERE
You are looping through your arrays in an unusual way. Here is an example of how to loop through the elements of an array of filenames in Bash.
files=( "/home/User/FileName1" "/home/User/FileName2" "/home/User/FileName3" )
for fileName in "${files[#]}" ; do
echo "$fileName"
done
Also instead of using globbing to build your array – DIRARR=(*/); – you might want to consider using a loop to iterate over the files, for example:
for fileName in /home/* ; do
echo "$fileName"
done
Hope this helps.
EDIT - Try this:
Note: On my system the following ignores '.' and '..'.
# To avoid confusion 'ACCT' would be better named as 'OWNER'.
# Loop through the files in: /home/
for filename in /home/* ; do
# Get the owner of $filename.
ACCT=$(stat -c '%U' "$filename")
# If the file is a directory NOT owned by root, run backup.
if [ -d "$filename" -a "$ACCT" != "root" ]; then
# Uncomment when satisfied
# run_backup "$ACCT" "$S3BUCKET"
echo "Run backup - not owned by root:"
echo "filename: $filename owner: $ACCT"
# If the file is a directory owned by root, DO NOT run backup.
elif [ -d "$filename" -a "$ACCT" = "root" ]; then
# Remove elif clause when satisfied.
echo "Do not run backup - owned by root:"
echo "filename: $filename owner: $ACCT"
fi
done
Please note use of $() in the "ACCT=" line.