makedeb

@@ -6,32 +6,10 @@ buildfile="${workspace}/DEBBUILD"
 srcdir="${workspace}/src"
 pkgdir="${workspace}/pkg"
 
-_IN_FAKEROOT=0
-_BACKUP_STDOUT=3        # use file descriptor 3 to backup stdout
-_BACKUP_STDERR=4        # use file descriptor 4 to backup stderr
-_ARGLIST=("$@")
+FAKE_PACKAGE=0
 OVERRIDE_SOURCE=0
+SHOW_TARGET_FILE=0
 QUIET=0
-PACKAGELIST=0
-IS_DYNAMICV_PKGVER=0
-
-_STEP_CLEAN=1
-_STEP_RETRIEVE_SOURCE=1
-_STEP_EXTRACT_SOURCE=1
-_STEP_UPDATE_PKGVER=1
-_STEP_CHCECK_PACKAGE_EXISTS=1
-_STEP_BUILD=1
-_STEP_PACKAGE=1
-_STEP_CREATE_ARCHIVE=1
-
-_STOP_AFTER_CLEAN=0
-_STOP_AFTER_RETRIEVE_SOURCE=0
-_STOP_AFTER_EXTRACT_SOURCE=0
-_STOP_AFTER_UPDATE_PKGVER=0
-#_STEP_CHCECK_PACKAGE_EXISTS=0  # stop after checking exists is useless
-_STOP_AFTER_BUILD=0
-_STOP_AFTER_PACKAGE=0
-_STOP_AFTER_CREATE_ARCHIVE=0
 
 set -o functrace
 set -o nounset
@@ -72,15 +50,10 @@ function msg2 {
     local mesg=$1; shift
     printf "${BLUE}  ->${ALL_OFF}${BOLD} ${mesg}${ALL_OFF}\n" "$@"
 }
-function warn {
-    (( QUIET )) && return 0
-    local mesg=$1; shift
-    printf "${YELLOW}==>${ALL_OFF}${BOLD}${YELLOW} ${mesg}${ALL_OFF}\n" "$@" >&2
-}
 function err {
     (( QUIET )) && return 0
     local mesg=$1; shift
-    printf "${RED}==>${ALL_OFF}${BOLD}${RED} ${mesg}${ALL_OFF}\n" "$@" >&2
+    printf "${RED}  ->${ALL_OFF}${BOLD}${RED} ${mesg}${ALL_OFF}\n" "$@" >&2
 }
 
 function get_full_version {
@@ -113,24 +86,11 @@ function debian_control {
     [[ -n "${priority:=optional}" ]] && echo "Priority: ${priority}"
 
     echo "Description: ${pkgdesc}"
+
 }
 
-function generate_control {
-    msg "Generating control info..."
-    mkdir -p "${pkgdir}/DEBIAN"
-    echo 2 > "${pkgdir}/DEBIAN/compat"
-    debian_control > "${pkgdir}/DEBIAN/control"
-
-    function debian_hooks_warpper { is_function "$1" && "$1" > "$2" && chmod +x "$2" || true; }
-    debian_hooks_warpper debian_preinst  ${pkgdir}/DEBIAN/preinst
-    debian_hooks_warpper debian_postinst ${pkgdir}/DEBIAN/postinst
-    debian_hooks_warpper debian_prerm    ${pkgdir}/DEBIAN/prerm
-    debian_hooks_warpper debian_postrm   ${pkgdir}/DEBIAN/postrm
-}
-
-
 function generate_deb {
-    msg "Generating deb package..."
+    msg "generating deb package..."
 
     local tmpdir="$(mktemp --directory)"
     local data_tgz="${tmpdir}/data.tar.gz"
@@ -147,12 +107,7 @@ function generate_deb {
 
     rm -rf "${tmpdir}"
 
-    msg2 "Generated deb: %s" "$(get_deb_name)"
-}
-
-function step_create_archive {
-    generate_control
-    generate_deb
+    msg2 "generated deb: %s" "$(get_deb_name)"
 }
 
 function url_type {
@@ -183,12 +138,10 @@ function create_soft_link {
 
     if [[ ! -e "${src}" ]] ; then
         err "soft_link src %s not exist" "${src}"
-        return 1
     fi
 
     if [[ -e "${tgt}" ]] ; then
         err "soft_link tgt %s already exist" "${tgt}"
-        return 1
     fi
 
     ln --symbolic "${src}" "${tgt}"
@@ -205,17 +158,17 @@ function url_fragment {
 
 # $1 is relative path to file
 # $2 is url/(relative path to materials)
-function retrieve_source_single {
+function retrive_source_single {
     local file_name="$1"
     local url="$2"
 
     case "$(url_type "${url}")" in
         "git")
             if [[ -d "${file_name}" ]]; then
-                msg2 "Updating ${file_name} from ${url} with git..."
+                msg2 "updating ${file_name} from ${url} with git..."
                 git --git-dir="${workspace}/${file_name}" fetch --all
             else
-                msg2 "Cloning ${file_name} from ${url} with git..."
+                msg2 "cloning ${file_name} from ${url} with git..."
                 local git_source="${url##git+}"
                 git_source="${git_source%%#*}"
                 git clone --mirror "${git_source}" "${workspace}/${file_name}"
@@ -225,21 +178,21 @@ function retrieve_source_single {
             if (( ! OVERRIDE_SOURCE )) && [[ -f "${workspace}/${file_name}" ]] ; then
                 msg2 "${file_name} already exists, skip download"
             else
-                msg2 "Retrieving ${file_name} from ${url} with curl..."
+                msg2 "retriving ${file_name} from ${url} with curl..."
                 curl --location "${url}" --output "${workspace}/${file_name}"
             fi
             ;;
         "file")
             if [[ -e "${workspace}/${url}" ]] ; then
-                msg2 "Found local file: ${url}"
+                msg2 "found local file: ${url}"
             else
-                err "Local file not found: ${url}"
+                err "local file not found: ${url}"
                 err "Aborting..."
                 exit 1
             fi
             ;;
         *)
-            err "Retrieving url as type ${url_type} not supported"
+            err "retriving url as type ${url_type} not supported"
             err "Aborting..."
             exit 1
     esac
@@ -261,7 +214,7 @@ function extract_source_single {
             if [[ -n "${frag}" ]] ; then
                 case "${frag%%=*}" in
                     "branch")
-                        ref="refs/remotes/origin/${frag##*=}"
+                        ref="refs/heads/${frag##*=}"
                         ;;
                     "tag")
                         ref="refs/tags/${frag##*=}"
@@ -276,7 +229,7 @@ function extract_source_single {
                 esac
             fi
 
-            msg2 "Extracting git ${workspace}/${file_name} with reference ${ref}"
+            msg2 "extracting git ${workspace}/${file_name} with reference ${ref}"
             git clone --shared "${workspace}/${file_name}" "${srcdir}/${file_name}"
             git -C "${srcdir}/${file_name}" switch --force-create makedeb --no-track "${ref}"
             ;;
@@ -308,11 +261,11 @@ function decompress_source_file {
         *.zip)
             cmd="unzip" ;;
         *)
-            msg2 "No need to decompress, skip %s" "${file}"
+            msg2 "no need to decompress, skip %s" "${file}"
             return
     esac
 
-    msg2 "Decompressing %s with %s" "${file}" "${cmd}"
+    msg2 "decompressing %s with %s" "${file}" "${cmd}"
     local res=0
     case "$cmd" in
         "tar")
@@ -370,24 +323,24 @@ function check_source_validation {
     done
 }
 
-function step_clean_dir {
-    msg "Cleaning \$srcdir and \$pkgdir..."
+function clean_dir {
+    msg "cleaning \$srcdir and \$pkgdir..."
     rm -rf "${srcdir}" "${pkgdir}"
     mkdir -p "${srcdir}" "${pkgdir}"
 }
 
-function step_retrieve_source {
-    msg "Retrieving source..."
+function retrieve_source {
+    msg "retrieving source..."
     for s in "${source[@]}"; do
         file_name="${s%%::*}"
         url="${s##*::}"
 
-        retrieve_source_single "${file_name}" "${url}"
+        retrive_source_single "${file_name}" "${url}"
     done
 }
 
-function step_extract_source {
-    msg "Extracting source..."
+function extract_source {
+    msg "extracting source..."
     for s in "${source[@]}"; do
         file_name="${s%%::*}"
         url="${s##*::}"
@@ -396,12 +349,12 @@ function step_extract_source {
     done
 }
 
-function step_update_pkgver {
+function update_pkgver {
     if ! is_function pkgver; then
         return
     fi
 
-    msg "Updating pkgver..."
+    msg "updating pkgver..."
     newpkgver="$(run_function_safe pkgver)"
     if [[ "${newpkgver}" != "${pkgver:-}" ]] ; then
         mapfile -t bfcontent < "${buildfile}"
@@ -418,41 +371,33 @@ function step_update_pkgver {
 
         source "${buildfile}"
 
-        msg2 "Updated version: ${newpkgver}"
+        msg2 "updated version: ${newpkgver}"
     fi
 
     pkgver="${newpkgver}"
 }
 
+function generate_control {
+
+    msg "generating control info..."
+    mkdir -p "${pkgdir}/DEBIAN"
+    echo 2 > "${pkgdir}/DEBIAN/compat"
+    debian_control > "${pkgdir}/DEBIAN/control"
+
+    function debian_hooks_warpper { is_function "$1" && "$1" > "$2" && chmod +x "$2" || true; }
+    debian_hooks_warpper debian_preinst  ${pkgdir}/DEBIAN/preinst
+    debian_hooks_warpper debian_postinst ${pkgdir}/DEBIAN/postinst
+    debian_hooks_warpper debian_prerm    ${pkgdir}/DEBIAN/prerm
+    debian_hooks_warpper debian_postrm   ${pkgdir}/DEBIAN/postrm
+}
+
 function run_function {
     if is_function "$1" ; then
-        msg "Run function: $1"
+        msg "run function: $1"
         run_function_safe "$1"
     fi
 }
 
-function disable_stdout {
-    exec {_BACKUP_STDOUT}>&1
-    exec 1>/dev/null
-}
-
-function enable_stdout {
-    exec 1>&"$_BACKUP_STDOUT"
-}
-
-function show_help {
-    echo "$0 [OPTIONS]"
-    echo "      -Q                      quiet, disable log"
-    echo "      -f|--force              don't check if package already exists, will"
-    echo "      --nobuild               retrieve source, update pkgver and stop"
-    echo "                              override the existed package"
-    echo "      --noextract             use the current srcdir to build package, skip"
-    echo "                              those retrieve source steps"
-    echo "      --noarchive             retrieve and build, but don't package to .deb"
-    echo "      --packagelist           show file to be generated by current DEBBUILD"
-    echo "      --is-dynamic-pkgver     exit with 0 if DEBBUILD will update pkgver later"
-    echo "      -h|--help               show this message"
-}
 
 ##
 ## Here start the build logic
@@ -460,36 +405,11 @@ function show_help {
 
 while (( "$#" >= 1 )); do
     case "$1" in
-        -F)                         _IN_FAKEROOT=1 ;;
-        -Q)                         QUIET=1 ;;
-        -f|--force)                 _STEP_CHCECK_PACKAGE_EXISTS=0 ;;
-        --noextract)
-            _STEP_CLEAN=0
-            _STEP_RETRIEVE_SOURCE=0
-            _STEP_EXTRACT_SOURCE=0
-            _STEP_UPDATE_PKGVER=0
-            ;;
-        --nobuild)
-            _STEP_CHCECK_PACKAGE_EXISTS=0
-            _STEP_BUILD=0
-            _STEP_PACKAGE=0
-            _STEP_CREATE_ARCHIVE=0
-
-            _STOP_AFTER_UPDATE_PKGVER=1
-            ;;
-        --noarchive)
-            _STEP_CHCECK_PACKAGE_EXISTS=0
-            _STEP_CREATE_ARCHIVE=0
-
-            _STOP_AFTER_BUILD=1
-            ;;
-        --packagelist)              PACKAGELIST=1 ;;
-        --is-dynamic-pkgver)        IS_DYNAMICV_PKGVER=1 ;;
-        -h|--help) show_help ; exit 0;;
-
-        *)  err "Unkown option $1"
-            err "Use $0 --help for help"
-            exit 1 ;;
+        -F)  FAKE_PACKAGE=1 ;;
+        -OS) OVERRIDE_SOURCE=1 ;;
+        -STF) SHOW_TARGET_FILE=1 ; QUIET=1 ;;
+        -Q) QUIET=1 ;;
+        *)   err "Unkown option $1"; break ;;
     esac
     shift
 done
@@ -497,75 +417,39 @@ done
 source "${buildfile}"
 
 
-if (( PACKAGELIST )) ; then
-    echo "$(get_deb_name)"
-    exit 0
-fi
-
-if (( IS_DYNAMICV_PKGVER )) ; then
-    if is_function pkgver ; then
+if (( ! FAKE_PACKAGE )) ; then
+    if (( SHOW_TARGET_FILE )) && ! is_function pkgver ; then
+        echo "$(get_deb_name)"
         exit 0
-    else
-        exit 4
     fi
-fi
 
-if (( QUIET )) ; then
-    disable_stdout
-fi
-
-if (( ! _IN_FAKEROOT )) ; then
     check_source_validation
 
-    if (( _STEP_CLEAN )) ; then
-        step_clean_dir
+    clean_dir
+
+    retrieve_source
+
+    extract_source
+
+    update_pkgver
+
+    if (( SHOW_TARGET_FILE )) && is_function pkgver ; then
+        echo "$(get_deb_name)"
+        exit 0
     fi
 
-    if (( _STEP_RETRIEVE_SOURCE )) ; then
-        step_retrieve_source
-    fi
+    run_function build
 
-    if (( _STEP_EXTRACT_SOURCE )) ; then
-        step_extract_source
-    fi
+    # recursive call self to run rest task in fakeroot
+    msg "entering fakeroot environment..."
+    fakeroot -- bash -$- "${BASH_SOURCE[0]}" -F "${ARGLIST[@]}" || exit $?
+    msg "leaving fakeroot environment..."
 
-    if (( _STEP_UPDATE_PKGVER )) ; then
-        step_update_pkgver
-        (( _STOP_AFTER_UPDATE_PKGVER )) && msg "Sources are ready."
-    fi
-
-    # Check if the package already exists
-    if (( _STEP_CHCECK_PACKAGE_EXISTS )) ; then
-        if [[ -f "$(get_deb_name)" ]] ; then
-            warn "%s %s" \
-                    "The package $(get_deb_name) already exists, skipping build." \
-                    "(use -f to force re-build)"
-            exit 0
-        fi
-    fi
-
-    if (( _STEP_BUILD )) ; then
-        run_function build
-        (( _STOP_AFTER_BUILD )) && msg "Package directory is ready."
-    fi
-
-    if (( _STEP_PACKAGE || _STEP_CREATE_ARCHIVE )) ; then
-        # recursive call self to run rest task in fakeroot
-        msg "Entering fakeroot environment..."
-        fakeroot -- bash -$- "${BASH_SOURCE[0]}" -F "${_ARGLIST[@]}" || exit $?
-        msg "Leaving fakeroot environment..."
-    fi
+    msg "builds has done"
 else
-    if (( _STEP_PACKAGE )) ; then
-        run_function package
-    fi
+    run_function package
 
-    if (( _STEP_CREATE_ARCHIVE )) ; then
-        step_create_archive
-        msg "Finish making: $(get_deb_name) $(date)"
-    fi
-fi
+    generate_control
 
-if (( QUIET )) ; then
-    enable_stdout
+    generate_deb
 fi