./configure should test for presence of pygments

SFTtech / openage

Free (as in freedom) open source clone of the Age of Empires II engine 🚀

http://openage.dev

Other

12.73k stars 1.12k forks source link

./configure should test for presence of pygments #443

Closed d33tah closed 8 years ago

d33tah commented 8 years ago

As in the title, ./configure passed but "make" resulted in an error that probably should have been caught earlier.

d33tah commented 8 years ago

@TheJJ: I don't know CMake, so "just do it" doesn't really seem to apply to me. Thanks for quick labeling though ;)

TheJJ commented 8 years ago

:D the checks are performed here: https://github.com/SFTtech/openage/blob/master/buildsystem/CheckRuntimeDependencies.cmake#L8

and pygments is already in there, so i wonder why it's not detected as missing.

d33tah commented 8 years ago

Virtualenv maybe?

TheJJ commented 8 years ago

Uh, I never saw anyone to use a virtualenv for openage. So it's totally untested and may require lot's of work. Doesn't your distro allow you to install it via its package manager?