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