Move the source integrity checking into its own function as the code
was duplicated and is now more complicated with the separation of the
two checks types.

Signed-off-by: Allan McRae <[email protected]>
---
 scripts/makepkg.sh.in |   41 +++++++++++++++++------------------------
 1 files changed, 17 insertions(+), 24 deletions(-)

diff --git a/scripts/makepkg.sh.in b/scripts/makepkg.sh.in
index 296bab3..0682e4f 100644
--- a/scripts/makepkg.sh.in
+++ b/scripts/makepkg.sh.in
@@ -753,6 +753,21 @@ check_pgpsigs() {
        fi
 }
 
+check_source_integrity() {
+       if (( SKIPCHECKSUMS && SKIPPGPCHECK )); then
+               warning "$(gettext "Skipping all source file integrity 
checks.")"
+       elif (( SKIPCHECKSUMS )); then
+               warning "$(gettext "Skipping verification of source file 
checksums.")"
+               check_pgpsigs
+       elif (( SKIPPGPCHECK )); then
+               warning "$(gettext "Skipping verification of source file PGP 
signatures.")"
+               check_checksums
+       else
+               check_checksums
+               check_pgpsigs
+       fi
+}
+
 extract_sources() {
        msg "$(gettext "Extracting Sources...")"
        local netfile
@@ -2214,18 +2229,7 @@ if (( SOURCEONLY )); then
                        (( SOURCEONLY == 2 )); then
                download_sources
        fi
-       if (( SKIPCHECKSUMS && SKIPPGPCHECK )); then
-               warning "$(gettext "Skipping all source file integrity 
checks.")"
-       elif (( SKIPCHECKSUMS )); then
-               warning "$(gettext "Skipping verification of source file 
checksums.")"
-               check_pgpsigs
-       elif (( SKIPPGPCHECK )); then
-               warning "$(gettext "Skipping verification of source file PGP 
signatures.")"
-               check_checksums
-       else
-               check_checksums
-               check_pgpsigs
-       fi
+       check_source_integrity
        cd "$startdir"
 
        # if we are root or if fakeroot is not enabled, then we don't use it
@@ -2299,18 +2303,7 @@ elif (( REPKG )); then
        fi
 else
        download_sources
-       if (( SKIPCHECKSUMS && SKIPPGPCHECK )); then
-               warning "$(gettext "Skipping all source file integrity 
checks.")"
-       elif (( SKIPCHECKSUMS )); then
-               warning "$(gettext "Skipping verification of source file 
checksums.")"
-               check_pgpsigs
-       elif (( SKIPPGPCHECK )); then
-               warning "$(gettext "Skipping verification of source file PGP 
signatures.")"
-               check_checksums
-       else
-               check_checksums
-               check_pgpsigs
-       fi
+       check_source_integrity
        extract_sources
 fi
 
-- 
1.7.6


Reply via email to