Skip to content

Minor fix for 7zip-related error#27

Closed
0505daniel wants to merge 2 commits intochkwon:mac_fixfrom
0505daniel:mac_fix
Closed

Minor fix for 7zip-related error#27
0505daniel wants to merge 2 commits intochkwon:mac_fixfrom
0505daniel:mac_fix

Conversation

@0505daniel
Copy link
Copy Markdown

Fixed an issue where tests on Windows failed due to a missing 7z.exe executable:

ERROR: LoadError: IOError: could not spawn 'C:\hostedtoolcache\windows\julia\1.11.4\x64\bin..\libexec\7z.exe' x 'D:\a\Concorde.jl\Concorde.jl\deps\win_dir\concorde.exe.gz' -y '-oD:\a\Concorde.jl\Concorde.jl\deps\win_dir': no such file or directory (ENOENT)
This update ensures that tests pass on Windows by handling the missing executable.

Note: The current implementation installs 7zip if a local installation is not found. I can modify the code to throw an error instead of installing it if preferred.

@0505daniel 0505daniel changed the title **Minor fix for 7zip-related error** Minor fix for 7zip-related error Mar 18, 2025
@chkwon
Copy link
Copy Markdown
Owner

chkwon commented Mar 18, 2025

Thanks for PR.

I think this issue was resolved by #26 Could you check?

@0505daniel
Copy link
Copy Markdown
Author

You're right! I'll go ahead and close the PR. Thanks.

@0505daniel 0505daniel closed this Mar 18, 2025
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