[Seaside-dev] Remove / rename Zinc-Seaside

Max Leske maxleske at gmail.com
Thu May 7 08:02:12 UTC 2020


On 7 May 2020, at 9:13, Johan Brichau wrote:

> Hey Max,
>
> The Zinc-Seaside package is indeed moved to the Seaside project a while back: https://github.com/SeasideSt/Seaside/issues/1050 <https://github.com/SeasideSt/Seaside/issues/1050>
>
> Do you have a sequence I can try to see what the problem still is?

See the mail from Sven and my response, I think we can consider the name clash when loading both baselines solved.

This leaves the question of renaming the package. I think it is more natural to call the package Seaside-Zinc but I can also see Sven's point.


Cheers,
Max

>
> Renaming it is a good idea anyway.
>
> Cheers
> Johan
>
>> On 7 May 2020, at 09:07, Max Leske <maxleske at gmail.com> wrote:
>>
>> Hi everyone,
>>
>> I want to propose that we clean up the current situation with the Zinc-Seaside package. While it may not be much of a problem when we run tests or talk about the default setup it becomes a real problem when your baseline loads both Seaside3 and ZincHTTPComponents.
>>
>> The problem partly lies with Metacello (I already talked to Dale about it), which calculates the packages to load for a project by also including all loaded packages from that project, even when that package is not part of the
>> loading instruction for the project. That means, when I load Seaside3, which loads Zinc-Seaside, and then ZincHTTPComponents, Zinc-Seaside will be loaded again from ZincHTTPComponents because Zinc-Seaside already exists in the image.
>>
>> There's no way to solve this issue in the baseline (not even with fake dependency chains). From my point of view we, therefore, have two options:
>> - rename Zinc-Seaside in the Seaside3 image to Seaside-Zinc. That would be good because it matches the Seaside prefix (I think we should do this anyway). This would also make it possible for the Zinc-Seaside package to remain in the Zinc project if we decide we want that.
>> - delete Zinc-Seaside from the Zinc project (I think we should do this too), to avoid confusion.
>>
>> I talked to Sven two or three weeks ago and he thought that "ownership" of Zinc-Seaside currently resided with the Seaside project. So the question is, is there any reason to keep Zinc-Seaside in the Zinc project?
>>
>>
>> Please let me know what you think. I'll be happy to provide the pull requests to both projects.
>>
>>
>> Cheers,
>> Max_______________________________________________
>> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <http://lists.squeakfoundation.org/pipermail/seaside-dev/attachments/20200507/4df2aa08/attachment.sig>


More information about the seaside-dev mailing list