[Seaside-dev] Re: [Seaside Commits] Seaside 2.9: Seaside-Platform-mls.65.mcz

[Julian Fitzell]

I guess that would be fine as long as there are tests defining the
protocol that is expected of the returned FileName object. Without
those tests or a WAFileName object (also with tests), a platform would
have no idea what they needed to implement or whether their
implementation was correct.

Are you doing this because you want to use Seaside-FileSystem or just
because it's there? If the former, go nuts. If the latter, we should
check whether anyone wants to keep it - I don't think we were
currently very happy with the coverage of the Sport Platform and it's
an abstract class with no concrete implementations currently. Maybe we
should just move the FileSystem stuff out to another repository or
delete it altogether. Philippe?

Julian

On Thu, Jun 4, 2009 at 11:28 AM, Michael Lucas-Smith
<mlucas-smith at cincom.com> wrote:
> Can I do what Seaside-Flow and Comet-Core do? where they extend the
> expectations of the supporting platform?
>
> Julian Fitzell wrote:
>>
>> Squeak doesn't have a FileName class and I don't think it's in the
>> ANSI standard so it's not really appropriate in the Platform unless
>> the Platform itself is going to provide WAFileName, I think.
>>
>> Also, I think Seaside-FileSystem is just sort of a proof of concept
>> thing and we shouldn't let it influence anything too much.
>>
>> Julian
>>
>> On Thu, Jun 4, 2009 at 10:54 AM,
>> <squeak-dev-noreply at lists.squeakfoundation.org> wrote:
>>
>>>
>>> Michael Lucas-Smith uploaded a new version of Seaside-Platform to project
>>> Seaside 2.9:
>>> http://www.squeaksource.com/Seaside29/Seaside-Platform-mls.65.mcz
>>>
>>> ==================== Summary ====================
>>>
>>> Name: Seaside-Platform-mls.65
>>> Author: mls
>>> Time: 4 June 2009, 10:54:11 am
>>> UUID: b41a2369-84e9-4ac0-827e-bd69c67d0d05
>>> Ancestors: Seaside-Platform-mls.64
>>>
>>> - Added abstract #filenameNamed: to WAPlatform. The package
>>> Seaside-FileSystem assumed you would have SpFilename, which should be a
>>> platform choice.
>>>
>>> =============== Diff against Seaside-Platform-mls.64 ===============
>>>
>>> Item was added:
>>> + ----- Method: WAPlatform>>filenameNamed: (in category 'file library')
>>> -----
>>> + filenameNamed: aPathString
>>> +       "Return a Filename object for aPathstring."
>>> +       self subclassResponsibility!
>>>
>>>
>>> _______________________________________________
>>> commits mailing list
>>> To unsubscribe, email commits-leave at lists.seaside.st
>>> http://lists.seaside.st/listinfo/commits
>>>
>>>
>>
>> _______________________________________________
>> seaside-dev mailing list
>> seaside-dev at lists.squeakfoundation.org
>> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>>
>>
>
> _______________________________________________
> seaside-dev mailing list
> seaside-dev at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>