SubhashCBS / tortoisesvn

Automatically exported from code.google.com/p/tortoisesvn
0 stars 0 forks source link

remember "Omit externals" setting #664

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
From the mailing list:
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=30881
80

It would be very nice if the value of the "Omit externals" checkbox was 
remembered across dialogs like "Checkout", "Update to revision" and "Export".

Original issue reported on code.google.com by tortoisesvn on 10 Sep 2014 at 3:44

GoogleCodeExporter commented 9 years ago
Alternatively, the default value of the "Omit externals" checkbox could be 
specified as an option - possibly using the already existing (Advanced) setting 
called "IncludeExternals".

Original comment by gweron...@gmail.com on 11 Sep 2014 at 6:44

GoogleCodeExporter commented 9 years ago
Please read the docs first about what the "IncludeExternals" setting does.

Original comment by tortoisesvn on 11 Sep 2014 at 7:02

GoogleCodeExporter commented 9 years ago
So I had:

"By default, TortoiseSVN always runs an update with externals included. This 
avoids problems with inconsistent working copies. If you have however a lot of 
externals set, an update can take quite a while. Set this value to false to run 
the default update with externals excluded. To update with externals included, 
either run the Update to revision... dialog or set this value to true again."

Here are my thoughts:

This flag could be used to also control the initial check/uncheck value of the 
related checkbox each time we open the dialogs "Checkout" and "Update to 
revision". (Perhaps not in "Export", but that is less important anyway.)

The documentation would have to be slightly modified to include this addition. 
Since it would only affect the initial checkbox value, the "Update to revision" 
dialog can still be used to fetch the externals.

This is just a suggestion. A new option would work equally well, or just 
remembering the checkbox value used last time.

Original comment by gweron...@gmail.com on 11 Sep 2014 at 7:32

GoogleCodeExporter commented 9 years ago
Here is my use case:

We use svn:externals to avoid bulky checkouts. I always use 
IncludeExternals=False and bring in externals only when needed.

Thus, when i Update, externals are omitted. Bliss!

However, when I need to do Checkout or Update To Revision, I often forget that 
I have to explicitly check "Omit externals". Doh! Lots of externals rattling 
in.... ;)

Original comment by gweron...@gmail.com on 11 Sep 2014 at 7:40

GoogleCodeExporter commented 9 years ago
This issue was closed by revision r25856.

Original comment by tortoisesvn on 12 Sep 2014 at 6:41

GoogleCodeExporter commented 9 years ago
Hmm, I found a case that is not covered - the auto-update that is done when 
up-to-date check fails on a Commit. 

It would be very nice if the same remembered checkbox value could be honored 
there as well. Or perhaps, since  it is actually most similar to a normal 
Update, it is rather the "Include Externals" advanced setting that should be 
used here?!

Original comment by gweron...@gmail.com on 16 Sep 2014 at 7:58

GoogleCodeExporter commented 9 years ago
Yes, this is a normal update. So if you want that a normal update excludes the 
externals, you should use the advanced setting for this.

Original comment by tortoisesvn on 16 Sep 2014 at 4:34

GoogleCodeExporter commented 9 years ago
Only that it does not currently work that way... May I suggest adding a new 
issue:

Auto-update on Commit out-dated check does not honor advanced setting "Include 
Externals".

Original comment by gweron...@gmail.com on 16 Sep 2014 at 4:48

GoogleCodeExporter commented 9 years ago
No need for a new issue to open: will fix this right now...

Original comment by tortoisesvn on 16 Sep 2014 at 4:57

GoogleCodeExporter commented 9 years ago
Fixed in r25865
https://code.google.com/p/tortoisesvn/source/detail?r=25865

Original comment by tortoisesvn on 16 Sep 2014 at 5:04

GoogleCodeExporter commented 9 years ago
Great, but there seem to be more places where an Update does not honor that 
advanced setting. At least in the "Check for Modifications" dialog when 
right-clicking and choosing "Update".

Original comment by gweron...@gmail.com on 17 Sep 2014 at 10:14

GoogleCodeExporter commented 9 years ago
https://code.google.com/p/tortoisesvn/source/detail?r=25867

Original comment by tortoisesvn on 17 Sep 2014 at 8:15

GoogleCodeExporter commented 9 years ago
(Fixed in r25867.)

Original comment by gweron...@gmail.com on 17 Sep 2014 at 10:58