TTimo / GtkRadiant

The open source, cross platform level editor for idtech games
http://icculus.org/gtkradiant/
Other
582 stars 152 forks source link

XML parser failed on win xp #595

Closed Garux closed 6 years ago

Garux commented 6 years ago

@TTimo i'v tried recent build @ xp box this:

Copy D:/Program Files/GtkRadiant-20180122/installs/Q3Pack/install//missionpack/textures/base_wall2/techfloor_kc_blue.tga -> N:\q2\4/missionpack/textures/base_wall2/techfloor_kc_blue.tga Writing game file: D:/Program Files/GtkRadiant-20180122/games/q3.game Scanning for game description files: D:/Program Files/GtkRadiant-20180122/games D:/Program Files/GtkRadiant-20180122/games/q3.game XML parser failed on 'D:/Program Files/GtkRadiant-20180122/games/q3.game' mGameCombo == NULL No games setup, aborting

GetLastError: ?????? ?????? ?? ????????.

An unrecoverable error has occured. Would you like to edit Preferences before exiting Radiant?

GtkRadiant-20180122\games\q3.game:

<?xml version="1.0" encoding="iso-8859-1" standalone="yes"?>
<game
  name="Quake III Arena and mods"
  enginepath_win32="N:\q2\4"
  gametools_win32="D:/Program Files/GtkRadiant-20180122/installs/Q3Pack/game"
  prefix=".q3a"
  basegame="baseq3"
/>
Pan7 commented 6 years ago

It fails here https://github.com/TTimo/GtkRadiant/blob/master/radiant/preferences.cpp#L1244 for you but i can't reproduce it myself.

Pan7 commented 6 years ago

is libxml2.dll missing?

TTimo commented 6 years ago

It’s probably a non-xp-friendly libxml2.dll

Garux commented 6 years ago

There is no any libxml, plus no one is linked I wonder how it works on other systems kinda default

TTimo commented 6 years ago

I suspect we statically link it (libxml2.lib) but it was likely compiled with a non XP friendly toolchain.

Pan7 commented 6 years ago

Maybe the "Visual C++ Redistributable for Visual Studio 2017" is missing? x86 https://go.microsoft.com/fwlink/?LinkId=746571 x64 https://go.microsoft.com/fwlink/?LinkId=746572

Garux commented 6 years ago

woo-hoo finally running on xp after 6 years default though redist 2017 requirement is unobvious (no dlls missing even); Do newer windows prompt about that?

ensiform commented 6 years ago

The link is mentioned as mandatory on irc I know that, but newer versions of Windows might ship with it.

Kill XP with fire!!!