fix: More error messages print to standard error

Author: hyperupcall

pkg/bin/basalt

@@ -34,7 +34,7 @@ readlinkf_posix() {
 }
 
 if [ "$0" != "${BASH_SOURCE[0]}" ]; then
-printf '%s\n' "Error: basalt: Script must not be sourced"
+printf '%s\n' "Error: basalt: Script must not be sourced" >&2
 return 1
 fi
 

pkg/bin/basalt-package-init

@@ -34,7 +34,7 @@ readlinkf_posix() {
 }
 
 if [ "$0" != "${BASH_SOURCE[0]}" ]; then
-printf '%s\n' "Error: basalt-package-init: Script must not be sourced"
+printf '%s\n' "Error: basalt: Script must not be sourced" >&2
 return 1
 fi
 

pkg/share/scripts/basalt-package-init.sh

@@ -5,19 +5,19 @@
 
 basalt.package-init() {
 if [ -z "$BASALT_GLOBAL_REPO" ]; then
-printf '%s\n' "Error: basalt.package-init: Variable '\$BASALT_GLOBAL_REPO' is empty" >&2
+printf '%s\n' "Error: basalt: Variable '\$BASALT_GLOBAL_REPO' is empty" >&2
 exit 1
 fi
 
 if [ ! -f "$BASALT_GLOBAL_REPO/pkg/src/public/basalt-global.sh" ]; then
-printf '%s\n' "Error: basalt.package-init: Failed to find file 'basalt-global.sh' in '\$BASALT_GLOBAL_REPO'" >&2
+printf '%s\n' "Error: basalt: Failed to find file 'basalt-global.sh' in '\$BASALT_GLOBAL_REPO'" >&2
 exit 1
 fi
 # shellcheck source=../../../pkg/src/public/basalt-global.sh
 source "$BASALT_GLOBAL_REPO/pkg/src/public/basalt-global.sh"
 
 if [ ! -f "$BASALT_GLOBAL_REPO/pkg/src/public/basalt-package.sh" ]; then
-printf '%s\n' "Error: basalt.package-init: Failed to find file 'basalt-package.sh' in '\$BASALT_GLOBAL_REPO'" >&2
+printf '%s\n' "Error: basalt: Failed to find file 'basalt-package.sh' in '\$BASALT_GLOBAL_REPO'" >&2
 exit 1
 fi
 # shellcheck source=../../../pkg/src/public/basalt-package.sh
@@ -32,18 +32,18 @@ basalt.package-init() {
 local __basalt_target=
 __basalt_target=$(readlink "$__basalt_file")
 if ! cd "${__basalt_target%/*}"; then
-printf '%s\n' "Error: basalt.package-init: Could not cd to '${__basalt_target%/*}'" >&2
+printf '%s\n' "Error: basalt: Could not cd to '${__basalt_target%/*}'" >&2
 exit 1
 fi
 else
 if ! cd "${__basalt_file%/*}"; then
-printf '%s\n' "Error: basalt.package-init: Could not cd to '${__basalt_file%/*}'" >&2
+printf '%s\n' "Error: basalt: Could not cd to '${__basalt_file%/*}'" >&2
 exit 1
 fi
 fi
 
 if ! cd "$__old_cd"; then
-printf '%s\n' "Error: basalt.package-init: Could not cd back to '$__old_cd'" >&2
+printf '%s\n' "Error: basalt: Could not cd back to '$__old_cd'" >&2
 exit 1
 fi
 

pkg/src/commands/basalt-global-init.sh

@@ -16,7 +16,7 @@ basalt-global-init() {
 local basalt_global_repo=
 if [ -L "$0" ]; then
 if ! basalt_global_repo=$(readlink -f "$0"); then
-printf '%s\n' "printf '%s\n' \"Error: basalt-package-init: Invocation of readlink failed\""
+printf '%s\n' "printf '%s\n' \"Error: basalt-package-init: Invocation of readlink failed\" >&2"
 printf '%s\n' 'exit 1'
 fi
 basalt_global_repo=${basalt_global_repo%/*}

pkg/src/public/basalt-global.sh

@@ -37,14 +37,14 @@ basalt.load() {
 return
 ;;
 -*)
-printf '%s\n' "Error: basalt.load: Flag '$arg' not recognized"
+printf '%s\n' "Error: basalt: Flag '$arg' not recognized" >&2
 return 1
 ;;
 esac done
 
 if (($# == 0)); then
 # WET (gamma)
-printf '%s\n' "Error: basalt.load: Must specify arguments
+printf '%s\n' "Error: basalt: Must specify arguments >&2
 
 Usage:
  basalt.load [flags] <package> <file>
@@ -71,7 +71,7 @@ Pass '--help' for more info"
 local __basalt_file="${2:-}"
 
 if [ -z "$__basalt_pkg_path" ]; then
-printf '%s\n' "Error: basalt.load: Missing package as first parameter"
+printf '%s\n' "Error: basalt: Missing package as first parameter" >&2
 if [ "$__basalt_flag_global" = 'yes' ]; then
 return 1
 else
@@ -95,7 +95,7 @@ Pass '--help' for more info"
 fi
 
 if ((${#__basalt_pkg_path_full_array[@]} > 1)); then
-printf '%s\n' "Error: basalt.load: Multiple versions of the package '$__basalt_pkg_path' exists"
+printf '%s\n' "Error: basalt: Multiple versions of the package '$__basalt_pkg_path' exists" >&2
 
 if [ "$__basalt_flag_global" = 'yes' ]; then
 return 1
@@ -120,9 +120,9 @@ Pass '--help' for more info"
 
 local regex="^[^/]+/[^/]+$"
 if [[ "$__basalt_pkg_path" =~ $regex ]]; then
-printf '%s\n' "Error: basalt.load: Package '$__basalt_pkg_path' is not a valid. It must include the website domain"
+printf '%s\n' "Error: basalt: Package '$__basalt_pkg_path' is not a valid. It must include the website domain" >&2
 else
-printf '%s\n' "Error: basalt.load: Package '$__basalt_pkg_path' is not installed $__basalt_str"
+printf '%s\n' "Error: basalt: Package '$__basalt_pkg_path' is not installed $__basalt_str" >&2
 fi
 
 if [ "$__basalt_flag_global" = 'yes' ]; then
@@ -133,7 +133,7 @@ Pass '--help' for more info"
 fi
 
 if [ ! -f "$__basalt_pkg_path_full/$__basalt_file" ]; then
-printf '%s\n' "Error: basalt.load: File '$__basalt_file' not found in package '$__basalt_pkg_path'"
+printf '%s\n' "Error: basalt: File '$__basalt_file' not found in package '$__basalt_pkg_path'"f >&2
 if [ "$__basalt_flag_global" = 'yes' ]; then
 return 1
 else

pkg/src/public/basalt-package.sh

@@ -7,13 +7,13 @@ basalt.package-load() {
 local __basalt_shopt_nullglob=
 
 if [ -z "${BASALT_PACKAGE_DIR:-}" ]; then
-printf '%s\n' "Error: basalt.package-load: Variable '\$BASALT_PACKAGE_DIR' is empty" >&2
+printf '%s\n' "Error: basalt: Variable '\$BASALT_PACKAGE_DIR' is empty" >&2
 exit 1
 fi
 
 # These checks always ensure the generated files are in sync the 'basalt.toml'
 if [ ! -f "$BASALT_PACKAGE_DIR/.basalt/generated/done.sh" ]; then
-printf '%s\n' "Error: basalt.package-load: Command 'basalt install' must be ran again in '$BASALT_PACKAGE_DIR'" >&2
+printf '%s\n' "Error: basalt: Command 'basalt install' must be ran again in '$BASALT_PACKAGE_DIR'" >&2
 exit 1
 fi
 
@@ -22,7 +22,7 @@ basalt.package-load() {
 basalt_file_last_modified_at=$(stat --format '%Y' "$BASALT_PACKAGE_DIR/basalt.toml")
 done_file_last_modified_at=$(stat --format '%Y' "$BASALT_PACKAGE_DIR/.basalt/generated/done.sh")
 if ((basalt_file_last_modified_at >= done_file_last_modified_at)); then # '>=' so automated 'basalt install' work on fast computers
-printf '%s\n' "Error: basalt.package-load: Command 'basalt install' must be ran again in '$BASALT_PACKAGE_DIR'" >&2
+printf '%s\n' "Error: basalt: Command 'basalt install' must be ran again in '$BASALT_PACKAGE_DIR'" >&2
 exit 1
 fi
 
@@ -52,7 +52,7 @@ basalt.package-load() {
 
 if [ -f "$__basalt_package.basalt/generated/source_all.sh" ]; then
 if BASALT_PACKAGE_DIR=$__basalt_package source "$__basalt_package.basalt/generated/source_all.sh"; then :; else
-printf '%s\n' "Error: basalt.package-load: Could not successfully source 'source_all.sh'" >&2
+printf '%s\n' "Error: basalt: Could not successfully source 'source_all.sh'" >&2
 return $?
 fi
 fi
@@ -74,7 +74,7 @@ basalt.package-load() {
 
 if [ -f "$__basalt_package.basalt/generated/source_all.sh" ]; then
 if BASALT_PACKAGE_DIR=$__basalt_package source "$__basalt_package.basalt/generated/source_all.sh"; then :; else
-printf '%s\n' "Error: basalt.package-load: Could not successfully source 'source_all.sh'" >&2
+printf '%s\n' "Error: basalt: Could not successfully source 'source_all.sh'" >&2
 return $?
 fi
 fi

pkg/src/util/init.sh

@@ -38,7 +38,7 @@ init.get_basalt_package_dir() {
 
 printf '%s' "$PWD"
 ); then
-printf '%s\n' "Error: basalt.package-init: Could not find basalt.toml" >&2
+printf '%s\n' "Error: basalt: Could not find basalt.toml" >&2
 exit 1
 fi
 }