search

sphair / ClanLib

ClanLib is a cross platform C++ toolkit library.
Other
344 stars 76 forks source link

Too complicated build system #101

Open Xottab-DUTY opened 6 years ago

Xottab-DUTY commented 6 years ago

According to this code:

#include <ClanLib/core.h>

We should move Sources/API dir to include/ClanLib dir. With this change we can build project that depend on ClanLib out of the box without doing anything. This will make compilation significantly easier.

Without it we should configure it with configure.exe before (if we don't then we'll fail to build)

I can make a PR with it. I just need to know what I need to change additionally, since it seems that just moving the folder is not enough. (we should change the build scripts)

ArtHome12 commented 6 years ago

And what considerations led to such an architecture?

dpjudas commented 6 years ago

If you file a PR with this change it would get merged. You have to make sure the configure tool supports the new location though. You can probably get away with coding this by replacing "API" with "ClanLib" all over the project, including the Setup folder.

@ArtHome12 It looked better visually, that's really it.