search

CleverRaven / Cataclysm-DDA

Cataclysm - Dark Days Ahead. A turn-based survival game set in a post-apocalyptic world.
http://cataclysmdda.org
Other
10.63k stars 4.17k forks source link

Crash on new game: tried to get invalid profession: unemployed #8801

Closed lungelimb closed 10 years ago

lungelimb commented 10 years ago

Just pulled last pull-request a1e53c5ed6e12dff0291053a893f20570034791e @ubuntu 14.04.

Snaaty commented 10 years ago

Can confirm, latest experimental seems bugged.

KA101 commented 10 years ago

Probably the scenario expansion, but worked fine last night. Recompiling.

i2amroy commented 10 years ago

I can't replicate this, any further info (using an old save, particular start scenario/location, etc.) would be helpful.

KA101 commented 10 years ago

"New game" seems to rule out an old-save issue, but yeah. I can't replicate it with current information.

kyrahabattoir commented 10 years ago

Allright here are my infos on this: capture (Disregard the weird letters it's a terminal issue on my end)

OS: Gentoo Linux amd64 Game version: ncurses

Crash happens right after "please wait while the world is loading" after clicking one of the options to setup a character (custom/random). I tested against the last 22 commits and the problem appears at:

commit caff454956b8efebc4fb395a28c32e7ba165faae Title: Fixes Description: Fixes the fixes that necessitated fixes.

KA101 commented 10 years ago

Our IRC discussion (KyrahAbattoir was a major participant) traced it to a Linux-only problem with that commit, hence why Windows devs couldn't replicate. Thanks for the checkup, KyrahAbattoir.

kevingranade commented 10 years ago

@kyrahabattoir can you take a look at my branch in #8823 and see if it resolves the issue for you? I can't reproduce it here.

VlasovVitaly commented 10 years ago

I don't know roots of problem and i didn't look in to code but #8823 fixes issue for me.

Snaaty commented 10 years ago

Is that issue already up in the latest experimental? I tried 2029, and I still get that crash upon pressing "New Game".

NVM: I just saw that PR isn't merged yet.

kevingranade commented 10 years ago

It is not because we needed confirmation that it works. Sounds like it got it though.

kyrahabattoir commented 10 years ago

@kevingranade Well I'm not crashing on your branch.