Skip to content

Update gol script#106

Open
k127 wants to merge 1 commit intoclarisma:mainfrom
k127:patch-1
Open

Update gol script#106
k127 wants to merge 1 commit intoclarisma:mainfrom
k127:patch-1

Conversation

@k127
Copy link
Copy Markdown

@k127 k127 commented Jan 3, 2024

Fix issue for symlinked gol script on macos/unix:

$ ./gol                                                                                                                                                                                                          
readlink: illegal option -- f
usage: readlink [-n] [file ...]
Fehler: Hauptklasse com.geodesk.gol.GolTool konnte nicht gefunden oder geladen werden
Ursache: java.lang.ClassNotFoundException: com.geodesk.gol.GolTool

Source from https://stackoverflow.com/a/246128/211514.

Fix issue for symlinked `gol` script on unix. Source from <https://stackoverflow.com/a/246128/211514>.
@clarisma
Copy link
Copy Markdown
Owner

Thanks for submitting this! (Sorry it took so long to review, have been busy on the Python side). I tried to reproduce the error -- on my configuration, the JAR loads properly even if referenced via symlink.

gol/0.1.9
OpenJDK 19
Ubuntu 22.04

I used this command to create symlink:

ln -s /path/to/gol gol

It also works if the target is on another volume.

On which OS are you experiencing this issue?

On the other hand, your proposed code works fine, so I may just go ahead and merge it. I just wanted to make sure there isn't a secondary issue that interferes with the symlink resolution.

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