Why the options "Allow ambiguous URLs" and "Allow ambiguous UUIDs" are enabled by default?

16 views
Skip to first unread message

Pavel Lyalyakin

unread,
Jun 7, 2021, 2:50:02 PMJun 7
to tortois...@googlegroups.com
Hello,

I see that the option "Allow ambiguous URLs" and "Allow ambiguous UUIDs" are by default enabled in TortoiseSVN 1.14.1 (and most likely in all earlier versions). However, the description of them in the manual is written in a fashion as if they were by default disabled (i.e., unchecked):
[[[
If you don't, leave it unchecked to improve performance.
]]]

Do I understand correctly that these options are enabled by default in all TortoiseSVN versions? Is there a rationale for enabling these options by default?

--
With best regards,
Pavel Lyalyakin
VisualSVN Team

Stefan

unread,
Jul 13, 2021, 1:17:33 AMJul 13
to TortoiseSVN-dev
On Monday, June 7, 2021 at 8:50:02 PM UTC+2 Pavel Lyalyakin wrote:

Do I understand correctly that these options are enabled by default in all TortoiseSVN versions? Is there a rationale for enabling these options by default?


yes, these options are enabled by default. That's because this is the safe option. When this option is disabled but the user connects to wrongly configured/set-up repos then there will be crashes and completely wrong logs being shown.
So the user can disable these options to get a little more performance, but then the user is responsible to make sure that uuids and urls are not ambiguous, or life with the consequences...

If it were the other way around, users would get crashes/wrong logs and not know why, so they either would just think that TSVN is unstable/crap or complain here about "the bug".

Stefan

Daniel Sahlberg

unread,
Jul 16, 2021, 5:36:07 PMJul 16
to TortoiseSVN-dev
I believe the wording is a bit counter intuitive because it implies, at least to me, that it is by default ("leave it") unchecked.

I would propose a rewording like this:

[[[
Index: doc/source/en/TortoiseSVN/tsvn_dug/dug_settings_logcache.xml
===================================================================
--- doc/source/en/TortoiseSVN/tsvn_dug/dug_settings_logcache.xml        (revision 29091)
+++ doc/source/en/TortoiseSVN/tsvn_dug/dug_settings_logcache.xml        (working copy)
@@ -43,7 +43,7 @@
             the same URL for all repositories. Older versions of
             <literal>svnbridge</literal> would do this. If you need to
             access such repositories you will have to check this option.
-            If you don't, leave it unchecked to improve performance.
+            If you don't, uncheck it to improve performance.
           </para>
         </listitem>
       </varlistentry>
@@ -57,7 +57,7 @@
             reasons this is a bad idea - a UUID should be
             <emphasis>unique</emphasis>. However, the log cache will
             still work in this situation if you check this box.
-            If you don't need it, leave it unchecked to improve performance.
+            If you don't need it, uncheck it to improve performance.
           </para>
         </listitem>
       </varlistentry>
]]]

Kind regards,
Daniel Sahlberg

Reply all
Reply to author
Forward
0 new messages