fix: Don't rm -f on Flatpak lists before creating
There won't be anything to remove unless the module is called more than once, and if it's called more than once, the user would expect every Flatpak to be in the respective lists, so there's no need to call rm -f on it.
This commit is contained in:
parent
6ad9056d20
commit
634cf88d23
1 changed files with 4 additions and 4 deletions
|
|
@ -25,7 +25,7 @@ get_yaml_array USER_REMOVE '.user.remove[]' "$1"
|
|||
|
||||
echo "Creating system Flatpak install list"
|
||||
if [[ ${#SYSTEM_INSTALL[@]} -gt 0 ]]; then
|
||||
rm -f $SYS_INSTALL_LIST && touch $SYS_INSTALL_LIST
|
||||
touch $SYS_INSTALL_LIST
|
||||
for flatpak in "${SYSTEM_INSTALL[@]}"; do
|
||||
echo "Adding to system flatpak installs: $(printf ${flatpak})"
|
||||
echo $flatpak >> $SYS_INSTALL_LIST
|
||||
|
|
@ -34,7 +34,7 @@ fi
|
|||
|
||||
echo "Creating system Flatpak removals list"
|
||||
if [[ ${#SYSTEM_REMOVE[@]} -gt 0 ]]; then
|
||||
rm -f $SYS_REMOVE_LIST && touch $SYS_REMOVE_LIST
|
||||
touch $SYS_REMOVE_LIST
|
||||
for flatpak in "${SYSTEM_REMOVE[@]}"; do
|
||||
echo "Adding to system flatpak removals: $(printf ${flatpak})"
|
||||
echo $flatpak >> $SYS_REMOVE_LIST
|
||||
|
|
@ -43,7 +43,7 @@ fi
|
|||
|
||||
echo "Creating user Flatpak install list"
|
||||
if [[ ${#USER_INSTALL[@]} -gt 0 ]]; then
|
||||
rm -f $USER_INSTALL_LIST && touch $USER_INSTALL_LIST
|
||||
touch $USER_INSTALL_LIST
|
||||
for flatpak in "${USER_INSTALL[@]}"; do
|
||||
echo "Adding to user flatpak installs: $(printf ${flatpak})"
|
||||
echo $flatpak >> $USER_INSTALL_LIST
|
||||
|
|
@ -52,7 +52,7 @@ fi
|
|||
|
||||
echo "Creating user Flatpak removals list"
|
||||
if [[ ${#USER_REMOVE[@]} -gt 0 ]]; then
|
||||
rm -f $USER_REMOVE_LIST && touch $USER_REMOVE_LIST
|
||||
touch $USER_REMOVE_LIST
|
||||
for flatpak in "${USER_REMOVE[@]}"; do
|
||||
echo "Adding to user flatpak removals: $(printf ${flatpak})"
|
||||
echo $flatpak >> $USER_REMOVE_LIST
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue