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

mkdir and copy interpret rel URLs differently #457

Open
andre-merzky opened this issue Aug 26, 2015 · 5 comments
Open

mkdir and copy interpret rel URLs differently #457

andre-merzky opened this issue Aug 26, 2015 · 5 comments

Comments

@andre-merzky
Copy link
Member

f = File('ssh://stampede/work/user/out.dat')
f.copy('/tmp/data/out.dat', CREATE_PARENTS)

that code is semantically not expressing a remote transfer, as the copy target is not an abs URL, so is interpreted in the f.url context, ie. on stampede. That indeed works for the CREATE_PARENTS interpretation, but the file transfer will attempt to move the file to localhost...

@vivek-bala
Copy link
Contributor

Hey Andre, is this issue address in any commit or PR?

@andre-merzky
Copy link
Member Author

No, this is still open. We work around this by always using full URLs, but that behavior is surprising (to say the least) for unsuspecting users and needs fixing.

@vivek-bala
Copy link
Contributor

For the example above, I think I would have expected the output to be copied to localhost and not on Stampede, FWIW :) But I mostly personally prefer full URLs wherever possible.

@andre-merzky
Copy link
Member Author

For the example above, I think I would have expected the output to be copied to localhost and not on Stampede, FWIW :)

Well, the holy SAGA spec says otherwise ... ;-)

@vivek-bala
Copy link
Contributor

Alrighty! Leaving this ticket open in that case.

@andre-merzky andre-merzky removed their assignment Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants