comparison tests/run-tests.py @ 28455:412ee35a8005

chg: do not write pidfile

Current pidfile logic will only keep the pid of the newest server, which is
not very useful if we want to kill all servers, and will become outdated if
the server auto exits after being idle for too long.

Besides, the server-side pidfile writing logic runs before chgserver gets
confighash so it's not trivial to append confighash to pidfile basename like
we did for socket file.

This patch removes --pidfile from the command starting chgserver and switches
to an alternative way (unlink socket file) to stop the server.

28454:8062869860b8

            sys.exit(1)

    def _killchgdaemons(self):
        """Kill all background chg command servers spawned by tests"""
        for f in os.listdir(self._chgsockdir):
            if not f.endswith(b'.pid'):
                continue
            killdaemons(os.path.join(self._chgsockdir, f))

    def _outputcoverage(self):
        """Produce code coverage output."""
        from coverage import coverage

28455:412ee35a8005

            sys.exit(1)

    def _killchgdaemons(self):
        """Kill all background chg command servers spawned by tests"""
        for f in os.listdir(self._chgsockdir):
            if '.' in f:
                continue
            os.unlink(os.path.join(self._chgsockdir, f))

    def _outputcoverage(self):
        """Produce code coverage output."""
        from coverage import coverage