Browse Source

Revert "将所有exit改为return,并在适当的地方增加return"

This reverts commit cace23b822a2270ddf2f5c9614c85e86b3cf7012.
sunshineplan 7 years ago
parent
commit
d4124dfe9b
1 changed files with 24 additions and 22 deletions
  1. 24 22
      release/install-release.sh

+ 24 - 22
release/install-release.sh

@@ -97,7 +97,7 @@ downloadV2Ray(){
     curl ${PROXY} -L -H "Cache-Control: no-cache" -o ${ZIPFILE} ${DOWNLOAD_LINK}
     curl ${PROXY} -L -H "Cache-Control: no-cache" -o ${ZIPFILE} ${DOWNLOAD_LINK}
     if [ $? != 0 ];then
     if [ $? != 0 ];then
         colorEcho ${RED} "Failed to download! Please check your network or try again."
         colorEcho ${RED} "Failed to download! Please check your network or try again."
-        return 3
+        exit 1
     fi
     fi
     return 0
     return 0
 }
 }
@@ -111,7 +111,7 @@ installSoftware(){
     getPMT
     getPMT
     if [[ $? -eq 1 ]]; then
     if [[ $? -eq 1 ]]; then
         colorEcho $YELLOW "The system package manager tool isn't APT or YUM, please install ${COMPONENT} manually."
         colorEcho $YELLOW "The system package manager tool isn't APT or YUM, please install ${COMPONENT} manually."
-        return 2 
+        exit 
     fi
     fi
     colorEcho $GREEN "Installing $COMPONENT" 
     colorEcho $GREEN "Installing $COMPONENT" 
     if [[ $SOFTWARE_UPDATED -eq 0 ]]; then
     if [[ $SOFTWARE_UPDATED -eq 0 ]]; then
@@ -124,7 +124,7 @@ installSoftware(){
     $CMD_INSTALL $COMPONENT
     $CMD_INSTALL $COMPONENT
     if [[ $? -ne 0 ]]; then
     if [[ $? -ne 0 ]]; then
         colorEcho ${RED} "Install ${COMPONENT} fail, please install it manually."
         colorEcho ${RED} "Install ${COMPONENT} fail, please install it manually."
-        return 2
+        exit
     fi
     fi
     return 0
     return 0
 }
 }
@@ -150,7 +150,7 @@ extract(){
     unzip $1 -d "/tmp/v2ray/"
     unzip $1 -d "/tmp/v2ray/"
     if [[ $? -ne 0 ]]; then
     if [[ $? -ne 0 ]]; then
         colorEcho ${RED} "Extracting V2Ray failed!"
         colorEcho ${RED} "Extracting V2Ray failed!"
-        return 2
+        exit
     fi
     fi
     return 0
     return 0
 }
 }
@@ -264,7 +264,7 @@ installInitScript(){
         fi
         fi
         return
         return
     elif [[ -n "${SERVICE_CMD}" ]] && [[ ! -f "/etc/init.d/v2ray" ]]; then
     elif [[ -n "${SERVICE_CMD}" ]] && [[ ! -f "/etc/init.d/v2ray" ]]; then
-        installSoftware "daemon" || return $?
+        installSoftware "daemon"
         cp "/tmp/v2ray/v2ray-${NEW_VER}-linux-${VDIS}/systemv/v2ray" "/etc/init.d/v2ray"
         cp "/tmp/v2ray/v2ray-${NEW_VER}-linux-${VDIS}/systemv/v2ray" "/etc/init.d/v2ray"
         chmod +x "/etc/init.d/v2ray"
         chmod +x "/etc/init.d/v2ray"
         update-rc.d v2ray defaults
         update-rc.d v2ray defaults
@@ -281,6 +281,7 @@ Help(){
     echo "  -l, --local           Install from a local file"
     echo "  -l, --local           Install from a local file"
     echo "      --remove          Remove installed V2Ray"
     echo "      --remove          Remove installed V2Ray"
     echo "  -c, --check           Check for update"
     echo "  -c, --check           Check for update"
+    exit  
 }
 }
 
 
 remove(){
 remove(){
@@ -294,11 +295,11 @@ remove(){
         rm -rf "/usr/bin/v2ray" "/etc/systemd/system/v2ray.service"
         rm -rf "/usr/bin/v2ray" "/etc/systemd/system/v2ray.service"
         if [[ $? -ne 0 ]]; then
         if [[ $? -ne 0 ]]; then
             colorEcho ${RED} "Failed to remove V2Ray."
             colorEcho ${RED} "Failed to remove V2Ray."
-            return 0
+            exit
         else
         else
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
-            return
+            exit
         fi
         fi
     elif [[ -n "${SYSTEMCTL_CMD}" ]] && [[ -f "/lib/systemd/system/v2ray.service" ]];then
     elif [[ -n "${SYSTEMCTL_CMD}" ]] && [[ -f "/lib/systemd/system/v2ray.service" ]];then
         if pgrep "v2ray" > /dev/null ; then
         if pgrep "v2ray" > /dev/null ; then
@@ -308,11 +309,11 @@ remove(){
         rm -rf "/usr/bin/v2ray" "/lib/systemd/system/v2ray.service"
         rm -rf "/usr/bin/v2ray" "/lib/systemd/system/v2ray.service"
         if [[ $? -ne 0 ]]; then
         if [[ $? -ne 0 ]]; then
             colorEcho ${RED} "Failed to remove V2Ray."
             colorEcho ${RED} "Failed to remove V2Ray."
-            return 0
+            exit
         else
         else
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
-            return
+            exit
         fi
         fi
     elif [[ -n "${SERVICE_CMD}" ]] && [[ -f "/etc/init.d/v2ray" ]]; then
     elif [[ -n "${SERVICE_CMD}" ]] && [[ -f "/etc/init.d/v2ray" ]]; then
         if pgrep "v2ray" > /dev/null ; then
         if pgrep "v2ray" > /dev/null ; then
@@ -321,15 +322,15 @@ remove(){
         rm -rf "/usr/bin/v2ray" "/etc/init.d/v2ray"
         rm -rf "/usr/bin/v2ray" "/etc/init.d/v2ray"
         if [[ $? -ne 0 ]]; then
         if [[ $? -ne 0 ]]; then
             colorEcho ${RED} "Failed to remove V2Ray."
             colorEcho ${RED} "Failed to remove V2Ray."
-            return 0
+            exit
         else
         else
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "Removed V2Ray successfully."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
             colorEcho ${GREEN} "If necessary, please remove configuration file and log file manually."
-            return
+            exit
         fi       
         fi       
     else
     else
         colorEcho ${GREEN} "V2Ray not found."
         colorEcho ${GREEN} "V2Ray not found."
-        return 0
+        exit
     fi
     fi
 }
 }
 
 
@@ -347,31 +348,31 @@ checkUpdate(){
 
 
 main(){
 main(){
     #helping information
     #helping information
+    [[ "$HELP" == "1" ]] && Help
     [[ "$CHECK" == "1" ]] && checkUpdate
     [[ "$CHECK" == "1" ]] && checkUpdate
-    [[ "$HELP" == "1" ]] && Help && return
-    [[ "$REMOVE" == "1" ]] && remove && return
+    [[ "$REMOVE" == "1" ]] && remove
     
     
     sysArch
     sysArch
     # extract local file
     # extract local file
     if [[ $LOCAL_INSTALL -eq 1 ]]; then
     if [[ $LOCAL_INSTALL -eq 1 ]]; then
         echo "Install V2Ray via local file"
         echo "Install V2Ray via local file"
-        installSoftware unzip || return $?
+        installSoftware unzip
         rm -rf /tmp/v2ray
         rm -rf /tmp/v2ray
-        extract $LOCAL || return $?
+        extract $LOCAL
         FILEVDIS=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f4`
         FILEVDIS=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f4`
         SYSTEM=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f3`
         SYSTEM=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f3`
         if [[ ${SYSTEM} != "linux" ]]; then
         if [[ ${SYSTEM} != "linux" ]]; then
             colorEcho $RED "The local V2Ray can not be installed in linux."
             colorEcho $RED "The local V2Ray can not be installed in linux."
-            return 1
+            exit
         elif [[ ${FILEVDIS} != ${VDIS} ]]; then
         elif [[ ${FILEVDIS} != ${VDIS} ]]; then
             colorEcho $RED "The local V2Ray can not be installed in ${ARCH} system."
             colorEcho $RED "The local V2Ray can not be installed in ${ARCH} system."
-            return 1
+            exit
         else
         else
             NEW_VER=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f2`
             NEW_VER=`ls /tmp/v2ray |grep v2ray-v |cut -d "-" -f2`
         fi
         fi
     else
     else
         # download via network and extract
         # download via network and extract
-        installSoftware "curl" || return $?
+        installSoftware "curl"
         getVersion
         getVersion
         if [[ $? == 0 ]] && [[ "$FORCE" != "1" ]]; then
         if [[ $? == 0 ]] && [[ "$FORCE" != "1" ]]; then
             colorEcho ${GREEN} "Latest version ${NEW_VER} is already installed."
             colorEcho ${GREEN} "Latest version ${NEW_VER} is already installed."
@@ -387,11 +388,12 @@ main(){
         V2RAY_RUNNING=1
         V2RAY_RUNNING=1
         stopV2ray
         stopV2ray
     fi
     fi
-    installV2Ray || return $?
-    installInitScript || return $?
+    installV2Ray
+    installInitScript
     if [[ ${V2RAY_RUNNING} -eq 1 ]];then
     if [[ ${V2RAY_RUNNING} -eq 1 ]];then
         colorEcho ${BLUE} "Restarting V2Ray service."
         colorEcho ${BLUE} "Restarting V2Ray service."
-        startV2ray || return $?
+        startV2ray
+
     fi
     fi
     colorEcho ${GREEN} "V2Ray ${NEW_VER} is installed."
     colorEcho ${GREEN} "V2Ray ${NEW_VER} is installed."
     rm -rf /tmp/v2ray
     rm -rf /tmp/v2ray