Skip to content

Commit 566eab1

Browse files
author
Aaron Roller
committed
fix: avoid failure when no amros user AM-172/amros-user-check
1 parent 6d30f17 commit 566eab1

1 file changed

Lines changed: 10 additions & 1 deletion

File tree

debian/postinst

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,13 @@ set -e
77

88
# provides the prod_user we use
99
include_common(){
10+
# import amros_log_dir, amros_log_media_dir
1011
source "/opt/ros/melodic/lib/am_platform_scripts/scripts/amros/amros-constants.sh"
12+
# import user_exists
13+
source "/opt/ros/melodic/lib/am_platform_scripts/scripts/common/common.sh"
1114
}
1215

16+
1317
# creates a log directory granting access
1418
# outputs a message to the screen trace upon installation
1519
create_log_directory(){
@@ -25,7 +29,12 @@ create_log_directory(){
2529
echo "$OUTPUT_PREFIX '$dir' exists to receive logs"
2630
else
2731
mkdir -p "$dir"
28-
chown "$user_group:$user_group" "$dir"
32+
if [[ $(user_exists $user) ]];then
33+
chown "$user_group:$user_group" "$dir"
34+
else
35+
echo "WARNING: $user user does not exist. Making $dir global writeable."
36+
chmod +777 "$dir"
37+
fi
2938
echo "$OUTPUT_PREFIX '$dir' created to receive logs"
3039
fi
3140
}

0 commit comments

Comments
 (0)