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

Update sourcepawn submodule #2146

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update sourcepawn submodule #2146

wants to merge 2 commits into from

Conversation

tsuza
Copy link
Contributor

@tsuza tsuza commented May 1, 2024

The error reported in #2145 has been fixed in alliedmodders/sourcepawn@9594f99.
This PR just updates the submodule to the latest remote commit ( I have not personally compiled & tested it ).

@dvander
Copy link
Member

dvander commented May 1, 2024

Looks like there are some build errors, I'll try to address those in SP this week.

@dvander
Copy link
Member

dvander commented May 4, 2024

Oh, you just need to bump the AMTL version in your PR. That should fix it.

@tsuza
Copy link
Contributor Author

tsuza commented May 6, 2024

Sorry for the late commit. I hadn't realized that you had responded until now.

@tsuza
Copy link
Contributor Author

tsuza commented May 6, 2024

The mock test is failing due to it using "any" ( however, I don't know if it's a regression or it was intentional ). If we were to change it to "int", it'd compile & correctly sort the array.

However, I've noticed that the with the latest spcomp the plugin prints gibberish.
image (87)

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.

None yet

2 participants