Skip to content

fix: error thrown for latest arch linux& update tools#7

Open
lonelam wants to merge 1 commit intovenshine:masterfrom
lonelam:master
Open

fix: error thrown for latest arch linux& update tools#7
lonelam wants to merge 1 commit intovenshine:masterfrom
lonelam:master

Conversation

@lonelam
Copy link

@lonelam lonelam commented Apr 4, 2024

  1. The previous SCRIPT_DIR is not available for the latest WSL and arch linux environment. using absolute path is more reasonable.
  2. I just updated all the tools to the latest version, since the old tools are rather slow.

@jemshit
Copy link

jemshit commented Dec 18, 2025

Hi, i can confirm replacing SCRIPT_DIR="$(dirname "$0")" with SCRIPT_DIR="$(realpath $(dirname "$0"))" on macOs arm64 works, but I am not sure why do you need to update 62 files in this PR. Also realpath might not work on Windows, Claude suggests to use SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"

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