Re: filename ends with a space, cannot use the --targets option.

--00000000000053d2d806388d7311
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Den fre 27 juni 2025 kl 15:08 skrev Branko =C4=8Cibej <[email protected]>:

> On 27. 6. 25 11:54, Daniel Sahlberg wrote:
>
> Den tis 17 juni 2025 kl 15:58 skrev Branko =C4=8Cibej <[email protected]>:
>
>> TL;DR: Leave well enough alone. One can already do any amount of mischie=
f
>> 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? Currentl=
y:
> [[[
> $ 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?
>

To document that there are some filenames which, while valid in the file
system, are not possible to use in the file pointed to by --targets.
Basically, have something to point at and say "works as expected" when the
question from the original poster comes up again.

Cheers,
Daniel

--00000000000053d2d806388d7311
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><div dir=3D"ltr">Den fre 27 juni 2025 kl =
15:08 skrev Branko =C4=8Cibej &lt;<a href=3D"mailto:[email protected]" targe=
t=3D"_blank">[email protected]</a>&gt;:</div><div class=3D"gmail_quote"><blo=
ckquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left=
:1px solid rgb(204,204,204);padding-left:1ex"><u></u>

 =20
   =20
 =20
  <div>
    <div>On 27. 6. 25 11:54, Daniel Sahlberg
      wrote:<br>
    </div>
    <blockquote type=3D"cite">
     =20
      <div dir=3D"ltr">
        <div dir=3D"ltr">
          <div dir=3D"ltr">
            <div dir=3D"ltr">Den tis 17 juni 2025 kl 15:58 skrev Branko
              =C4=8Cibej &lt;<a href=3D"mailto:[email protected]" target=3D"=
_blank">[email protected]</a>&gt;:</div>
            <div class=3D"gmail_quote">
              <blockquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px=
 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
                <div>
                  <div>TL;DR: Leave well enough alone. One can already
                    do any amount of mischief with a Unix shell, or by
                    creating files programmatically. There&#39;s really no
                    need to add one more avenue for confusion by
                    &quot;fixing&quot; <font face=3D"monospace">--targets</=
font>.</div>
                </div>
              </blockquote>
              <div><br>
              </div>
              <div>Makes sense to me. Should we document this some way
                in svn help? Currently:</div>
              <div>[[[</div>
              <div>$ svn help add</div>
              <div>[...]</div>
              <div>
                <div>Valid options:</div>
                <div>=C2=A0 --targets ARG=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=
 =C2=A0 : pass contents of file
                  ARG as additional args</div>
                <div>=C2=A0[...]</div>
              </div>
              <div>]]]</div>
              <div><br>
              </div>
              <div>We could update the svnbook as well, it currently
                says:</div>
              <div>[[[</div>
              <div><code style=3D"color:rgb(0,0,0)">--targets</code><span s=
tyle=3D"color:rgb(0,0,0);font-family:serif;font-size:medium">=C2=A0</span><=
em style=3D"color:rgb(0,0,0);font-family:serif;font-size:medium"><code>FILE=
NAME</code></em></div>
              <div>
                <dl>
                  <dd style=3D"color:rgb(0,0,0);font-family:serif;font-size=
:medium;line-height:1.33em">
                    <p style=3D"line-height:1.33em">Tells Subversion to
                      read additional target paths for the operation
                      from=C2=A0<em><code>FILENAME</code></em>.=C2=A0<em><c=
ode>FILENAME</code></em>=C2=A0should
                      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.</p>
                  </dd>
                  ]]]
                </dl>
              </div>
              <div><br>
              </div>
            </div>
          </div>
        </div>
      </div>
    </blockquote>
    <br>
    <br>
    I&#39;m sorry, what&#39;s there to fix? </div></blockquote><div><br></d=
iv><div>To document that there are some filenames which, while valid in the=
 file system, are not possible to use in the file pointed to by --targets. =
Basically, have something to point at and say &quot;works as expected&quot;=
 when the question from the original poster comes up again.</div><div><br><=
/div><div>Cheers,</div><div>Daniel</div><div><br></div></div></div>
</div>

--00000000000053d2d806388d7311--