-
Notifications
You must be signed in to change notification settings - Fork 12
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
is ** pattern supported ? #20
Comments
Hello @haoadoresorange Yep, I just checked this and it seems to work fine: ➜ ~ mkdir tmp-folder
➜ ~ touch tmp-folder/tmp.txt
➜ ~ mkdir tmp-folder/tmp-folder2
➜ ~ tree tmp-folder
tmp-folder
├── tmp-folder2
└── tmp.txt
1 directory, 1 file
➜ ~ echo '**' > tmp-folder/.dropboxignore
➜ ~ cd tmp-folder
➜ tmp-folder tree .
.
├── tmp-folder2
└── tmp.txt
1 directory, 1 file
➜ tmp-folder dropboxignore ignore .
Total number of ignored files: 2
Total number of ignored folders: 2 There is probably and issue on file counting but the files are ignored from dropbox. |
any news @sp1thas ? |
Hello @haoadoresorange , you are right, based on current implementation, double asterisks are not supported. On the other hand, a recursive match is enabled by mistakes, therefore, I will start working on this asap in order to fix the wrong recursive matching and to support patterns like Thanks for your feedback. |
So I looked into it a bit and see that we have a very simple solution here:
Tell me if it covers everything, I will make a PR. |
@haoadoresorange I totally agree about the second and the third bullet of you comment. It's not really clear to me how we gonna use note 1: we must keep the format of |
I just tested it, it isn't working (I think), because of one line that seems pretty weird
overall however, I think the script does what you did in your script already. |
Hi, this is a really nice utility to have and thank you very much for sharing it ! I can confirm that
Is this intentional? I don't know what these files and folders do and I am wondering if it is safe to auto-ignore them. If so, it would be nice to have that in the documentation so totally clueless folk like me are not caught unawares! |
Thanks @haoadoresorange for your effort, unfortunately I didn't had the time to investigate the case of @smalltimer thank you so much for your feedback. dropboxignore should not auto-ignore anything, therefore I would like to ask you to provide a reproducible example of your case. Please provide some the basic folder structure, the current directory, the content of the |
Hi actually stopped using dropboxignore a while ago ): I switched to insync which allow an ignore file for dropbox and other services. |
is ** pattern supported (any location) ?
The text was updated successfully, but these errors were encountered: