Wulf2k / DaS-PC-MPChan

77 stars 30 forks source link

Crash on startup #97

Closed Chronial closed 7 years ago

Chronial commented 8 years ago

The program crashes on startup for some users. Error message is "... stopped working".

Reports: http://steamcommunity.com/app/211420/discussions/0/364042703864810164/

https://www.reddit.com/r/darksouls/comments/4ilq2x/dscm_dark_souls_connectivity_mod/d3nslny

Chronial commented 8 years ago

https://www.reddit.com/r/darksouls/comments/4lil96/daspcmpchan_has_stopped_working/

Chronial commented 8 years ago

Gave ZiggyTheLegend a version without autoattach, still crashes.

Wulf2k commented 8 years ago

Weird.

Is it irc then somehow? On May 29, 2016 7:18 PM, "Chronial" notifications@github.com wrote:

Gave ZiggyTheLegend a version without autoattach, still crashes.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Wulf2k/DaS-PC-MPChan/issues/97#issuecomment-222391281, or mute the thread https://github.com/notifications/unsubscribe/AOMBriI9Hr615OAolbFKJu8rWgWSncBdks5qGizXgaJpZM4IpWYr .

Chronial commented 8 years ago

I will try to check

Wulf2k commented 8 years ago

I'll have the kids in bed within the hour and I'll take a closer look as well. On May 29, 2016 19:29, "Chronial" notifications@github.com wrote:

I will try to check

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Wulf2k/DaS-PC-MPChan/issues/97#issuecomment-222391743, or mute the thread https://github.com/notifications/unsubscribe/AOMBriMmDq8YDwlJxFxsT9err1s_S8geks5qGi9TgaJpZM4IpWYr .

Chronial commented 8 years ago

Found the problem: They are using an old version of .net (4.0), but DSCM requires 4.5. As you did not actually use any 4.5 features in the previous stable, that did run on 4.0. But I used quite a few 4.5 features, so that doesn't work anymore. And apparently .net is not smart enough to register this and just crashes instead …

So there's two things we can do:

  1. Tell people to get the current .net version from https://www.microsoft.com/en-us/download/confirmation.aspx?id=42643 and make it clear that DSCM needs 4.5 (which is preinstalled since windows 8) or
  2. Remove all 4.5 features and target 4.0. That will be quite annoying since the auto-updater uses a lot of 4.5 (Async and the zip module).

What do you think?

Wulf2k commented 8 years ago

I'm fine with 4.5 being required.

Chronial commented 8 years ago

Then that should be stated on

Chronial commented 8 years ago

Doesn't look like we got more reports for this, so I guess it's fix. Leaving this open till the nexus page contains a note about 4.5.

Wulf2k commented 7 years ago

Nexus does note that 4.5 is required.