Skip to content

Fix installer BOM and logger permissions#341

Open
kovtcharov wants to merge 1 commit intomainfrom
kalin/installer-logger-fixes
Open

Fix installer BOM and logger permissions#341
kovtcharov wants to merge 1 commit intomainfrom
kalin/installer-logger-fixes

Conversation

@kovtcharov
Copy link
Collaborator

Summary

  • Fix UTF-8 BOM issue in Windows installer script
  • Update logger to write to user space by default
  • Add permission error handling with helpful messages

Changes

Install Script Fix

  • Remove UTF-8 BOM from install.ps1 that caused PowerShell execution error
  • Fixes: The term '#' is not recognized error when running installer

Logger Improvements

  • Change default log location from current directory to ~/.gaia/gaia.log
  • Add PermissionError handling with user-friendly fallback
  • Prevents errors when running from protected directories (e.g., C:\windows\system32)

Testing

  • Verified installer runs without BOM error
  • Tested logger writes to user space from any directory
  • Confirmed helpful error messages display when permissions are insufficient

These fixes improve the first-run experience and prevent common permission issues on Windows.

- Remove UTF-8 BOM from install.ps1 to fix PowerShell execution error
- Update logger to default to ~/.gaia/gaia.log instead of current directory
- Add graceful PermissionError handling with helpful user messages
- Prevents permission errors when running from protected directories like C:\windows\system32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants