question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Evaluate and possible improve `TempDir`

See original GitHub issue

The TempDir extension may or may not be thread-safe - it’s hard to say because its tests definitely aren’t (many static fields). The tests need to be made thread-safe, so we can then evaluate the extension itself. Right now, I see no reason why it shouldn’t be possible to make it thread-safe (if it isn’t already).

Also, can we change the name to TemporaryDirectory or something else that doesn’t sound like letters cost actual money?

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:10 (10 by maintainers)

github_iconTop GitHub Comments

2reactions
nipafxcommented, Sep 11, 2020

The verdict is in, Jupiter’s @TempDir will not get custom FS support. To recapitulate:

  • Jupiter and Pioneer both have a @TempDir extension
  • both work in essentially the same way (for the user)
  • custom file systems is the only thing Pioneer has that Jupiter’s hasn’t
  • there are quite a few features that Jupiter has that Pioneer doesn’t
  • there are a number of open requests to Jupiter’s feature (see Bish’s list)

Nota bene: @sormuras tried a different approach to the same topic (link, link)

In light of that, here’s my proposal:

  • we move forward with the deletion of @TempDir as currently proposed by #327 and close this issue
  • we release 1.0
  • we create a new one that aims at a larger-picture reevaluation of this feature set, including a closer look at Jupiter’s @TempDir and Christian’s experiments
  • we focus on working through currently open PRs, but put that topic on the top of the list right after that
1reaction
Bukamacommented, Sep 8, 2020

As I suggest to remove the TempDir months ago because of this, my answer is of course: Yes remove it.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Multiple @TempDirs should result in multiple directories #1967
I think a test's temporary files should all be confined to a single directory tree. There's no need to have multiple top-level temps....
Read more >
Clear windows temp directory to improve application ...
Clear windows temp directory to improve application performance in an Orion Deployment. This article covers clearing windows temp directory ...
Read more >
How can I automatically create (and remove) a temp directory ...
If all targets are up to date, then evaluating TMPDIR will create the directory, and all 's rules will never be executed to...
Read more >
tempfile: Create Names for Temporary Files - Rdrr.io
For tempfile a character vector giving the names of possible (temporary) files. ... For tempdir , the path of the per-session temporary directory....
Read more >
4 Upgrading Oracle Management Service and Oracle ...
This chapter describes the different ways of upgrading your Oracle Management Service (OMS) and Oracle Management Repository (Management Repository) of 13c ...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found