← Back to team overview

launchpad-reviewers team mailing list archive

Re: [Merge] lp:~allenap/maas/try-out-saucelabs into lp:maas

 

> This foot-shooter in src/maastesting/utils.py is a bit cryptic:
> 
> 1167    +def preexec_fn():
> 1168    +    # Revert Python's handling of SIGPIPE. See
> 1169    +    # http://bugs.python.org/issue1652 for more info.
> 1170    +    signal.signal(signal.SIGPIPE, signal.SIG_DFL)
> 
> Can you explain its intended usage in the docstring, so that people don't do
> anything stupid with it?

People should really look at the linked bug, to be honest.

-- 
https://code.launchpad.net/~allenap/maas/try-out-saucelabs/+merge/106217
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~allenap/maas/try-out-saucelabs into lp:maas.


References