quilt-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Quilt-dev] [PATCH 12/39] backup-files: Simplify find loop


From: Jean Delvare
Subject: [Quilt-dev] [PATCH 12/39] backup-files: Simplify find loop
Date: Sat, 19 Mar 2011 10:52:42 +0100
User-agent: quilt/0.48-17.1

Let find report errors by itself.

Signed-off-by: Jean Delvare <address@hidden>
Reviewed-by: Raphael Hertzog <address@hidden>
---
 quilt/scripts/backup-files.in |   15 +++------------
 test/delete.test              |    2 +-
 2 files changed, 4 insertions(+), 13 deletions(-)

--- a/quilt/scripts/backup-files.in
+++ b/quilt/scripts/backup-files.in
@@ -193,21 +193,12 @@ while [ $I -lt address@hidden ]; do
 
        case "${FILELIST[$I]}" in
        -)
-               find "$OPT_PREFIX" -mindepth 1 \( -type f -o -type d \) -print 
2>/dev/null \
+               find "$OPT_PREFIX" -type f -print \
                | while read
                do
-                       if [ -d "$REPLY" ]
-                       then
-                               if ! [ -r "$REPLY" ] || ! [ -x "$REPLY" ]
-                               then
-                                       echo "$REPLY: Permission denied"
-                                       exit 1
-                               fi
-                       else
-                               process_file "${REPLY#$OPT_PREFIX}"
-                       fi
+                       process_file "${REPLY#$OPT_PREFIX}"
                done
-               if [ $? != 0 ]; then
+               if [ ${PIPESTATUS[0]} != 0 ]; then
                        exit 1
                fi
                ;;
--- a/test/delete.test
+++ b/test/delete.test
@@ -77,7 +77,7 @@ Test the delete command.
 
        $ quilt delete "test3"
        > Removing patch %{P}test3
-       > .pc/test3/dir: Permission denied
+       >~ find: `?\.pc/test3/dir'?: Permission denied
 
        $ chmod a+rx .pc/test3/dir
 




reply via email to

[Prev in Thread] Current Thread [Next in Thread]