Skip to content

Commit 5fe99e2

Browse files
author
Michael Vasseur
committed
Disabling the debug_script is quite hard
For languages/problem executables we have something else what we can disable but this doesn't work for `default_debug_scripts`. We can either store this as a fact with the executable or remove this as the default option in the config.
1 parent 133063f commit 5fe99e2

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

judge/judgedaemon.main.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -911,7 +911,7 @@ function fetch_executable_internal(
911911
[$runpath, $workdir, $tmpfile]));
912912
system($debug_cmd, $retval);
913913
if ($retval !== 0) {
914-
disable('judgehost', 'hostname', $myhost, "Running '$runpath' failed.");
914+
disable('run_script', 'run_script_id', $judgeTask['run_script_id'], "Running '$runpath' failed.");
915915
}
916916

917917
request(

webapp/src/Service/DOMJudgeService.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -605,6 +605,7 @@ public function setInternalError(array $disabled, ?Contest $contest, ?bool $enab
605605
$contestProblem->setAllowJudge($enabled);
606606
}
607607
}
608+
// FIXME: Also disable debug scripts
608609
$this->em->flush();
609610
if ($enabled) {
610611
foreach ($executable->getLanguages() as $language) {

0 commit comments

Comments
 (0)