upstream/restic
1
0
Fork 0
mirror of https://github.com/restic/restic.git synced 2026-02-03 04:20:45 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

backend: always start backend process in separate process group

Browse source 
The process group is necessary to properly handle ctrl-c.
This commit is contained in:
Michael Eischer 2025-04-14 20:18:58 +02:00
parent 4c19d6410f
commit 914bd699be
1 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
internal/backend/util/foreground_unix.go
View file

@ -25,6 +25,12 @@ func tcsetpgrp(fd int, pid int) error {
}
func startForeground(cmd *exec.Cmd) (bg func() error, err error) {
// run the command in its own process group
// this ensures that sending ctrl-c to restic will not immediately stop the backend process.
cmd.SysProcAttr = &unix.SysProcAttr{
Setpgid: true,
}
// open the TTY, we need the file descriptor
tty, err := os.OpenFile("/dev/tty", os.O_RDWR, 0)
if err != nil {
@ -52,11 +58,6 @@ func startForeground(cmd *exec.Cmd) (bg func() error, err error) {
signal.Ignore(unix.SIGTTIN)
signal.Ignore(unix.SIGTTOU)
// run the command in its own process group
cmd.SysProcAttr = &unix.SysProcAttr{
Setpgid: true,
}
// start the process
err = cmd.Start()
if err != nil {
@ -92,5 +93,6 @@ func startFallback(cmd *exec.Cmd) (bg func() error, err error) {
bg = func() error {
return nil
}
return bg, cmd.Start()
}