Hi Tobias,

When WebClient was added, HTTPSocket was modified to use WebClient when 
it's present, but still work when it's not. I simply followed that path.
WebClient still has no dependencies in the Trunk, so one should be able to
safely unload it.
But sure, we can just "nuke" HTTPSocket & co. in favor of WebClient.

Levente

On Thu, 22 Mar 2018, Tobias Pape wrote:

> Hi Levente
>
>
> I really like the reuse-idea.
> Also, WebClient is part of Squeak/Trunk since 4.5 I think. I think it is safe to assume that it's there. Let's rather completely deprecate HTTPSocket instead :)
>
> Best regards
> 	-Tobias
>> On 22.03.2018, at 19:33, commits at source.squeak.org wrote:
>> 
>> Levente Uzonyi uploaded a new version of Monticello to project The Trunk:
>> http://source.squeak.org/trunk/Monticello-ul.679.mcz
>> 
>> ==================== Summary ====================
>> 
>> Name: Monticello-ul.679
>> Author: ul
>> Time: 22 March 2018, 7:32:51.372118 pm
>> UUID: 7f5a3bf7-0cda-4200-b035-d9ce789f19d5
>> Ancestors: Monticello-ul.678
>> 
>> - enable #useSharedWebClientInstance by default when WebClient is present
>> 
>> =============== Diff against Monticello-ul.678 ===============
>> 
>> Item was changed:
>>  ----- Method: MCHttpRepository class>>useSharedWebClientInstance (in category 'preferences') -----
>>  useSharedWebClientInstance
>>
>>  	<preference: 'Use shared WebClient instance'
>>  		category: 'Monticello'
>>  		description: 'When true, use a shared WebClient instance to speed up downloads from MCHttpRepositories. Requires WebClient to be present.'
>>  		type: #Boolean>
>> + 	^UseSharedWebClientInstance ifNil: [ Smalltalk hasClassNamed: #WebClient ]!
>> - 	^UseSharedWebClientInstance ifNil: [ false ]!
>> 
>>