ruarai / CompilePal

A tool to assist in the compiling of source engine maps
GNU General Public License v3.0
224 stars 25 forks source link

Startup Crash #72

Closed Exactol closed 5 years ago

Exactol commented 5 years ago

Moved to new issue to avoid clogging up the release thread.

gameinfo.txt

https://drive.google.com/file/d/1wXB1Iopr0eqtPshl_KI6u6lSv44UMDN3/view?usp=sharing

Originally posted by @CaskoChan in https://github.com/ruarai/CompilePal/issues/62#issuecomment-491045758

Exactol commented 5 years ago

Can you try deleting your gameconfig.json and restarting CompilePal?

Caskoo commented 5 years ago

That gameconfig.json was from a beta you posted on the tf2 forums I think. Because, your latest build, doesnt generate a gameconfig.json . When I get home from work today, I'll give it a go and let you know if anything changes

Caskoo commented 5 years ago

Nope still get the same error, even when i deleted the gameconfiguration.json

debug.log

2019-05-12T17-01-13.txt

Caskoo commented 5 years ago

Any update on this issue?

Caskoo commented 5 years ago

@Exactol i dont know how to compile github projects, so if you post development builds after you make commits, thats would be cool. Again thanks for all your work.

Exactol commented 5 years ago

The plan for dev builds after V26 is to post them in the downloads section, but marked as pre-release. Might look into adding some kind of continuous integration to automatically compile and post a pre-release for each commit.

For now, I'll post updates builds here @CaskoChan .

Caskoo commented 5 years ago

Sounds good.

Exactol commented 5 years ago

Actually just realized that I asked for the wrong file. Could you send me the GameConfig.txt located in the bin folder of the source mod you are compiling for?

Caskoo commented 5 years ago

GameConfig.txt

Exactol commented 5 years ago

I see the problem, your GameConfig starts with "GameConfig.txt", while Compile Pal expects it to start with "Configs". I'm not sure if this is an invalid GameConfig format, or if it's just that Compile Pal can't handle it. @CaskoChan

Caskoo commented 5 years ago

Huh interesting, Im using https://forum.facepunch.com/dev/bvenk/Slammin-Source-map-tools/1/, im not sure if it changed the gameconfig, could you maybe updated compile pal to be more flexible with gameconfig?

EDIT

After renaming the "GameConfig.txt"with "Configs", compile pal open and compile with no issues sofar. Thanks for your help. Just an idea would be to add error messages here and there.

Exactol commented 5 years ago

I can't find any other people who use the Slammin tools who's game config contained "GameConfig.txt", so I'm not sure why yours was like that. For the future, Compile Pal will no longer crash if it reads a bad game config and will log it. If this comes up more, I'll add support for reading game configs that start with "GameConfig.txt"

MAGNAT2645 commented 5 years ago

I've same error when trying to launch v26:

System.NullReferenceException: object Reference not set to an instance of an object.
   in CompilePalX.GameConfigurationParser.Parse(String filename)
   in CompilePalX.LaunchWindow..ctor()