Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Script error after update #2758

Closed
breadbyte opened this issue Jul 12, 2024 Discussed in #2753 · 1 comment · Fixed by #2760
Closed

Script error after update #2758

breadbyte opened this issue Jul 12, 2024 Discussed in #2753 · 1 comment · Fixed by #2760
Assignees

Comments

@breadbyte
Copy link
Member

Discussed in #2753

Originally posted by gi-b716 July 5, 2024
I wrote a script when I was using version 20240415-263. Now I updated to 20240703-271, and the script gave the following error:
pkRfz28.png
(Starting from line 16)
I can see that Centry is mentioned in the error. I looked through the commit records and found that Sentry was introduced in this commit. But I think it has nothing to do with my script. What should I do?

@breadbyte
Copy link
Member Author

The issue here is that we cache MCC's executable for the scripting system.
When updating the MCC version, the cache is not updated.
We just check if the executable already exists in the temp folder, and trying to fetch a new library that didn't exist in the previous version (Sentry in this case), returns an error since it doesn't exist in the cached version.

A temporary fix is to clear the user's temp folder (as that's where MCC's scripting cache is located).
Ideally we would re-cache the executable when MCC is updated.

@breadbyte breadbyte self-assigned this Jul 12, 2024
breadbyte added a commit to breadbyte/Minecraft-Console-Client that referenced this issue Jul 13, 2024
fixes MCCTeam#2758

this happened because we added sentry in a future release, but the cache was using an old executable, so it couldn't find the assembly.
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 a pull request may close this issue.

1 participant