gilmorem560 / TheForceEngine

Modern "Jedi Engine" replacement supporting Dark Forces, Outlaws and the mods.
https://TheForceEngine.github.io
GNU General Public License v2.0
3 stars 0 forks source link

Compilation error for linux_port_implementation branch #4

Open orderorder opened 2 years ago

orderorder commented 2 years ago

make: *** There is no rule to build the objective 'TheForceEngine/TFE_DarkForces/gameList.o', required for 'TheForceEngine/tfe'. Stop.

gilmorem560 commented 2 years ago

Couldn't say what's going on here, perhaps a case issue? I didn't hit this snag in my build, but may give this a shot later just to see what may be going on. Could be out of step with upstream if you tried to merge in your local environment too. I suggest checking to ensure the gameList.cpp file is actually there and didn't get inadvertently deleted. Tried to make the makefile pretty cut and dry so this shouldn't be too difficult to diagnose.

orderorder commented 2 years ago

when I use:

git clone https://github.com/gilmorem560/TheForceEngine --branch linux_port_implementation

The files "gameList.cpp" and "gameList.h" dont exist for this branch

When i copy these files from another branch then I get this error:

TheForceEngine/TFE_FileSystem/paths.cpp: En la función ‘void TFE_Paths::addSingleFilePath(const char*, const char*)’:
TheForceEngine/TFE_FileSystem/paths.cpp:235:17: error: ‘_strlwr’ no se declaró en este ámbito
  235 |                 _strlwr(fileNameLC);
      |                 ^~~~~~~
make: *** [Makefile:231: TheForceEngine/TFE_FileSystem/paths.o] Error 1
gilmorem560 commented 2 years ago

Looks like the file was legitimately dropped from upstream, I probably synced from trunk at some point but didn't get a chance to run through and bump the Makefile. Feel free to adjust and open a PR if you want, otherwise I'll get to it eventually, just know this isn't a high priority for me at present. The Makefile should be pretty cut-and-dry, dunno what upstream's final build plan is, but I tend to stick to flat Makefiles unless I really need some sort of configure goop draped over things.

orderorder commented 2 years ago

Ok. Thank you for your effort to bring this game to linux

gilmorem560 commented 2 years ago

Credit is due to upstream too, Lucius could've gone with DirectX or very tightly coupled Win32 logic, but instead the project already uses a host of portable libraries. I probably wouldn't have touched this if it was DirectX.