[gpsd-commit-watch] [SCM] GPSD branch, master, updated. release-3.16-579-gbb66fc3

[Fred Wright]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "GPSD".

The branch, master has been updated
       via  bb66fc3b0987a8a9f9b2cee48be2cd195612ad40 (commit)
       via  447c7762b8794ef7876853ee69b8645b77f10f85 (commit)
       via  8cb5632098d97b055bfb214dd5fce2985fc4ef89 (commit)
       via  7c3d72518f7f29ddb2060acc4e8eb0b45457dc7e (commit)
      from  92e30abaa3f7fb298d05ebe207c16f7096d0c15a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit bb66fc3b0987a8a9f9b2cee48be2cd195612ad40
Author: Fred Wright <address@hidden>
Date:   Sun Feb 19 00:15:15 2017 -0800

    Makes connect failure to dead fake.py daemon more informative.
    
    This generates a more specific exception when a connect from
    client_add() fails due to the daemon's having died.  Aside from real
    daemon failures, this also catches some broken valgrind cases.
    
    TESTED:
    Ran "scons build-all check" on OSX.  Also ran failing valgrind-audit
    and observed more informative (and less verbose) backtrace.  Note that
    this is due to an OSX issue with valgrind itself, not the daemon.

commit 447c7762b8794ef7876853ee69b8645b77f10f85
Author: Fred Wright <address@hidden>
Date:   Sat Feb 18 23:55:52 2017 -0800

    Reworks local exception definitions in fake.py.
    
    This reduces duplicated code by using subclasses for local exceptions.
    It's not entirely clear that limiting __str__() to only certain
    exceptions is necessary, but it preserves that behavior.
    
    TESTED:
    Ran "scons build-all check" on several platforms.

commit 8cb5632098d97b055bfb214dd5fce2985fc4ef89
Author: Fred Wright <address@hidden>
Date:   Sat Feb 18 19:53:34 2017 -0800

    Reworks DaemonInstance in fake.py.
    
    This does two things:
    
    1) It splits DaemonInstance into a base class and subclass, where the
    base class is a more generic subprogram runner.  This makes it easier
    to add additional program runners without code duplication.
    
    2) It switches from os.system() to subprocess.Popen() as the mechanism
    for running subprograms.  This is generally cleaner, and in
    "background mode" is able to use natural parallelism rather than the
    ugly kludge of using shell backgrounding.  Aside from being less ugly,
    this also makes termination of the "background" process directly
    visible, avoiding, e.g., hangs when valgrind bombs.
    
    TESTED:
    Ran "scons build-all check" on several platforms.  Also ran
    valgrind-audit where valgrind was installed, though this has issues on
    some platforms.

commit 7c3d72518f7f29ddb2060acc4e8eb0b45457dc7e
Author: Fred Wright <address@hidden>
Date:   Sat Feb 18 17:04:56 2017 -0800

    Fixes subprocess name pollution in SConstruct.
    
    It's cleaner not to bring individual items from the subprocess module
    into the top-level namespace.  There's no efficiency benefit in
    selective imports, since the entire module is always processed.
    
    TESTED:
    Ran "scons build-all check" on OSX.

-----------------------------------------------------------------------

Summary of changes:
 SConstruct  |   6 +-
 gps/fake.py | 181 +++++++++++++++++++++++++++++++-----------------------------
 2 files changed, 98 insertions(+), 89 deletions(-)


hooks/post-receive
-- 
GPSD