diff options
| author | Craig Jennings <c@cjennings.net> | 2026-01-10 00:54:17 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-01-10 01:43:58 -0600 |
| commit | 41014590cc54ec490505a3851fa59ef8c5d6aebe (patch) | |
| tree | 94e1734227a8bb2f99d6dc6374a30f4eaa93b065 | |
| parent | 173d3a70e32e3ce1c83cf98346ab9d007782c5b6 (diff) | |
fix(archsetup): copy archsetup to user directory before stow
Previously, stow created dotfile symlinks pointing to wherever archsetup
was run from (e.g., /root/archsetup or /tmp). If run from a location
inaccessible to the user, zsh would fail to read its config on first
login and prompt for new user setup.
The fix copies archsetup to ~/code/archsetup before running stow, ensuring
symlinks always point to a user-accessible location.
Also adds a test to verify dotfiles are readable by the user (not just root).
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
| -rwxr-xr-x | archsetup | 13 | ||||
| -rwxr-xr-x | scripts/testing/run-test.sh | 16 |
2 files changed, 29 insertions, 0 deletions
@@ -464,6 +464,19 @@ create_user () { user_customizations() { action="User Customizations" && display "title" "$action" + # Copy archsetup to user's home directory so dotfile symlinks are accessible. + # Without this, stow creates symlinks pointing to wherever archsetup was run from + # (e.g., /root/archsetup), which the user cannot read after login. + user_archsetup_dir="/home/$username/code/archsetup" + action="copying archsetup to user's home directory" && display "task" "$action" + (mkdir -p "$user_archsetup_dir" && \ + cp -r "$archsetup_dir"/* "$user_archsetup_dir/" && \ + chown -R "$username": "/home/$username/code" \ + >> "$logfile" 2>&1) || error "error" "$action" "$?" + + # Update dotfiles_dir to point to user-accessible location + dotfiles_dir="$user_archsetup_dir/dotfiles" + action="linking dotfiles into place" && display "task" "$action" (cd $dotfiles_dir && stow --target=/home/$username --no-folding --adopt system \ >> "$logfile" 2>&1 ) || error "error" "$action" "$?" diff --git a/scripts/testing/run-test.sh b/scripts/testing/run-test.sh index 4bcb55b..e73f29c 100755 --- a/scripts/testing/run-test.sh +++ b/scripts/testing/run-test.sh @@ -296,6 +296,22 @@ else VALIDATION_PASSED=false fi +# Check if dotfile symlinks are accessible by user (not just root) +# This catches the bug where stow creates symlinks pointing to /root/archsetup or /tmp +# Use sudo -u to avoid starting a full login shell which might hang +step "Checking if dotfiles are readable by user cjennings" +DOTFILE_CHECK=$(sshpass -p "$ROOT_PASSWORD" ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null \ + "root@$VM_IP" "sudo -u cjennings cat /home/cjennings/.zshrc > /dev/null 2>&1 && echo OK || echo FAIL" 2>/dev/null) +if [ "$DOTFILE_CHECK" = "OK" ]; then + success "Dotfiles are readable by user cjennings" +else + error "Dotfiles are NOT readable by user cjennings (symlinks may point to inaccessible location)" + # Show where the symlink points for debugging + sshpass -p "$ROOT_PASSWORD" ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null \ + "root@$VM_IP" "ls -la /home/cjennings/.zshrc" >> "$LOGFILE" 2>&1 + VALIDATION_PASSED=false +fi + # Check if yay is installed step "Checking if yay (AUR helper) is installed" if sshpass -p "$ROOT_PASSWORD" ssh -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null \ |
