From d52e2bb9c20216972754c054e8534bca28baab66 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Mon, 17 Jul 2017 15:45:44 -0400 Subject: [PATCH] Use config_parse_sec_fix_0() also for JobRunningTimeoutSec 2d79a0bbb9f651656384a0a86ed814e6306fb5dd did that for TimeoutSec=, 89beff89edba592366b2960bd830d3f6e602c2c7 did that for JobTimeoutSec=, and 0004f698df1410ef8b6ab3fb5f4b41a60c91182c did that for x-systemd.device-timeout=. But after parsing x-systemd.device-timeout=xxx we write it out as JobRunningTimeoutSec=xxx. Two options: - write out JobRunningTimeoutSec=, - change JobRunningTimeoutSec= to behave like the other options. I think it would be confusing for JobRunningTimeoutSec= to have different syntax then TimeoutSec= and JobTimeoutSec=, so this patch implements the second option. Fixes #6264, https://bugzilla.redhat.com/show_bug.cgi?id=1462378. (cherry picked from commit 4a06cbf8387555c7c04a1ee6f0c5a6f858bf4b19) --- src/core/load-fragment-gperf.gperf.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/core/load-fragment-gperf.gperf.m4 b/src/core/load-fragment-gperf.gperf.m4 index 5b5a86250e..7fb39cf948 100644 --- a/src/core/load-fragment-gperf.gperf.m4 +++ b/src/core/load-fragment-gperf.gperf.m4 @@ -194,7 +194,7 @@ Unit.OnFailureIsolate, config_parse_job_mode_isolate, 0, Unit.IgnoreOnIsolate, config_parse_bool, 0, offsetof(Unit, ignore_on_isolate) Unit.IgnoreOnSnapshot, config_parse_warn_compat, DISABLED_LEGACY, 0 Unit.JobTimeoutSec, config_parse_sec_fix_0, 0, offsetof(Unit, job_timeout) -Unit.JobRunningTimeoutSec, config_parse_sec, 0, offsetof(Unit, job_running_timeout) +Unit.JobRunningTimeoutSec, config_parse_sec_fix_0, 0, offsetof(Unit, job_running_timeout) Unit.JobTimeoutAction, config_parse_emergency_action, 0, offsetof(Unit, job_timeout_action) Unit.JobTimeoutRebootArgument, config_parse_unit_string_printf, 0, offsetof(Unit, job_timeout_reboot_arg) Unit.StartLimitIntervalSec, config_parse_sec, 0, offsetof(Unit, start_limit.interval)