[issue5870] subprocess.DEVNULL

Changes by Ross Lagerwall : -- assignee: gregory.p.smith -> rosslagerwall resolution: -> accepted stage: needs patch -> committed/rejected status: open -> closed ___ Python tracker _

[issue5870] subprocess.DEVNULL

Roundup Robot added the comment: New changeset eaf93e156dff by Ross Lagerwall in branch 'default': Issue #5870: Add subprocess.DEVNULL constant. http://hg.python.org/cpython/rev/eaf93e156dff -- nosy: +python-dev ___ Python tracker

[issue5870] subprocess.DEVNULL

Changes by Santoso Wijaya : -- nosy: +santa4nt ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pytho

[issue5870] subprocess.DEVNULL

Changes by Ned Deily : -- nosy: +mmarkk ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/m

[issue5870] subprocess.DEVNULL

Georg Brandl added the comment: Right, sorry then :) -- assignee: -> gregory.p.smith nosy: +gregory.p.smith ___ Python tracker ___ __

[issue5870] subprocess.DEVNULL

Ross Lagerwall added the comment: I think if you look closely at the patch, the fd does not stay open the whole time. It is opened if necessary in _get_handles() with e.g.: elif stdin == DEVNULL: p2cread = self._get_devnull() and then closed in _execute_child() with: if hasattr(self, '_d

[issue5870] subprocess.DEVNULL

Georg Brandl added the comment: Hmm, we don't like these open-for-eternity file descriptors; we had such a thing for os.urandom() but removed it (see #1177468). I'm okay with DEVNULL (or even just NULL) as a shorthand, but it should open (and close) the devnull device each time just as if a n

[issue5870] subprocess.DEVNULL

Ross Lagerwall added the comment: Here is a fairly simple patch that adds the subprocess.DEVNULL constant. -- keywords: +patch nosy: +rosslagerwall versions: +Python 3.3 -Python 2.7 Added file: http://bugs.python.org/file20215/5870_v1.patch ___ Pytho

[issue5870] subprocess.DEVNULL

Changes by Giampaolo Rodola' : -- nosy: +giampaolo.rodola ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:/

[issue5870] subprocess.DEVNULL

Changes by Amaury Forgeot d'Arc : -- stage: -> needs patch ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http

[issue5870] subprocess.DEVNULL

Antoine Pitrou added the comment: Lucas, Windows has a /dev/null-like device. I think Jean's suggestion is reasonable, but it should be a special constant instead of creating a file descriptor unconditionnally (that is, the file should only be open if needed). Also, a patch should be provided (w

[issue5870] subprocess.DEVNULL

Lucas Prado Melo added the comment: -1 on this one. It is not a portable decision (only *nix OSes do have /dev/null). Also, why would we want it as a default constant? The subprocess module would need to open /dev/null every time. Despite that, I can't see how would someone use the redirection o

[issue5870] subprocess.DEVNULL

Changes by OG7 : -- nosy: +OG7 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/op

[issue5870] subprocess.DEVNULL

Jean Brouwers added the comment: Typo. That should be ..., os.O_RDWR) -- ___ Python tracker ___ ___ Python-bugs-list mailing list Uns

[issue5870] subprocess.DEVNULL

New submission from Jean Brouwers : The subprocess module exposes the PIPE and STDOUT constants to be used for the stdin, stdout or stderr keyword arguments. Often, stderr needs to be redirected to /dev/null (on posix). It would nice if another constant was available for that purpose, e.g. s