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.

Sign up for GitHub

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

Jump to bottom

Default includes (SM1.11) can not found/read #5

Closed
Bara opened this issue Dec 25, 2021 · 2 comments · Fixed by #6
Closed

Default includes (SM1.11) can not found/read #5

Bara opened this issue Dec 25, 2021 · 2 comments · Fixed by #6

Comments

@Bara
Copy link

Bara commented Dec 25, 2021

Since the sourcepawn (compiler) update from november (2021) or so, it seems like that Setup-SP will no longer work with latest SM1.11 versions.

Examples:
https://github.com/surftimer/Surftimer-Official/runs/4343458952
https://github.com/Bara/Multi-Colors/actions/runs/1511711441

While testing this in a VM with Ubuntu 20.04 and VSCode, I couldn't reproduce this.
@c0rp3n
Copy link

c0rp3n commented Jan 3, 2022

Yup I have also ran into this issue when updating colorlib.

The current workaround is adding -i $includePath when calling spcomp.

So in my case this worked out to:

spcomp -E -w234 -O2 -v2 -i $includePath -i $SCRIPTS_PATH/include $file

clugg added a commit to clugg/sm-json that referenced this issue Feb 8, 2022
@clugg
Fix failing CI pipeline for SM1.11
efaecbd 
See rumblefrog/setup-sp#5
clugg added a commit to clugg/sm-json that referenced this issue Feb 8, 2022
@clugg
Fix failing CI pipeline for SM1.11
269b5da 
See rumblefrog/setup-sp#5
eisbaer66 added a commit to higps/robogithub that referenced this issue Jul 3, 2022
@eisbaer66
workaround for rumblefrog/setup-sp#5;
c3f2212
caxanga334 added a commit to caxanga334/tf-bewithrobots-redux that referenced this issue Jul 5, 2022
@caxanga334
Workaround
c7ecd40 
Workaround for rumblefrog/setup-sp#5
@rumblefrog rumblefrog pinned this issue Jul 6, 2022
sapphonie added a commit to sapphonie/SOAP-TF2DM that referenced this issue Jul 6, 2022
@sapphonie
fix for rumblefrog/setup-sp#5
7c6dde5
@rumblefrog rumblefrog mentioned this issue Jul 6, 2022
Merged
@rumblefrog rumblefrog unpinned this issue Jul 6, 2022
@rumblefrog
Copy link
Owner

rumblefrog commented Jul 6, 2022 •
edited

This has been fixed on master, pin to master or @v1.2.0

@Vauff Vauff mentioned this issue Aug 2, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: sp 1.12+ include path resolution rumblefrog/setup-sp
3 participants
@rumblefrog @c0rp3n @Bara