<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">&lt;<a href="mailto:philippe.marschall@gmail.com">philippe.marschall@gmail.com</a>&gt;</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 &lt;<a href="mailto:dhenrich@vmware.com">dhenrich@vmware.com</a>&gt;:<br>
<div class="im">&gt; As of right now, those are the latest versions of the two packages and the<br>
&gt; method WABaseElement&gt;&gt;target: is defined in both, so I&#39;m not sure which<br>
&gt; package is the correct one ...<br>
&gt;<br>
&gt; I found this out by accident when I moved the methods between<br>
&gt; Grease-Pharo-Core and Seaside-Pharo-Core....<br>
&gt;<br>
&gt; It is beginning to look like some packages were not dirtied correctly during<br>
&gt; development...or....some mcz files were lost:<br>
&gt;<br>
&gt; Name: Seaside-Core-pmm.693<br>
&gt; Author: pmm<br>
&gt; Time: 5 February 2011, 10:32:16 am<br>
&gt; UUID: f138cacb-220e-418d-91b2-0f01b792f065<br>
&gt; Ancestors: Seaside-Core-as.692<br>
&gt;<br>
&gt; Name: Seaside-HTML5-pmm.19<br>
&gt; Author: pmm<br>
&gt; Time: 8 February 2011, 10:17:04 pm<br>
&gt; UUID: b0de019c-bfae-424d-9ac4-fb54fecfaca5<br>
&gt; Ancestors: Seaside-HTML5-as.18<br>
&gt;<br>
&gt; So it&#39;s possible that a later Seaside-Core mcz file went missing...<br>
<br>
</div>Fixed, thanks.<br>
<div class="im"><br>
&gt; Another odd thing about Seaside-HTML5 that I haven&#39;t figured out yet is that<br>
&gt; WAOrderedListTag&gt;&gt;type: in the image, doesn&#39;t match the version that is in<br>
&gt; the package?<br>
<br>
</div>That&#39;s strange.<br>
<div class="im"><br>
&gt; I&#39;ll spend a little more time today checking for more oddities (check for<br>
&gt; differences for all packages...)<br>
&gt;<br>
&gt; But I will hold off the repackaging of the WAUrlEncoder methods until<br>
&gt; things...<br>
&gt;<br>
&gt; Looking at Issue 640, there were later versions of Seaside-Core committed<br>
&gt; somewhere:<br>
&gt;<br>
&gt; Name: Seaside-Core-as.694<br>
&gt; Author: as<br>
&gt; Time: 6 February 2011, 9:08:58 pm<br>
&gt; UUID: 5450d506-8db5-414b-996a-b434ec532cea<br>
&gt; Ancestors: Seaside-Core-pmm.693<br>
&gt;<br>
&gt; Name: Seaside-Core-as.695<br>
&gt; Author: as<br>
&gt; Time: 6 February 2011, 9:39:13 pm<br>
&gt; UUID: f017aa59-fdf2-604e-84c2-776bfdab18c2<br>
&gt; Ancestors: Seaside-Core-as.694<br>
&gt;<br>
&gt; These files are no longer present in the Seaside30 repository (if they ever<br>
&gt; were), so perhaps, Philippe&#39;s changes got lost along the way (committed<br>
&gt; after Seaside-Core-as.695 possibly)...not a good idea to delete packages, in<br>
&gt; case a mistake like this one is made...hard to tell what other work was lost<br>
&gt; by the deletion... if indeed mcz files were deleted...<br>
<br>
</div>As Julian said, they were moved. I don&#39;t understand why they are in<br>
the repo again.<br>
<div class="im"><br>
&gt; Also it looks like there is active seaside30 development work going on on<br>
&gt; today (Issue 641) is this something that I should wait to be finished or<br>
&gt; should I try to exclude the additional work or what ... Normally I can get<br>
&gt; my snapshot of mcz files done in an hour or two, but since the packaging is<br>
&gt; messed up and possibly mcz files missing I&#39;ll have to wait until at least<br>
&gt; 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>