Make `rbt install` work correctly on Windows.
Review Request #8354 — Created Aug. 26, 2016 and submitted
Unfortunately, the technique of using
NamedTemporaryFile
in a context manager
and yielding the filename back, then reopening it from elsewhere in the code
caused various permission errors when running on Windows.This change rearranges the code a bit to make sure we flush and close the file,
and then handles cleanup ourselves.
Used it on OS X, and a customer reports success running on Windows.
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 2 (+24 -17) |
-
Tool: Pyflakes Processed Files: rbtools/commands/install.py Tool: PEP8 Style Checker Processed Files: rbtools/commands/install.py
Commit: |
|
||||
---|---|---|---|---|---|
Diff: |
Revision 3 (+17 -11) |