Merge branch 'master' of https://github.com/awalsh128/cache-apt-pkgs-action

Author: awalsh128

.github/workflows/build-distribute.yml

@@ -6,6 +6,11 @@ on:
     branches:
       - dev-v2
   workflow_call:
+    inputs:
+      create_release:
+        description: 'Create a GitHub release (disabled by default for dry-run/PR builds)'
+        default: false
+        type: boolean
 permissions:
   contents: write
   id-token: write
@@ -73,7 +78,7 @@ jobs:
   create-release:
     needs: build-and-release
     runs-on: ubuntu-latest
-    if: success()
+    if: success() && (github.event_name != 'workflow_call' || inputs.create_release == true)
     steps:
       - name: Checkout
         uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0 # v7.0.0

.github/workflows/shellcheck.yml

@@ -155,84 +155,3 @@ For more context and information see [issue #57](https://github.com/awalsh128/ca
 ### Cache Limits
 
 A repository can have up to 5GB of caches. Once the 5GB limit is reached, older caches will be evicted based on when the cache was last accessed. Caches that are not accessed within the last week will also be evicted. To get more information on how to access and manage your actions's caches, see [GitHub Actions / Using workflows / Cache dependencies](https://docs.github.com/en/actions/using-workflows/caching-dependencies-to-speed-up-workflows#viewing-cache-entries).
-
-## Development
-
-### Prerequisites
-
-- **Go 1.20+** (for building the `apt_query` binary) - version specified in `go.mod`
-  - Install from [golang.org](https://golang.org/dl/) or via package manager
-  - Verify installation: `go version`
-- **ShellCheck** (for linting shell scripts) - install via:
-  - macOS: `brew install shellcheck`
-  - Linux: `sudo apt-get install shellcheck` or see [shellcheck installation guide](https://github.com/koalaman/shellcheck#installing)
-  - Windows: Available via [scoop](https://scoop.sh/) or [chocolatey](https://chocolatey.org/)
-
-### Building
-
-The project includes Go binaries (`apt_query-arm64` and `apt_query-x86`) that are used by the shell scripts to query APT package information.
-
-**Build all packages:**
-```bash
-go build -v ./...
-```
-
-**Build for specific architecture:**
-```bash
-# For ARM64 (Apple Silicon, ARM servers)
-GOARCH=arm64 go build -o apt_query-arm64 ./src/cmd/apt_query
-
-# For x86_64 (Intel/AMD)
-GOARCH=amd64 go build -o apt_query-x86 ./src/cmd/apt_query
-```
-
-**Run tests:**
-```bash
-go test -v ./...
-```
-
-### Linting
-
-This project uses [ShellCheck](https://github.com/koalaman/shellcheck) to ensure shell script quality and catch common errors. The configuration is stored in `.shellcheckrc`.
-
-**Run ShellCheck locally:**
-```bash
-shellcheck *.sh
-```
-
-**IDE Integration:**
-
-Many IDEs and editors can automatically run ShellCheck:
-
-- **VS Code**: Install the [ShellCheck extension](https://marketplace.visualstudio.com/items?itemName=timonwong.shellcheck)
-- **Vim/Neovim**: Use [ALE](https://github.com/dense-analysis/ale) or [coc-shellcheck](https://github.com/josa42/coc-shellcheck)
-- **IntelliJ/CLion**: ShellCheck is integrated in recent versions
-- **Sublime Text**: Install [SublimeLinter-shellcheck](https://github.com/SublimeLinter/SublimeLinter-shellcheck)
-
-**Go Linting:**
-
-This project uses [golangci-lint](https://golangci-lint.run/) for Go code quality checks.
-
-**Run golangci-lint locally:**
-```bash
-# Install golangci-lint (if not already installed)
-# macOS: brew install golangci-lint
-# Linux: See https://golangci-lint.run/usage/install/
-
-golangci-lint run
-```
-
-**IDE Integration:**
-
-- **VS Code**: Install the [Go extension](https://marketplace.visualstudio.com/items?itemName=golang.go) for syntax highlighting, auto-completion, and built-in linting
-- **IntelliJ/GoLand**: Built-in Go support with linting and formatting
-- **Vim/Neovim**: Use [vim-go](https://github.com/fatih/vim-go) for Go development
-
-### CI/CD
-
-The GitHub Actions workflows will automatically:
-- **Build and test** Go code on pull requests
-- **Run ShellCheck** on shell scripts (blocks PRs on failures)
-- **Run golangci-lint** on Go code (blocks PRs on failures)
-
-All checks run on pull requests and pushes to `master`, `dev`, and `staging` branches.

.shellcheckrc

@@ -19,7 +19,7 @@ cache_dir="${1}"
 add_repository="${3}"
 
 # List of the packages to use.
-input_packages="${*:4}"
+input_packages="${@:4}"
 
 if ! apt-fast --version > /dev/null 2>&1; then
   log "Installing apt-fast for optimized installs..."
@@ -31,7 +31,7 @@ if ! apt-fast --version > /dev/null 2>&1; then
 fi
 
 # Add custom repositories if specified
-if [[ -n "${add_repository}" ]]; then
+if [ -n "${add_repository}" ]; then
   log "Adding custom repositories..."
   for repository in ${add_repository}; do
     log "- Adding repository: ${repository}"
@@ -71,10 +71,7 @@ install_log_filepath="${cache_dir}/install.log"
 
 log "Clean installing ${package_count} packages..."
 # Zero interaction while installing or upgrading the system via apt.
-# Note: sudo doesn't affect redirects, but we want the output in the file anyway
-# shellcheck disable=SC2024
-# We intentionally redirect output here; the redirect happens as the current user which is fine
-sudo DEBIAN_FRONTEND=noninteractive apt-fast --yes install "${packages}" > "${install_log_filepath}"
+sudo DEBIAN_FRONTEND=noninteractive apt-fast --yes install ${packages} > "${install_log_filepath}"
 log "done"
 log "Installation log written to ${install_log_filepath}"
 
@@ -84,7 +81,7 @@ installed_packages=$(get_installed_packages "${install_log_filepath}")
 log "Installed package list:"
 for installed_package in ${installed_packages}; do
   # Reformat for human friendly reading.  
-  log "- $(echo "${installed_package}" | awk -F= '{print $1" ("$2")"}')"
+  log "- $(echo ${installed_package} | awk -F\= '{print $1" ("$2")"}')"
 done
 
 log_empty_line
@@ -96,7 +93,7 @@ for installed_package in ${installed_packages}; do
 
   # Sanity test in case APT enumerates duplicates.
   if test ! -f "${cache_filepath}"; then
-    read -r package_name package_ver < <(get_package_name_ver "${installed_package}")
+    read package_name package_ver < <(get_package_name_ver "${installed_package}")
     log "  * Caching ${package_name} to ${cache_filepath}..."
 
     # Pipe all package files (no folders), including symlinks, their targets, and installation control data to Tar.
@@ -107,9 +104,9 @@ for installed_package in ${installed_packages}; do
       while IFS= read -r f; do
         if test -f "${f}" -o -L "${f}"; then
           get_tar_relpath "${f}"
-          if [[ -L "${f}" ]]; then
+          if [ -L "${f}" ]; then
             target="$(readlink -f "${f}")"
-            if [[ -f "${target}" ]]; then
+            if [ -f "${target}" ]; then
               get_tar_relpath "${target}"
             fi
           fi
@@ -123,7 +120,7 @@ for installed_package in ${installed_packages}; do
   # Comma delimited name:ver pairs in the all packages manifest.
   manifest_all="${manifest_all}${package_name}=${package_ver},"
 done
-log "done (total cache size $(du -h "${cache_dir}" | tail -1 | awk '{print $1}'))"
+log "done (total cache size $(du -h ${cache_dir} | tail -1 | awk '{print $1}'))"
 
 log_empty_line
 

README.md

@@ -17,16 +17,14 @@ set +e
 #   Filepath of the install script, otherwise an empty string.
 ###############################################################################
 function execute_install_script {
-  local package_name
-  package_name=$(basename "${2}" | awk -F= '{print $1}')  
-  local install_script_filepath
-  install_script_filepath=$(\
+  local package_name=$(basename ${2} | awk -F\= '{print $1}')  
+  local install_script_filepath=$(\
     get_install_script_filepath "${1}" "${package_name}" "${3}")
   if test ! -z "${install_script_filepath}"; then
     log "- Executing ${install_script_filepath}..."
     # Don't abort on errors; dpkg-trigger will error normally since it is
     # outside its run environment.
-    sudo sh -x "${install_script_filepath}" "${4}" || true
+    sudo sh -x ${install_script_filepath} ${4} || true
     log "  done"
   fi
 }
@@ -42,17 +40,9 @@ function execute_install_script {
 ###############################################################################
 function get_install_script_filepath {
   # Filename includes arch (e.g. amd64).
-  local filepath
-  # Use glob expansion instead of ls|grep for better handling of non-alphanumeric filenames
-  # Use nullglob to prevent literal match when no files found
-  shopt -s nullglob
-  for f in "${1}"var/lib/dpkg/info/"${2}"*."${3}"; do
-    if [[ -f "${f}" ]] && [[ "${f}" =~ ${2}'(:.*)?.'"${3}" ]]; then
-      filepath="${f}"
-      break
-    fi
-  done
-  shopt -u nullglob
+  local filepath="$(\
+    ls -1 ${1}var/lib/dpkg/info/${2}*.${3} 2> /dev/null \
+    | grep -E ${2}'(:.*)?.'${3} | head -1 || true)"
   test "${filepath}" && echo "${filepath}"
 }
 
@@ -76,7 +66,7 @@ function get_installed_packages {
       log_err "Unable to parse package name and version from \"${line}\""
       exit 2
     fi
-  done < <(grep "^Unpacking " "${install_log_filepath}")
+  done < <(grep "^Unpacking " ${install_log_filepath})
   if test -n "${dep_packages}"; then
     echo "${dep_packages:0:-1}"  # Removing trailing space.
   else
@@ -93,13 +83,13 @@ function get_installed_packages {
 ###############################################################################
 function get_package_name_ver {
   local ORIG_IFS="${IFS}"
-  IFS='=' read -r name ver <<< "${1}"
+  IFS=\= read name ver <<< "${1}"
   IFS="${ORIG_IFS}"
   # If version not found in the fully qualified package value.
   if test -z "${ver}"; then
     # This is a fallback and should not be used any more as its slow.
     log_err "Unexpected version resolution for package '${name}'"
-    ver="$(apt-cache show "${name}" | grep '^Version:' | awk '{print $2}')"
+    ver="$(apt-cache show ${name} | grep '^Version:' | awk '{print $2}')"
   fi
   echo "${name}" "${ver}"  
 }
@@ -115,19 +105,16 @@ function get_normalized_package_list {
   # Remove commas, and block scalar folded backslashes,
   # extraneous spaces at the middle, beginning and end
   # then sort.
-  local packages
-  packages=$(echo "${1}" \
+  local packages=$(echo "${1}" \
     | sed 's/[,\]/ /g; s/\s\+/ /g; s/^\s\+//g; s/\s\+$//g' \
     | sort -t' ')
-  local script_dir
-  script_dir="$(dirname -- "$(realpath -- "${0}")")"
+  local script_dir="$(dirname -- "$(realpath -- "${0}")")"
 
-  local architecture
-  architecture=$(dpkg --print-architecture)
-  if [[ "${architecture}" == "arm64" ]]; then
-    "${script_dir}"/apt_query-arm64 normalized-list "${packages}"
+  local architecture=$(dpkg --print-architecture)
+  if [ "${architecture}" == "arm64" ]; then
+    ${script_dir}/apt_query-arm64 normalized-list ${packages}
   else
-    "${script_dir}"/apt_query-x86 normalized-list "${packages}"
+    ${script_dir}/apt_query-x86 normalized-list ${packages}
   fi
 }
 
@@ -140,8 +127,8 @@ function get_normalized_package_list {
 #   The relative filepath to archive.
 ###############################################################################
 function get_tar_relpath {
-  local filepath="${1}"
-  if test "${filepath:0:1}" = "/"; then
+  local filepath=${1}
+  if test ${filepath:0:1} = "/"; then
     echo "${filepath:1}"
   else
     echo "${filepath}"
@@ -194,7 +181,7 @@ function validate_bool {
   if test "${1}" != "true" -a "${1}" != "false"; then
     log "aborted"
     log "${2} value '${1}' must be either true or false (case sensitive)."
-    exit "${3}"
+    exit ${3}
   fi
 }
 
@@ -208,12 +195,12 @@ function validate_bool {
 #   Log lines from write.
 ###############################################################################
 function write_manifest {  
-  if [[ ${#2} -eq 0 ]]; then 
+  if [ ${#2} -eq 0 ]; then 
     log "Skipped ${1} manifest write. No packages to install."
   else
     log "Writing ${1} packages manifest to ${3}..."
     # 0:-1 to remove trailing comma, delimit by newline and sort.
-    echo "${2:0:-1}" | tr ',' '\n' | sort > "${3}"
+    echo "${2:0:-1}" | tr ',' '\n' | sort > ${3}
     log "done"
   fi
 }

install_and_cache_pkgs.sh

@@ -29,12 +29,12 @@ test "${debug}" = "true" && set -x
 add_repository="${6}"
 
 # List of the packages to use.
-packages="${*:7}"
+packages="${@:7}"
 
 if test "${cache_hit}" = "true"; then
-  "${script_dir}"/restore_pkgs.sh "${cache_dir}" "${cache_restore_root}" "${execute_install_scripts}" "${debug}"
+  ${script_dir}/restore_pkgs.sh "${cache_dir}" "${cache_restore_root}" "${execute_install_scripts}" "${debug}"
 else
-  "${script_dir}"/install_and_cache_pkgs.sh "${cache_dir}" "${debug}" "${add_repository}" "${packages}"
+  ${script_dir}/install_and_cache_pkgs.sh "${cache_dir}" "${debug}" "${add_repository}" ${packages}
 fi
 
 log_empty_line