DOMjudge / domjudge-packaging

DOMjudge packaging for (Linux) distributions and live image
32 stars 40 forks source link

JudgeHost docker for other languages #120

Closed mateuy-dev closed 2 years ago

mateuy-dev commented 2 years ago

The current docker for judgehost does not support Kotlin either c#. Is there any image with this languages installed? Thanks,

vmcj commented 2 years ago

I think you'll have to build one yourself. So for C# you could add: https://docs.microsoft.com/en-us/dotnet/core/install/linux-ubuntu#2004-

You could also add this to the chroot in the "official" container by using dj_run_chroot and installing there.

mateuy-dev commented 2 years ago

I'll have it a look. I'd just think would be nice to have a docker image for a full judgeHost with everything installed. Otherwise each user has to repeat the same task of modifying the official container

nickygerritsen commented 2 years ago

But wat is 'everything'? We can't really create a Docker with the current 22 languages, that feels like a lot of overhead. Why would we ship fortran95 if no one really wants it. The Docker ships with all enabled languages by default and if people want others, they could easily copy https://github.com/DOMjudge/domjudge-packaging/tree/main/docker/judgehost and add their languages there.

eldering commented 2 years ago

Also, languages such as C# don't have packages available inside the common Linux distributions, but external repositories, or similar, which make them difficult to setup and maintain.

mateuy-dev commented 2 years ago

@nickygerritsen Everything was something like the top 10 used?

Ok, just close the issue. I was trying to avoid "repeated" work.

nickygerritsen commented 2 years ago

Note that for non-Docker setups we also offer the dj_make_chroot with defaults, which only installs Python 3, java and C/C++. Seems to me it is consistent with that and we offer the option to add languages yourself.

I get that it is a bit repeating, but as eldering said it's also non trivial for us to keep these languages up to date.