The Inbox: Installer-Core-cmm.380.mcz

> A new version of Installer-Core was added to project The Inbox:
> http://source.squeak.org/inbox/Installer-Core-cmm.380.mcz
>
> ==================== Summary ====================
>
> Name: Installer-Core-cmm.380
> Author: cmm
> Time: 13 August 2013, 5:56:00.893 pm
> UUID: 565892cf-7178-4806-8fb1-63523b83de58
> Ancestors: Installer-Core-cmm.379
>
> Fix for using Installer with MCDirectoryRepository's.
>
> =============== Diff against Installer-Core-cmm.379 ===============
>
> Item was changed:
>   ----- Method: InstallerMonticello>>project: (in category 'accessing') -----
>   project: name
>
>         project := name.
>         packages := nil.
>
>         (mc respondsTo: #location:) ifTrue:[ mc := mc copy location: root , name ].
> +       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root / name ].
> -       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root ,'/', name ].

>> UUID: 565892cf-7178-4806-8fb1-63523b83de58
>> Ancestors: Installer-Core-cmm.379
>>
>> Fix for using Installer with MCDirectoryRepository's.
>>
>> =============== Diff against Installer-Core-cmm.379 ===============
>>
>> Item was changed:
>>   ----- Method: InstallerMonticello>>project: (in category 'accessing') -----
>>   project: name
>>
>>         project := name.
>>         packages := nil.
>>
>>         (mc respondsTo: #location:) ifTrue:[ mc := mc copy location: root ,
>> name ].
>> +       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root
>> / name ].
>> -       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root
>> ,'/', name ].
>
> IIRC the nasty thing about this kind of bug is that it works on Linux
> (but not on any other OS that doesn't use / as a separator, like
> Windows).
>
> frank
>

>
>
>
> On 8/14/13 4:44 AM, "Frank Shearar" <[hidden email]> wrote:
>
>>> UUID: 565892cf-7178-4806-8fb1-63523b83de58
>>> Ancestors: Installer-Core-cmm.379
>>>
>>> Fix for using Installer with MCDirectoryRepository's.
>>>
>>> =============== Diff against Installer-Core-cmm.379 ===============
>>>
>>> Item was changed:
>>>   ----- Method: InstallerMonticello>>project: (in category 'accessing') -----
>>>   project: name
>>>
>>>         project := name.
>>>         packages := nil.
>>>
>>>         (mc respondsTo: #location:) ifTrue:[ mc := mc copy location: root ,
>>> name ].
>>> +       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root
>>> / name ].
>>> -       (mc respondsTo: #directory:) ifTrue: [ mc := mc copy directory: root
>>> ,'/', name ].
>>
>> IIRC the nasty thing about this kind of bug is that it works on Linux
>> (but not on any other OS that doesn't use / as a separator, like
>> Windows).
>>
>> frank
>>
>
>
> What about using FileDirectory slash ?