ahamilton/eris
1
0
Fork 0
Code Releases Activity

Remove log file.

Browse source 
- It was really only for debugging.
- The log would be redundant when eris outputs the log, when run
  without a tty.
This commit is contained in:
Andrew Hamilton 2020-04-25 15:18:43 +10:00
parent 41e7c72ded
commit 310451ae4d
2 changed files with 1 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

8
eris/__main__.py
View file

@ -573,7 +573,6 @@ class Log:
_GREY_BOLD_STYLE = termstr.CharStyle(termstr.Color.grey_100, is_bold=True) _GREY_BOLD_STYLE = termstr.CharStyle(termstr.Color.grey_100, is_bold=True)
_GREEN_STYLE = termstr.CharStyle(termstr.Color.lime) _GREEN_STYLE = termstr.CharStyle(termstr.Color.lime)
LOG_PATH = os.path.join(tools.CACHE_PATH, "log")
def __init__(self, appearance_changed_event): def __init__(self, appearance_changed_event):
self._appearance_changed_event = appearance_changed_event self._appearance_changed_event = appearance_changed_event
@ -596,18 +595,12 @@ class Log:
if timestamp is None else timestamp) if timestamp is None else timestamp)
line = termstr.TermStr(timestamp, Log._GREY_BOLD_STYLE) + " " + message line = termstr.TermStr(timestamp, Log._GREY_BOLD_STYLE) + " " + message
self.lines.append(line) self.lines.append(line)
with open(Log.LOG_PATH, "a") as log_file:
print(line, file=log_file)
self._appearance = None self._appearance = None
self._appearance_changed_event.set() self._appearance_changed_event.set()
def log_command(self, message, timestamp=None): def log_command(self, message, timestamp=None):
self.log_message(message, char_style=Log._GREEN_STYLE) self.log_message(message, char_style=Log._GREEN_STYLE)
def delete_log_file(self):
with contextlib.suppress(FileNotFoundError):
os.remove(Log.LOG_PATH)
def appearance(self, dimensions): def appearance(self, dimensions):
if self._appearance is None or \ if self._appearance is None or \
fill3.appearance_dimensions(self._appearance) != dimensions: fill3.appearance_dimensions(self._appearance) != dimensions:
@ -1133,7 +1126,6 @@ def main(root_path, loop, worker_count=None, editor_command=None, theme=None,
pickle_path, jobs_added_event, appearance_changed_event, root_path, pickle_path, jobs_added_event, appearance_changed_event, root_path,
loop) loop)
screen.editor_command = editor_command screen.editor_command = editor_command
log.delete_log_file()
log.log_message("Program started.") log.log_message("Program started.")
jobs_added_event.set() jobs_added_event.set()
asyncio.ensure_future(summary.sync_with_filesystem(log)) asyncio.ensure_future(summary.sync_with_filesystem(log))

2
tests/__main___test.py
View file

@ -222,7 +222,7 @@ class MainTestCase(unittest.TestCase):
with contextlib.redirect_stdout(io.StringIO()): with contextlib.redirect_stdout(io.StringIO()):
__main__.main(root_path, loop, worker_count=2, __main__.main(root_path, loop, worker_count=2,
is_being_tested=True) is_being_tested=True)
for file_name in ["summary.pickle", "creation_time", "log", for file_name in ["summary.pickle", "creation_time",
"foo-metadata", "foo-contents"]: "foo-metadata", "foo-contents"]:
self.assertTrue(os.path.exists(".eris/" + file_name)) self.assertTrue(os.path.exists(".eris/" + file_name))
self.assertEqual(_mount_total(), mount_total) self.assertEqual(_mount_total(), mount_total)