Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

.rpt2_cache should be in /tmp #34

Closed
tkrotoff opened this issue Sep 27, 2017 · 3 comments
Closed

.rpt2_cache should be in /tmp #34

tkrotoff opened this issue Sep 27, 2017 · 3 comments
Labels
controversial kind: feature New feature or request kind: optimization Performance, space, size, etc improvement scope: docs Documentation could be improved. Or changes that only affect docs solution: already possible This is already possible / this feature already exists solution: workaround available There is a workaround available for this issue

Comments

@tkrotoff
Copy link

Instead of having .rpt2_cache directory created inside the "working/current" directory, it should be in /tmp so that users don't have to .gitignore it.
@ezolenko
Copy link
Owner

ezolenko commented Sep 27, 2017 •
edited
Loading

You should be able to override the location in plugin options:

plugins: [
    typescript( { cacheRoot: `${require('temp-dir')}/.rpt2_cache` }),
]

I don't want to move cache out by default, because there will be potential for conflict (single system building multiple copies of the same repository simultaneously for example). Also not a nice thing to do to CI machines.

@enil
Copy link

enil commented Oct 25, 2017

At least I think it should be documented the plugin creates a temp directory in the current directory.

ezolenko added a commit that referenced this issue Oct 25, 2017
@ezolenko
- documentation for #34
5b85359
@tkrotoff tkrotoff closed this as completed Nov 3, 2017
@mattmazzola mattmazzola mentioned this issue Jan 31, 2018
Closed
@developit developit mentioned this issue Jul 19, 2018
Closed
mirismaili added a commit to live-html-editor/TextWrap that referenced this issue May 11, 2019
@mirismaili
".rpt2_cache" was moved to OS's temp directory. See:
da18c4f 
ezolenko/rollup-plugin-typescript2#34 (comment)
@agilgur5 agilgur5 added kind: feature New feature or request solution: workaround available There is a workaround available for this issue scope: docs Documentation could be improved. Or changes that only affect docs solution: already possible This is already possible / this feature already exists labels May 26, 2022
@agilgur5 agilgur5 changed the title .rpt2_cache should be in /tmp .rpt2_cache should be in /tmp May 26, 2022
@agilgur5 agilgur5 added controversial kind: optimization Performance, space, size, etc improvement labels May 26, 2022
@agilgur5
Copy link
Collaborator

Note that the default cache directory was moved to inside node_modules in #133, which was a cross-platform alternative that similarly removed the need for having to add to .gitignore.

The cache location can still be changed with cacheRoot as per above comment.

Repository owner locked as resolved and limited conversation to collaborators May 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
controversial kind: feature New feature or request kind: optimization Performance, space, size, etc improvement scope: docs Documentation could be improved. Or changes that only affect docs solution: already possible This is already possible / this feature already exists solution: workaround available There is a workaround available for this issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@tkrotoff @enil @ezolenko @agilgur5