On 27. 6. 25 11:54, Daniel Sahlberg wrote:
Den tis 17 juni 2025 kl 15:58 skrev Branko Čibej <br...@apache.org>:TL;DR: Leave well enough alone. One can already do any amount of mischief with a Unix shell, or by creating files programmatically. There's really no need to add one more avenue for confusion by "fixing" --targets.Makes sense to me. Should we document this some way in svn help? Currently:[[[ $ svn help add [...] Valid options: --targets ARG : pass contents of file ARG as additional args [...] ]]] We could update the svnbook as well, it currently says: [[[ |--targets|/|FILENAME|/ Tells Subversion to read additional target paths for the operation from /|FILENAME|/. /|FILENAME|/ should contain one path per line, with each path expected to use the same encoding and formatting that it would if you had specified it directly as an argument on the command line. ]]]
I'm sorry, what's there to fix? This is perfectly true. Do not confuse "the command line" with "something some shell somewhere allows you to do". --targets is supposed to help scripting, not support nonsense. I've not yet heard one argument for supporting filenames with trailing spaces in those files other than "bash can do it".
Well, it's not bash. -- Brane
