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

invalid cross-device link. Get it here: #3

Closed
rasoso opened this issue Nov 2, 2014 · 6 comments
Closed

invalid cross-device link. Get it here: #3

rasoso opened this issue Nov 2, 2014 · 6 comments
Assignees
Labels

Comments

@rasoso
Copy link

rasoso commented Nov 2, 2014

Hi!

Thanks for ah!

Got this error message on linux:
"Cannot save trace: rename /tmp/ah193667227 /home//.ah/traces/39bdec46c4709dc800a82c8c163357a4: invalid cross-device link. Get it here: /tmp/ah193667227"

/home and /tmp are on different devices.

That's EXDEV for rename(2):
"EXDEV oldpath and newpath are not on the same mounted filesystem."

Is it possible to change the directory for temporary files from /tmp to /home, or other workaround?

@9seconds
Copy link
Owner

9seconds commented Nov 2, 2014

Sure. Will fix it pretty quick.

@9seconds 9seconds added the bug label Nov 2, 2014
@9seconds 9seconds self-assigned this Nov 2, 2014
@9seconds
Copy link
Owner

9seconds commented Nov 2, 2014

Hi, @rasoso could you please check that it works now?
https://www.dropbox.com/sh/d4vpkc8rh617ctl/AACDBH9KFJdsFgI6cXwepkgTa?dl=0

@rasoso
Copy link
Author

rasoso commented Nov 2, 2014

tested it ok by updating from repo (to a0cf1de) and recompiling it.

Thanks, that was quick indeed.

Is it possible to change defaultTmpDir , for example in order to use in-memory tmpfs rather than disk without having to recompile?

@rasoso
Copy link
Author

rasoso commented Nov 2, 2014

More details about the test:

  • no more error message (ok)
  • trace is moved from ~/.ah/tmp/ to ~/.ah/traces/ (ok)
  • trace is removed from ~/.ah/traces/ (ok?)

@9seconds
Copy link
Owner

9seconds commented Nov 2, 2014

No, it is not ok (last step of the test). Please check the master.

Also, I have added an argument -m which allows you to set a temporary directory to work with. I think it is better to default to good old /tmp and give a possibility to set temporary directory explicitly if it won't work with one's setup for some reasons.

9seconds added a commit that referenced this issue Nov 2, 2014
- Execution problems on OS X (issue #1)
- Possibility to set custom temporary directory (issue #3)
@9seconds
Copy link
Owner

9seconds commented Nov 2, 2014

I close this issue if it is ok. $HOME and /tmp on the same mount point looks reasonable default for me since it is default for Ubuntu and OS X, and after some thoughts I do not really want to use another temporary directory. But of course I am open for discussion of any kind!

@9seconds 9seconds closed this as completed Nov 2, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants