Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(68)

Issue 140069: patch for issue 1030 - redirect stout and stderr on windows (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 month, 1 week ago by timitos
Modified:
3 weeks, 2 days ago
Reviewers:
ced
CC:
SVN Base:
Visibility:
Public.

Patch Set 1

Total comments: 2

Patch Set 2 : redirect stdout and stderr only when executed as an exe

Total comments: 5

Patch Set 3 : adding missing space after comma

Patch Set 4 : close stderr and stdout

Total comments: 1

Patch Set 5 : use os.devnull

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M bin/tryton View 1 2 3 4 1 chunk 16 lines 0 comments Download

Messages

Total messages: 13
ced
http://codereview.appspot.com/140069/diff/1/2 File bin/tryton (right): http://codereview.appspot.com/140069/diff/1/2#newcode11 Line 11: if os.name == 'nt': It should be done ...
4 weeks, 1 day ago
timitos
http://codereview.appspot.com/140069/diff/1/2 File bin/tryton (right): http://codereview.appspot.com/140069/diff/1/2#newcode11 Line 11: if os.name == 'nt': On 2009/11/05 12:57:22, ced ...
4 weeks, 1 day ago
ced
http://codereview.appspot.com/140069/diff/2002/3002 File bin/tryton (right): http://codereview.appspot.com/140069/diff/2002/3002#newcode12 Line 12: if os.name == 'nt' and hasattr(sys,"frozen"): missing space ...
4 weeks, 1 day ago
timitos
http://codereview.appspot.com/140069/diff/2002/3002 File bin/tryton (right): http://codereview.appspot.com/140069/diff/2002/3002#newcode12 Line 12: if os.name == 'nt' and hasattr(sys,"frozen"): On 2009/11/05 ...
4 weeks, 1 day ago
timitos
http://codereview.appspot.com/140069/diff/2002/3002 File bin/tryton (right): http://codereview.appspot.com/140069/diff/2002/3002#newcode13 Line 13: if not os.path.isdir(os.path.join(os.environ['APPDATA'],"Tryton")): On 2009/11/05 19:43:46, ced wrote: ...
4 weeks, 1 day ago
ced
I'm wondering if it will not be better to simply close stdout and stderr.
4 weeks, 1 day ago
timitos
On 2009/11/05 21:47:05, ced wrote: > I'm wondering if it will not be better to ...
3 weeks, 6 days ago
ced
On 2009/11/07 14:59:42, timitos wrote: > On 2009/11/05 21:47:05, ced wrote: > > I'm wondering ...
3 weeks, 6 days ago
timitos
new patchset uploaded. i have the code from boot_common.py of py2exe. please notice that py2exe ...
3 weeks, 5 days ago
ced
http://codereview.appspot.com/140069/diff/5001/5002 File bin/tryton (right): http://codereview.appspot.com/140069/diff/5001/5002#newcode21 bin/tryton:21: sys.stdout = NoWhere() Why not: sys.stdout = open(os.devnull, 'w')
3 weeks, 5 days ago
timitos
seems to work. new patchset with os.devnull
3 weeks, 5 days ago
ced
Ok you can create a changeset for the issue.
3 weeks, 3 days ago
ced
3 weeks, 3 days ago
Applied
Can be closed
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld r497