# HG changeset patch
# User "Yann E. MORIN" <yann.morin.1998@anciens.enib.fr>
# Date 1259535945 -3600
# Node ID 9030e84dafad478e4a393f608a5fb1195de86469
# Parent  184a960d2fc88757c237ef640eff135ee3ca57aa
scripts: handle custom or empty extension when extracting

diff -r 184a960d2fc8 -r 9030e84dafad scripts/functions
--- a/scripts/functions	Wed Nov 25 20:36:04 2009 +0100
+++ b/scripts/functions	Mon Nov 30 00:05:45 2009 +0100
@@ -297,8 +297,8 @@
 
 # Get the file name extension of a component
 # Usage: CT_GetFileExtension <component_name-component_version> [extension]
-# If found, echoes the extension to stdout
-# If not found, echoes nothing on stdout.
+# If found, echoes the extension to stdout, and return 0
+# If not found, echoes nothing on stdout, and return !0.
 CT_GetFileExtension() {
     local ext
     local file="$1"
@@ -311,11 +311,11 @@
     for ext in ${first_ext} .tar.gz .tar.bz2 .tgz .tar ''; do
         if [ -f "${CT_TARBALLS_DIR}/${file}${ext}" ]; then
             echo "${ext}"
-            break
+            exit 0
         fi
     done
 
-    return 0
+    exit 1
 }
 
 # Download an URL using wget
@@ -389,8 +389,7 @@
     local ext
 
     # Do we already have it in *our* tarballs dir?
-    ext=$(CT_GetFileExtension "${basename}" ${first_ext})
-    if [ -n "${ext}" ]; then
+    if ext="$( CT_GetFileExtension "${basename}" ${first_ext} )"; then
         CT_DoLog DEBUG "Already have '${basename}'"
         return 0
     fi
@@ -571,8 +570,11 @@
 CT_Extract() {
     local basename="$1"
     local nochdir="$2"
-    local ext=$(CT_GetFileExtension "${basename}")
-    CT_TestAndAbort "'${basename}' not found in '${CT_TARBALLS_DIR}'" -z "${ext}"
+    local ext
+
+    if ! ext="$(CT_GetFileExtension "${basename}")"; then
+      CT_TestAndAbort "'${basename}' not found in '${CT_TARBALLS_DIR}'" -z "${ext}"
+    fi
     local full_file="${CT_TARBALLS_DIR}/${basename}${ext}"
 
     # Check if already extracted