[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: TortoiseOverlays 1.1.0

From: Adrian Buehlmann <adrian_at_cadifra.com>
Date: Thu, 18 Mar 2010 20:20:02 +0100

On 18.03.2010 18:21, Stefan Küng wrote:
> On 18.03.2010 14:25, Adrian Buehlmann wrote:
>> (cc-ing tortoisehg-develop_at_lists.sourceforge.net)
>>
>> Apparently, if
>>
>> HKCU/Software/TortoiseOverlays, ShowUnversionedOverlay
>> HKCU/Software/TortoiseOverlays, ShowIgnoredOverlay
>>
>> are not set (in the registry of the user!), then these icons are now
>> dropped, starting with version 1.1.0 of TortoiseOverlays.
>
> Are you sure?
> If the registry value is not set (i.e., does not exist), the overlays
> are not dropped (at least that's how it is supposed to work).

I wouldn't call that "work". They are dropped if these values are zero.

BTW, this was introduced in the very same change r18986 that was sold as
switching the dropping priorities:

http://code.google.com/p/tortoisesvn/source/detail?spec=svn18986&r=18986

> Only if you create the registry key and set its value to 0, *then* the
> overlays are dropped.

Sure. I see that in the code.

So, if a user that happens to have TortoiseSVN installed sets his
setting using the GUI of TortoiseSVN to not show the Ignored or
Unversioned overlays, they are dropped by TortoiseOverlays.

Is this the way a shared component is supposed to work?

TortoiseSVN (=a client of TortoiseOverlays) decides to drop an overlay
shared with TortoiseHg?

Great. I might as well put a GUI in the TortoiseHg installer to "drop"
the Locked, Readonly, and Ignored overlays by removing them from
"Software\Microsoft\Windows\CurrentVersion\Explorer\ShellIconOverlayIdentifiers".

Or I could add a button to the TortoiseHg shell extension settings tool
labeled "Redeem overlays wasted for TortoiseSVN", acting on
"Software\Microsoft\Windows\CurrentVersion\Explorer\ShellIconOverlayIdentifiers".

Or leave away these GUI things altogether and remove them
unconditionally on install. After all, we (=TortoiseHg) happen to not
need them.

Do you see the problem?

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2461813

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2010-03-18 20:20:27 CET

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.