<div dir="ltr"><div>Then please go ahead and create a new version directly if you don't mind :)</div><div>I'm still getting sweaty palms each time Monticello warns me about ancestry while committing...</div><div><br></div><div>Thank you!</div><div>Tom<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Oct 4, 2020 at 6:07 PM Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu">leves@caesar.elte.hu</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">On Sun, 4 Oct 2020, Tom Beckmann wrote:<br>
<br>
> Hi Levente,<br>
> <br>
> good catch! Yes that would indeed be a better solution. Would you like me to submit a new version to the inbox or will you just move this to treated and create an according version in trunk directly? :)<br>
<br>
Whichever you prefer. :)<br>
<br>
<br>
Levente<br>
<br>
> <br>
> Best,<br>
> Tom<br>
> <br>
> On Sun, Oct 4, 2020 at 6:00 PM Levente Uzonyi <<a href="mailto:leves@caesar.elte.hu" target="_blank">leves@caesar.elte.hu</a>> wrote:<br>
>       Hi Tom,<br>
><br>
>       On Sat, 3 Oct 2020, <a href="mailto:commits@source.squeak.org" target="_blank">commits@source.squeak.org</a> wrote:<br>
><br>
>       > A new version of Monticello was added to project The Inbox:<br>
>       > <a href="http://source.squeak.org/inbox/Monticello-tobe.729.mcz" rel="noreferrer" target="_blank">http://source.squeak.org/inbox/Monticello-tobe.729.mcz</a><br>
>       ><br>
>       > ==================== Summary ====================<br>
>       ><br>
>       > Name: Monticello-tobe.729<br>
>       > Author: tobe<br>
>       > Time: 3 October 2020, 12:57:12.690975 pm<br>
>       > UUID: cbe1a05f-97e6-4780-9762-824be701f76a<br>
>       > Ancestors: Monticello-ul.728<br>
>       ><br>
>       > Include a dot when matching against extensions<br>
>       ><br>
>       > Packages of extension-less formats (e.g. tonel) will otherwise match for e.g. the MCStReader in MCReader>>#canReadFileNamed: if their name happens to end on an "st", e.g. "Roassal3-Sunburst"<br>
>       ><br>
>       > =============== Diff against Monticello-ul.728 ===============<br>
>       ><br>
>       > Item was changed:<br>
>       >  ----- Method: MCVersionReader class>>canReadFileNamed: (in category 'testing') -----<br>
>       >  canReadFileNamed: fileName<br>
>       > +     ^ (fileName endsWith: '.', self extension)!<br>
>       > -     ^ (fileName endsWith: self extension)!<br>
><br>
>       This is a reasonable change. But the very same method is implemented by<br>
>       the superclass MCReader. So, I think this method should be removed and<br>
>       the change should be applied to MCReader class>>canReadFileNamed:.<br>
>       What do you think?<br>
> <br>
><br>
>       Levente<br>
> <br>
> <br>
><br>
</blockquote></div>