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

            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.

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

                Created:
                Updated: