Summary

Move controlled_subprocess context manager to djblets/utils/misc.py

Review Request #2626 — Created Sept. 26, 2011 and submitted Oct. 5, 2011, 6:43 p.m.

Information

Owner

mike_conley

Repository

Djblets

Branch

master

Bugs

Depends On

Reviewers

Groups

djblets

People

Description

controlled_subprocess was introduced in http://reviews.reviewboard.org/r/2620 as a potential solution to our blocking patch subprocess problem.

It might be useful in the future, so I've brought it out into djblets/utils/misc.py.

Testing Done

Manual - see http://reviews.reviewboard.org/r/2620

Issues

Description	From	Last Updated
what is this?	TI timmylita	Oct. 20, 2011, 5:25 a.m.

Status: Closed (submitted)

Change Summary:

Pushed to master (8c7ee86)

djblets/util/misc.py (Diff revision 1)

Show all issues

what is this?

mike_conley Oct. 20, 2011, 5:25 a.m.

Hey timmylita:

This is the instance of Review Board that we use to vet patches for Review Board itself - thus it shouldn't be used for test messages and the like.

If you want to do some experimenting with a demo Review Board instance, try:

http://demo.reviewboard.org

Thanks,

-Mike

Ship it!

Ship It!

chipx86 Oct. 20, 2011, 5:24 a.m.

This server isn't for testing purposes. It's a production deployment. Please use demo.reviewboard.org.