<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    On 5/6/2019 12:04 AM, Phil B via Cuis-dev wrote:
    <blockquote
cite="mid:CAMJMOeipRS8tOBbTU-b_gcP1aZN5gaQ1DO7+7+AXH_n=XBmyhA@mail.gmail.com"
      type="cite">
      <div dir="ltr">I agree that a parse error is a bug.  But do we
        want it to apply any arbitrary changeset it finds there?  This
        seems like it could cause some confusion/problems down the road
        if someone accidentally drops a random changeset in there.  Keep
        in mind, many of these updates *must* be applied in sequence or
        bad things will happen.</div>
      <br>
      <div class="gmail_quote">
        <div dir="ltr" class="gmail_attr">On Sun, May 5, 2019 at 10:48
          PM Nahuel Garbezza via Cuis-dev <<a moz-do-not-send="true"
            href="mailto:cuis-dev@lists.cuis.st">cuis-dev@lists.cuis.st</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">Hi!<br>
            <div><br>
            </div>
            <div>I found that if there are files that do not start with
              a number under CoreUpdates, the Install New Updates
              feature breaks due to a parse error.</div>
            <div><br>
            </div>
            <div>Attached is a changeset to fix that (ignoring files
              that do not start with a number).</div>
            <div><br>
            </div>
            <div>Thanks,</div>
            <div>Nahuel.</div>
          </div>
          -- <br>
          Cuis-dev mailing list<br>
          <a moz-do-not-send="true" href="mailto:Cuis-dev@lists.cuis.st"
            target="_blank">Cuis-dev@lists.cuis.st</a><br>
          <a moz-do-not-send="true"
            href="https://lists.cuis.st/mailman/listinfo/cuis-dev"
            rel="noreferrer" target="_blank">https://lists.cuis.st/mailman/listinfo/cuis-dev</a><br>
        </blockquote>
      </div>
    </blockquote>
    <br>
    Ok, but Nahuel's code ignores those and doesn't install them.
    Current behavior is failure (that could mean "never put such files
    in Updates folder"). With Nahuel's code, and easy way to temporarily
    disable an update is to rename to add 'x' at the beginning of the
    name, for example (although this should never be pushed to GitHub).<br>
    <br>
    Looks reasonable to me. Opinions?<br>
    <br>
    Thanks,<br>
    <pre class="moz-signature" cols="72">-- 
Juan Vuletich
<a class="moz-txt-link-abbreviated" href="http://www.cuis-smalltalk.org">www.cuis-smalltalk.org</a>
<a class="moz-txt-link-freetext" href="https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev">https://github.com/Cuis-Smalltalk/Cuis-Smalltalk-Dev</a>
<a class="moz-txt-link-freetext" href="https://github.com/jvuletich">https://github.com/jvuletich</a>
<a class="moz-txt-link-freetext" href="https://www.linkedin.com/in/juan-vuletich-75611b3">https://www.linkedin.com/in/juan-vuletich-75611b3</a>
@JuanVuletich</pre>
  </body>
</html>