erikik8090 / codeguru-django

The WIP client-server architecture build
GNU General Public License v3.0
1 stars 0 forks source link

Unused environment variables #2

Closed alfasi closed 5 years ago

alfasi commented 5 years ago

The 3 required environment variables don't seem to be used, but still fail @ run time if not found, is this a part of future-feature?

https://github.com/erikik8090/codeguru-django/blob/e5589c88265eace910465edea99782e3ab356e05/war/codeguru_extreme/settings.py#L35-L37

erikik8090 commented 5 years ago

Knew I forgot something in the README. They should contain the path to the GNU toolchain assembler (as) and objcopy that were required to download in the README, and the last one should contain the path to the engine jar that is outputted in the console directory after running ant console. They are in use, when actually running the competition. I will add them to the README, and add a better error message when they are missing.