Error compiling on Ubuntu 16.04 x64 about devilution HOT 8 CLOSED

Hmmm I am just jumping on this and looking...

/usr/bin/i686-w64-mingw32-ld: cannot find -ldiabloui
/usr/bin/i686-w64-mingw32-ld: cannot find -lstorm

these files cannot be found.

from devilution.

Well I can see that but the binary /usr/bin/i686-w64-mingw32-ld does exist, I have no clue about the -lbiabloui and -lstorm I honestly have no clue why/how/what they are (well I guess source code to generate the dll's?) maybe it calls the relative path and can't find it. again idk I'm not a developer so am not very familiar with gcc/g++/MinGW-W64

from devilution.

Found the issue DiabloUI.lib and Storm.lib have to be rename to diabloui.lib and storm.lib, also side note diabloui.dll has to be renamed to DiabloUI.dll before you can compile. Another thing that should be mentioned is that STANDARD.snp has to be in the directory for Single Player mode to launch otherwise it will not get passed the menu.

Here is a fixed makefile for Linux that uses the correct case sensitivity https://pastebin.com/ybwykFcA

from devilution.

Found the issue DiabloUI.lib and Storm.lib have to be rename to diabloui.lib and storm.lib, also side note diabloui.dll has to be renamed to DiabloUI.dll before you can compile.

These issues related to upper and lower casing of files are likely to come up also in the future. Windows is case insensitive, while Linux (and Mac?) are case sensitive.

@galaxyhaxz Would it be worth lowercasing all file names? Or is it better to try to keep closer to the original and pertain the casing.

from devilution.

One thing I want to point out is that the original disk names the required files like
diabloui.dll
Storm.dll
SMACKW32.dll
DIABDAT.MPQ
STANDARD.snp
So it might be a good idea to just use the default case, anyways game plays great thanks again

from devilution.

Thanks for the info, I'll update the readme accordingly (forgot about the SNP files, though BATTLE.snp isn't needed).

@mewmew Another fail on my end, forgot that POSIX environments are case sensitive. Makefiles will use lowercase from now on. Also added your warning to suppress the strings.
galaxyhaxz/devilution@67f8485

from devilution.

Good :)

from devilution.

forgot that POSIX environments are case sensitive

Not just posix, don't forget that Windows 10 has an optional case sensitive mode now too.

from devilution.