Re: trailing spaces in exclude-from file
On Mon, Jan 24, 2022 at 1:08 AM Jürgen Bausa via rsync < rsync@lists.samba.org> wrote: > However, it didn't work in the beginning and took me some time to find > out, that a space at the end (e.g. "+ /mnt/Windows/Users " in the second > line) will be interpreted and thus, the directory is assumed to be "Users " > and not "Users". When working on some filter rules it helps to use --debug=FILTER or --debug=FILTER2 options. (While these are implied by -vv and -vvv respectively, it's better to be more specific in what you want to debug.) The FILTER2 level mentions the add_rule(RULE_HERE) call where a trailing space would be visible prior to the trailing paren, though it might not be immediately obvious. I've just made a change in git that makes rsync output a warning message in either debug-filter level if there is a trailing space or tab on a rule, which will help someone notice it if they didn't intend to specify it. ..wayne.. -- Please use reply-all for most replies to avoid omitting the mailing list. To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Re: trailing spaces in exclude-from file
On Mon, Jan 24, 2022, Paul Slootman via rsync wrote: >On Mon 24 Jan 2022, Jürgen Bausa via rsync wrote: >> >> However, that's just a proposal. But the behavior of trailing spaces is >> something I guess should be corrected. > >Is it? >How would you otherwise specify a space that you *do* intend to be >relevant? If rsync uses pcre, then one or more \s should do it. Bill -- INTERNET: b...@celestial.com Bill Campbell; Celestial Software LLC URL: http://www2.celestial.com/ 6641 E. Mercer Way Mobile: (206) 947-5591 PO Box 820 Fax:(206) 232-9186 Mercer Island, WA 98040-0820 Americans are so enamored of equality that they would rather be equal in slavery than unequal in freedom. -- Alexis de Tocqueville, 1805-1859 -- Please use reply-all for most replies to avoid omitting the mailing list. To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
re: trailing spaces in exclude-from file
On Mon 24 Jan 2022, Paul Slootman via rsync wrote: > >On Mon 24 Jan 2022, Jürgen Bausa via rsync wrote: >> >> However, that's just a proposal. But the behavior of trailing spaces is >> something I guess should be corrected. > >Is it? >How would you otherwise specify a space that you *do* intend to be >relevant? > > >Paul Thats a good point. Seems the problem is more complex than I thought I did not think that someone would intent to have a space at the end of a directory or file name. But this can of course happen. Changing the interpretation of trailing spaces would break backwards compabilty. So at least there should be a warning in the documentation. Maybe there is already one, but I didnt find it. jba -- Please use reply-all for most replies to avoid omitting the mailing list. To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html
Re: trailing spaces in exclude-from file
On Mon 24 Jan 2022, Jürgen Bausa via rsync wrote: > > However, that's just a proposal. But the behavior of trailing spaces is > something I guess should be corrected. Is it? How would you otherwise specify a space that you *do* intend to be relevant? Paul -- Please use reply-all for most replies to avoid omitting the mailing list. To unsubscribe or change options: https://lists.samba.org/mailman/listinfo/rsync Before posting, read: http://www.catb.org/~esr/faqs/smart-questions.html