pm_patch.env: two bug fixes about creating the patch backup

Signed-off-by: Roberto A. Foglietta <roberto.foglietta@gmail.com>
robang74 · Jul 2, 2023 · 932fee1 · 932fee1
1 parent 4fd9e3b
commit 932fee1
Showing 1 changed file with 10 additions and 2 deletions.
diff --git a/src/tools/pm_patch.env b/src/tools/pm_patch.env
@@ -126,7 +126,10 @@ remove_text_patch() {
   log "Revert patch"
   logbar
   log
-  pm_patch -R --no-backup-if-mismatch && success
+  pm_patch -R --no-backup-if-mismatch || failure
+  # RAF, BUG FIX: here below the logic of the original code which exits when
+  #               succed but inhibiting the back-up of the applied patch
+  # pm_patch -R --no-backup-if-mismatch && success
 }
 
 clean_backup_patch() {
@@ -229,7 +232,12 @@ install_text_patch() {
   log "Apply patch"
   logbar
   log
-  pm_patch --no-backup-if-mismatch && success
+  pm_patch --no-backup-if-mismatch || failure
+  # RAF, BUG FIX: here below the logic of the original code which exits when
+  #               succed but inhibiting the back-up of the applied patch
+  #               Morover, it is sane to exit in case of failure to avoid the
+  #               patch backup as the code above is doing
+  # pm_patch --no-backup-if-mismatch && success
 }
 
 create_backup_patch() {