[Seaside-dev] Seaside-Core-pmm.693 and Seaside-HTML5-pmm.19 define same method

[Dale Henrichs]

Avi,

I am not aware of anything else that is missing ... in the end it looks 
like there was a missed package commit and not a loss of information for 
either Issue 640 and Issue 641.

Since there is a big time shift between the west coast of the US and 
Europe, I try to put as much information into mail as possible. I don't 
get to rejoin the conversation for 12 hours:)

Dale

On 02/09/2011 11:49 PM, Avi Shefi wrote:
> Dale,
> Are there any more missing commits?
> Looking at the repo, issue 640 was reverted, and I accidentally
> submitted a fix to issue 641.
> Cheers,
> Avi.
> On Thu, Feb 10, 2011 at 8:34 AM, Philippe Marschall
> <philippe.marschall at gmail.com <mailto:philippe.marschall at gmail.com>> wrote:
>
>     2011/2/10 Dale Henrichs <dhenrich at vmware.com
>     <mailto:dhenrich at vmware.com>>:
>      > As of right now, those are the latest versions of the two
>     packages and the
>      > method WABaseElement>>target: is defined in both, so I'm not sure
>     which
>      > package is the correct one ...
>      >
>      > I found this out by accident when I moved the methods between
>      > Grease-Pharo-Core and Seaside-Pharo-Core....
>      >
>      > It is beginning to look like some packages were not dirtied
>     correctly during
>      > development...or....some mcz files were lost:
>      >
>      > Name: Seaside-Core-pmm.693
>      > Author: pmm
>      > Time: 5 February 2011, 10:32:16 am
>      > UUID: f138cacb-220e-418d-91b2-0f01b792f065
>      > Ancestors: Seaside-Core-as.692
>      >
>      > Name: Seaside-HTML5-pmm.19
>      > Author: pmm
>      > Time: 8 February 2011, 10:17:04 pm
>      > UUID: b0de019c-bfae-424d-9ac4-fb54fecfaca5
>      > Ancestors: Seaside-HTML5-as.18
>      >
>      > So it's possible that a later Seaside-Core mcz file went missing...
>
>     Fixed, thanks.
>
>      > Another odd thing about Seaside-HTML5 that I haven't figured out
>     yet is that
>      > WAOrderedListTag>>type: in the image, doesn't match the version
>     that is in
>      > the package?
>
>     That's strange.
>
>      > I'll spend a little more time today checking for more oddities
>     (check for
>      > differences for all packages...)
>      >
>      > But I will hold off the repackaging of the WAUrlEncoder methods until
>      > things...
>      >
>      > Looking at Issue 640, there were later versions of Seaside-Core
>     committed
>      > somewhere:
>      >
>      > Name: Seaside-Core-as.694
>      > Author: as
>      > Time: 6 February 2011, 9:08:58 pm
>      > UUID: 5450d506-8db5-414b-996a-b434ec532cea
>      > Ancestors: Seaside-Core-pmm.693
>      >
>      > Name: Seaside-Core-as.695
>      > Author: as
>      > Time: 6 February 2011, 9:39:13 pm
>      > UUID: f017aa59-fdf2-604e-84c2-776bfdab18c2
>      > Ancestors: Seaside-Core-as.694
>      >
>      > These files are no longer present in the Seaside30 repository (if
>     they ever
>      > were), so perhaps, Philippe's changes got lost along the way
>     (committed
>      > after Seaside-Core-as.695 possibly)...not a good idea to delete
>     packages, in
>      > case a mistake like this one is made...hard to tell what other
>     work was lost
>      > by the deletion... if indeed mcz files were deleted...
>
>     As Julian said, they were moved. I don't understand why they are in
>     the repo again.
>
>      > Also it looks like there is active seaside30 development work
>     going on on
>      > today (Issue 641) is this something that I should wait to be
>     finished or
>      > should I try to exclude the additional work or what ... Normally
>     I can get
>      > my snapshot of mcz files done in an hour or two, but since the
>     packaging is
>      > messed up and possibly mcz files missing I'll have to wait until
>     at least
>      > tomorrow before I pick up the latest mcz files ...
>
>     I was intentionally not publishing my changes and just attaching them
>     to the issue for this reason.
>
>     Cheers
>     Philippe
>     _______________________________________________
>     seaside-dev mailing list
>     seaside-dev at lists.squeakfoundation.org
>     <mailto:seaside-dev at lists.squeakfoundation.org>
>     http://lists.squeakfoundation.org/mailman/listinfo/seaside-dev
>
>