Uploaded image for project: 'Sourcetree For Mac'
  1. Sourcetree For Mac
  2. SRCTREE-1281

temporary files remaining behind

XMLWordPrintable

    • Our product teams collect and evaluate feedback from a number of different sources. To learn more about how we use customer feedback in the planning process, check out our new feature policy.

      SourceTree creates temporary files in $TMPDIR that are not removed when a repository window is closed, or the app exits. This happens under normal usage, apparently during synchronisation.

      That directory ought to be cleaned-up when the files are no longer necessary, as under normal operation the OS only does it at a restart (or possibly a re-login). Which is not a frequent event on a workstation class computer.

            [SRCTREE-1281] temporary files remaining behind

            Katherine Yabut made changes -
            Workflow Original: JAC Suggestion Workflow [ 3372345 ] New: JAC Suggestion Workflow 3 [ 3672589 ]
            Status Original: RESOLVED [ 5 ] New: Closed [ 6 ]
            Monique Khairuliana (Inactive) made changes -
            Workflow Original: SourceTree Bug Workflow [ 448942 ] New: JAC Suggestion Workflow [ 3372345 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            Monique Khairuliana (Inactive) made changes -
            Issue Type Original: Improvement [ 4 ] New: Suggestion [ 10000 ]
            Brian Ganninger (Inactive) made changes -
            Symptom Severity New: Minor [ 14432 ]
            Assignee Original: Steve Streeting [ sstreeting ]

            René Bertin added a comment -

            Can those rare cases apply across application restarts?

            René Bertin added a comment - Can those rare cases apply across application restarts?

            Steve Streeting (Inactive) added a comment -

            True, we use the standard NSTemporaryDirectory() approach which leaves cleanup to the OS. We could manually tidy up earlier.

            Implementation note: in rare cases the temp files are referred to in failure cases, such as when applying patches to stage hunks/lines, for diagnostic purposes. Probably want to associate with the underlying process and only delete on success.

            Steve Streeting (Inactive) added a comment - True, we use the standard NSTemporaryDirectory() approach which leaves cleanup to the OS. We could manually tidy up earlier. Implementation note: in rare cases the temp files are referred to in failure cases, such as when applying patches to stage hunks/lines, for diagnostic purposes. Probably want to associate with the underlying process and only delete on success.
            René Bertin created issue -

              Unassigned Unassigned
              3006afad0c03 René Bertin
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated: