<div dir="ltr">I've decided for the moment to put the tests in the PharoCompatibility package, since they are too close of Pharo's AST.<div>I've tried to make them work in squeak but I've found too many incompatibilities in the AST and some in the collections that would make the tests too verbose to make them reusable...</div><div>Maybe we can do an iteration afterwards.</div><div><br></div><div>Cheers,</div><div>Guille</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 9, 2019 at 5:21 PM Guillermo Polito <<a href="mailto:guillermopolito@gmail.com">guillermopolito@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi all,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 8, 2019 at 2:17 AM David T. Lewis <<a href="mailto:lewis@mail.msen.com" target="_blank">lewis@mail.msen.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> <br><br>
On Tue, May 07, 2019 at 03:36:32PM -0700, Eliot Miranda wrote:<br><br>> In VMMaker.oscog-GAP.2533 I would prefer to see printString instead of<br>
> asString.  asString is a horrible hack.  printString says what we mean.<br></blockquote><div><br></div><div>Sure. I've submitted another version to the inbox:</div><div><br></div><div><div>Name: VMMaker.oscog-GAP.2534</div><div>Author: GAP</div><div>Time: 9 May 2019, 5:12:22.055089 pm</div><div>UUID: cd6aae16-6f20-4ddb-9328-64b19b413cb1</div><div>Ancestors: VMMaker.oscog-eem.2532</div><div><br></div><div>Convert numeric constants to strings for concatenation using #printString.</div><div>Required for Pharo compatibility, where the arguments of concatenation are not automatically coerced to strings.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> ==== Patch 1 ====<br>
> > Name: VMMaker.oscog-GAP.2533<br>
> > Author: GAP<br>
> > Time: 7 May 2019, 10:57:16.034354 am<br>
> > UUID: 3209319f-60f3-4586-8d03-cfac3c9fddee<br>
> > Ancestors: VMMaker.oscog-eem.2532<br>
> ><br>
> > Convert numeric constants to strings for concatenation.<br>
> > Required for Pharo compatibility, where the arguments of concatenation are<br>
> > not automatically coerced to strings.<br>
> ><br>
> <br>
> I trust you on these.  I'm not really in a position to test this out right<br>
> now.  I suggest you push the  VMMakerCompatibilityForPharo6 packages to<br>
> VMMaker as soon as you'd like.  You can write VMMaker right?<br>
> <br>
<br>
+1<br>
<br>
Guille, if you want commit access to the repo, you can make an account<br>
on <a href="http://source.squeak.org" rel="noreferrer" target="_blank">source.squeak.org</a> and I'll add you the the VMMaker project. Or if<br>
it's easier, just say which packages you are ready to move and either<br>
I or Eliot (or any of the others with commit access) will move it to<br>
VMMaker for you.<br>
<br>
I note that VMMaker.oscog-GAP.2533 package is part of the main<br>
VMMaker.oscog package. If you need to have that tested on Squeak I<br>
can probably do that for you within the next couple of days if no<br>
one else gets to it first.<br></blockquote><div><br></div><div>Well, the change I did in that package was fairly small, and I'm using squeak to generate also some parts of the VM to compare to what I generate from Pharo.</div><div>So, I'd say I've tried it myself already.</div><div><br></div><div>For the Pharo specific package, I've already uploaded it to the VMMaker repository.</div><div>I had already rights to do it :).</div><div><br></div><div><div>Name: VMMakerCompatibilityForPharo6-GuillermoPolito.11</div><div>Author: GuillermoPolito</div><div>Time: 9 May 2019, 5:15:50.00473 pm</div><div>UUID: e1a685a9-d744-0d00-b3c1-ad600f1a7e9a</div><div>Ancestors: VMMakerCompatibilityForPharo6-GuillermoPolito.10</div><div><br></div></div><div>With these changes I can already compile and run Cog and almost all tests pass :)</div><div><br></div><div><div><img src="cid:ii_jvgsq2gp0" alt="image.png" width="425" height="401"><br></div></div><div><br></div><div>I have also a couple dozen tests about AST translation and C generation, mainly about those parts that are covered differently between Pharo's and Squeak's ASTs.</div><div>Would you like that I push them inside the VMMaker package or make a separate package?</div><div><br></div><div>Thanks!</div><div>Guille</div></div><div dir="ltr" class="gmail-m_-455395653493858782gmail_signature"><div dir="ltr"><div></div></div></div></div></div></div>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><table cellspacing="0" cellpadding="0" style="border-collapse:collapse">
<tbody>
<tr>
<td valign="top" style="width:86px;height:84px;border-style:solid;border-width:0px 1px 0px 0px;border-color:transparent rgb(0,0,0) transparent transparent;padding:4px">
<p style="margin:0px;min-height:14px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)"> </font><span style="font-family:Helvetica;font-size:10px"> </span><span style="font-family:Helvetica;font-size:10px"> </span><img src="https://www.cristal.univ-lille.fr//videos/presentation.jpg" width="200" height="112"><img src="https://intranet.cnrs.fr/Cnrs_pratique/communiquer/guides/PublishingImages/CNRS-filaire-Bichro-CMJN.jpg" alt="" style="font-size: 12.8px; margin: 5px;" width="96" height="96"></p>















</td><td valign="top" style="width:378px;height:84px;border-style:solid;border-width:0px 0px 0px 1px;border-color:transparent transparent transparent rgb(0,0,0);padding:4px"><p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)">Guille Polito</font></p><p style="margin:0px"><span style="color:rgb(0,0,0);font-family:Helvetica;font-size:10px">Research Engineer</span></p><p style="margin:0px;font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0);min-height:10px"><font></font></p><p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)">Centre de Recherche en Informatique, Signal et Automatique de Lille</font></p><p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)">CRIStAL - UMR 9189</font></p>
<p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)">French National Center for Scientific Research - <font><u><a href="http://www.cnrs.fr" target="_blank">http://www.cnrs.fr</a></u></font></font></p><p style="margin:0px;font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0);min-height:10px"><br></p>
<p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)"><b>Web:</b> <a href="http://guillep.github.io" target="_blank"><font><u>http://guillep.github.io</u></font></a></font></p>
<p style="margin:0px"><font face="Helvetica" color="#000000" style="font-stretch:normal;font-size:10px;line-height:normal;font-family:Helvetica;color:rgb(0,0,0)"><b>Phone: </b>+33 06 52 70 66 13</font></p>
</td>
</tr>
</tbody>
</table></div></div></div></div>