Learn more about these different git repos.
Other Git URLs
b3f7d7c
2006ab1
@@ -37,7 +37,6 @@
import pwd
import random
import re
- import signal
import socket
import struct
import sys
@@ -584,8 +583,9 @@
if pid:
os._exit(0)
os.setsid()
- signal.signal(signal.SIGHUP, signal.SIG_IGN)
- # fork again
+ # fork again, no need to ignore SIGHUP
+ # https://pagure.io/koji/issue/672
+ # https://code.activestate.com/recipes/278731-creating-a-daemon-the-/
pid = os.fork()
Related: https://pagure.io/koji/issue/672
@mikem
rebased onto 279d91243ab58c69bce034163828a5a6f0e7e8d6
pretty please pagure-ci rebuild
This is the first option from #672, "stop ignoring sighup"
LGTM
Metadata Update from @tkopecek: - Pull-request tagged with: testing-ready
Metadata Update from @mfilip: - Pull-request tagged with: testing-done
rebased onto 2006ab1
Commit 7a1778a fixes this pull-request
Pull-Request has been merged by tkopecek
Related: https://pagure.io/koji/issue/672