From 7f8c00c20a321140958777653a3cd61440439068 Mon Sep 17 00:00:00 2001 From: ridiculousfish Date: Fri, 17 Jul 2020 14:10:31 -0700 Subject: [PATCH] Remove job_t::wants_terminal This now lives in the job group, not individual jobs. --- src/parse_execution.cpp | 1 - src/proc.h | 3 --- 2 files changed, 4 deletions(-) diff --git a/src/parse_execution.cpp b/src/parse_execution.cpp index 414f27143..fd07cd0f4 100644 --- a/src/parse_execution.cpp +++ b/src/parse_execution.cpp @@ -1297,7 +1297,6 @@ end_execution_reason_t parse_execution_context_t::run_1_job(const ast::job_t &jo (ctx.job_group && ctx.job_group->wants_job_control()); job_t::properties_t props{}; - props.wants_terminal = wants_job_control && !ld.is_event; props.initial_background = job_node.bg.has_value(); props.skip_notification = ld.is_subshell || ld.is_block || ld.is_event || !parser->is_interactive(); diff --git a/src/proc.h b/src/proc.h index 68b0c53b9..cea47218f 100644 --- a/src/proc.h +++ b/src/proc.h @@ -409,9 +409,6 @@ class job_t { /// special job that should not be reported. bool skip_notification{}; - /// Whether the job wants to own the terminal when in the foreground. - bool wants_terminal{}; - /// Whether the job had the background ampersand when constructed, e.g. /bin/echo foo & /// Note that a job may move between foreground and background; this just describes what the /// initial state should be.