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

Fix script compile errors not being flushed to log file #342

Merged
merged 4 commits into from
Nov 27, 2022

Conversation

wolf109909
Copy link
Contributor

i can't spell before :(

@GeckoEidechse GeckoEidechse changed the title fix script compile errors not being saved to log file Fix script compile errors not being saved to log file Nov 23, 2022
Co-authored-by: GeckoEidechse <40122905+GeckoEidechse@users.noreply.github.com>
@wolf109909
Copy link
Contributor Author

uwu

@GeckoEidechse
Copy link
Member

@pg9182 wanna test this with Docker?

@GeckoEidechse GeckoEidechse added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Nov 27, 2022
Copy link
Member

@emma-miler emma-miler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good. Untested, but would work in theory.

@GeckoEidechse GeckoEidechse removed the needs code review Changes from PR still need to be reviewed in code label Nov 27, 2022
Copy link
Member

@GeckoEidechse GeckoEidechse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Confirmed working in testing by inserting syntax error into some Squirrel scripts on purpose

Before:

...
[21:36:40] [FILESYSTM] [info] Changing mod search path from C:\Program Files (x86)\Steam\steamapps\common\Titanfall2\R2Northstar\mods\Northstar.CustomServers\mod to R2Northstar/mods\Northstar.Custom
[21:36:40] [FILESYSTM] [info] Changing mod search path from C:\Program Files (x86)\Steam\steamapps\common\Titanfall2\R2Northstar\mods\Northstar.Custom\mod to R2Northstar/mods\Northstar.CustomServers
[21:36:40] [FILESYSTM] [info] Changing mod search path from C:\Program Files (x86)\Steam\steamap

After:

...
[22:47:38] [FILESYSTM] [info] Changing mod search path from C:\Program Files (x86)\Steam\steamapps\common\Titanfall2\R2Northstar\mods\Northstar.Custom\mod to R2Northstar/mods\Northstar.CustomServers
[22:47:38] [SCRIPT SV] [error] COMPILE ERROR expected ")", found identifier "asdasd"
[22:47:38] [SCRIPT SV] [error] lobby/_lobby.gnut line [22] column [33]

@GeckoEidechse GeckoEidechse removed the needs testing Changes from the PR still need to be tested label Nov 27, 2022
@GeckoEidechse GeckoEidechse changed the title Fix script compile errors not being saved to log file Fix script compile errors not being flushed to log file Nov 27, 2022
@GeckoEidechse GeckoEidechse merged commit 4a00450 into R2Northstar:main Nov 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants