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.

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

Consider having pub use its own naming schema for temporary directories #22586

Closed
ricowind opened this issue Feb 26, 2015 · 2 comments
Closed
Labels
type-enhancement A request for a change that isn't a bug

Comments

@ricowind
Copy link
Contributor

Using the system provided mkdtemp will make the directory only readable by the user running the script

Snippet from discussion:
could we simply fix this by not relying on system generated temporary dirs for this? You are already creating these in a directory you control, how about temp/temp_package_name_millisecondsSinceEpoch for the temporary name? Creating that with the normal Directory.create function? It should be a pretty local change if I remember correctly on how pub has the io library

Since there can potentially be more than one pub instance running (in different directories concurrently, getting the same package) - consider also including a hash of the directory you are invoking pub on in the dir name.

@nex3
Copy link
Member

nex3 commented Feb 26, 2015

I'm worried that any schema would either allow for potential conflicts or be burdensomely complex. Also, it seems hacky to add a workaround for this in pub when we could just fix issue #15078 and provide value for the entire ecosystem.

@DartBot
Copy link

DartBot commented Jun 5, 2015

This issue has been moved to dart-lang/pub#1258.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-enhancement A request for a change that isn't a bug
Projects
None yet
Development

No branches or pull requests

4 participants