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

annoying warning #36

Closed
kforner opened this issue Feb 20, 2017 · 1 comment
Closed

annoying warning #36

kforner opened this issue Feb 20, 2017 · 1 comment

Comments

@kforner
Copy link

@kforner kforner commented Feb 20, 2017

Hello,
I get plenty of messages:

There is already a file with this name in the temporary directory. It will be overwritten.

I saw that they come from localize_file(). It seems that the temp path (which is probably not needed for non-URL loaded files) is built using tempdir(), which explains the above message.
Why not using tempfile() to build a proper temporary path ? Or do not copy the file if it is not needed ?

P.S
tabulizer works great !

@leeper
Copy link
Member

@leeper leeper commented Feb 20, 2017

Yea, I have no idea why I did it that way. I'll put this on the todo list.

@leeper leeper closed this in eb30d7d Mar 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.