Fix installation order and properly initialize TRANSMISSION_REMOTE
- Move SCRIPT_DIR initialization before its usage - Ensure TRANSMISSION_REMOTE is initialized with default value - Add logging to show whether remote or local Transmission is being used - Initialize TRANSMISSION_REMOTE at the start of gather_configuration 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
parent
4c68a1ac07
commit
e05c8da811
@ -12,6 +12,9 @@ YELLOW='\033[0;33m'
|
|||||||
RED='\033[0;31m'
|
RED='\033[0;31m'
|
||||||
NC='\033[0m' # No Color
|
NC='\033[0m' # No Color
|
||||||
|
|
||||||
|
# Get current directory
|
||||||
|
SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
|
||||||
|
|
||||||
# Print header
|
# Print header
|
||||||
echo -e "${BOLD}==================================================${NC}"
|
echo -e "${BOLD}==================================================${NC}"
|
||||||
echo -e "${BOLD} Transmission RSS Manager Installer ${NC}"
|
echo -e "${BOLD} Transmission RSS Manager Installer ${NC}"
|
||||||
@ -26,9 +29,6 @@ if [ "$EUID" -ne 0 ]; then
|
|||||||
exit 1
|
exit 1
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# Get current directory
|
|
||||||
SCRIPT_DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )"
|
|
||||||
|
|
||||||
# Check for installation type
|
# Check for installation type
|
||||||
IS_UPDATE=false
|
IS_UPDATE=false
|
||||||
INSTALLATION_DETECTED=false
|
INSTALLATION_DETECTED=false
|
||||||
|
@ -75,6 +75,9 @@ validate_hostname() {
|
|||||||
|
|
||||||
function gather_configuration() {
|
function gather_configuration() {
|
||||||
log "INFO" "Starting configuration gathering"
|
log "INFO" "Starting configuration gathering"
|
||||||
|
|
||||||
|
# Initialize default values for Transmission mode
|
||||||
|
export TRANSMISSION_REMOTE=false
|
||||||
echo -e "${BOLD}Installation Configuration:${NC}"
|
echo -e "${BOLD}Installation Configuration:${NC}"
|
||||||
echo -e "Please provide the following configuration parameters:"
|
echo -e "Please provide the following configuration parameters:"
|
||||||
echo
|
echo
|
||||||
|
@ -4,6 +4,15 @@
|
|||||||
function install_dependencies() {
|
function install_dependencies() {
|
||||||
log "INFO" "Installing dependencies..."
|
log "INFO" "Installing dependencies..."
|
||||||
|
|
||||||
|
# Make sure TRANSMISSION_REMOTE variable is set
|
||||||
|
if [ -z "$TRANSMISSION_REMOTE" ]; then
|
||||||
|
# Set default to false if not defined
|
||||||
|
export TRANSMISSION_REMOTE=false
|
||||||
|
log "WARN" "TRANSMISSION_REMOTE variable was not set, defaulting to local installation"
|
||||||
|
fi
|
||||||
|
|
||||||
|
log "INFO" "Transmission mode: $([ "$TRANSMISSION_REMOTE" = true ] && echo "Remote" || echo "Local")"
|
||||||
|
|
||||||
# Check for package manager
|
# Check for package manager
|
||||||
if command -v apt-get &> /dev/null; then
|
if command -v apt-get &> /dev/null; then
|
||||||
# Update package index
|
# Update package index
|
||||||
|
Loading…
x
Reference in New Issue
Block a user