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

AXL doesn’t work on empty directories... should it? #39

Open
gonsie opened this issue May 22, 2019 · 2 comments
Open

AXL doesn’t work on empty directories... should it? #39

gonsie opened this issue May 22, 2019 · 2 comments

Comments

@gonsie
Copy link
Member

gonsie commented May 22, 2019

No description provided.

@adammoody
Copy link
Contributor

Are you asking about whether we should allow an app to transfer a directory?

@tonyhutter
Copy link
Collaborator

This came up because #42 allows you to specify source and destination directories for AXL_Add() as a convenience. So if you had:

./dir1/file1
./dir1/file2
./dir1/file3
./dir1/dir2/file4

...you could just do AXL_Add(id, "./dir1", dest), rather than doing an AXL_Add() on every file. The issue is that there isn't a way to copy an empty directory in AXL. So if you had...

./dir1/file1
./dir1/file2
./dir1/file3
./dir1/dir2/file4
./dir1/dir3/

dir3/ wouldn't get copied. I think we should add in empty directory copies for completeness. That way we can say that AXL_Add() follows the same semantics as cp -r, which is easy for a user to understand. This should be a super low priority task though, as I doubt there's any users that would need it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants