<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 2011/11/27 1:30, Michael Anstis wrote:
    <blockquote
cite="mid:CAAG9P0v+qASCJeOoyH-n4r1FNM=y8+YAnhVqZ7RaUyKVknB-4A@mail.gmail.com"
      type="cite">Top banana Jervis :)<br>
      <br>
      JVMDateConverter is only used by Drools Guvnor (for the decision
      tables) so might not (or should not be duplicated).<br>
      <br>
      I also don't understand why GuvnorTestBase and
      MockAssetItemIterator cannot simply live in guvnor-core as
      guvnor-drools references it.<br>
    </blockquote>
    GuvnorTestBase and MockAssetItemIterator are located under the test
    director. I dont know if there is anyway to reference them from
    guvnor-webapp-drools module.<br>
    <blockquote
cite="mid:CAAG9P0v+qASCJeOoyH-n4r1FNM=y8+YAnhVqZ7RaUyKVknB-4A@mail.gmail.com"
      type="cite">
      <br>
      I suspect we should split up the resources folders between
      guvnor-core and guvnor-drools as you say constants.properties.<br>
      <br>
      Anyway, I imported the new modules today and am able to run Guvnor
      Drools (preliminary tests) without problem.<br>
      <br>
      With kind regards,<br>
      <br>
      Mike<br>
      <br>
      <div class="gmail_quote">On 26 November 2011 06:40, Jervis Liu <span
          dir="ltr">&lt;<a moz-do-not-send="true"
            href="mailto:jliu@redhat.com">jliu@redhat.com</a>&gt;</span>
        wrote:<br>
        <blockquote class="gmail_quote" style="margin:0 0 0
          .8ex;border-left:1px #ccc solid;padding-left:1ex;">
          Done.<br>
          <br>
          To-do-list:<br>
          1. I have to duplicate GuvnorTestBase.java,
          MockAssetItemIterator.java<br>
          and JVMDateConverter in guvnor-webapp-core and
          guvnor-webapp-drools<br>
          modules. One solution is to create another module called<br>
          guvnor-webapp-testutils. Let me know if you have better ideas.<br>
          2. guvnor-webapp-core and guvnor-webapp-drools both have a
          same copy of<br>
          constants.properties. Need to remove unused constants from
          this file<br>
          (and different language versions) for each module.
          Straightforward to<br>
          do, but definitely takes time.<br>
          3. Need to refactor and split sever side code.<br>
          <br>
          Cheers,<br>
          Jervis<br>
          <div class="HOEnZb">
            <div class="h5">On 2011/11/24 10:30, Jervis Liu wrote:<br>
              &gt; Michael Anstis wrote:<br>
              &gt;&gt; Just a thought.<br>
              &gt;&gt;<br>
              &gt;&gt; When is the BRMS 5.3 freeze?<br>
              &gt;&gt;<br>
              &gt;&gt; Back-porting Guvnor fixes post this split is
              going to be a PITA.<br>
              &gt;&gt;<br>
              &gt; What I heard is after the Christmas.<br>
              &gt;&gt; On 23 November 2011 07:43, Jervis Liu &lt;<a
                moz-do-not-send="true" href="mailto:jliu@redhat.com">jliu@redhat.com</a><br>
              &gt;&gt; &lt;mailto:<a moz-do-not-send="true"
                href="mailto:jliu@redhat.com">jliu@redhat.com</a>&gt;&gt;
              wrote:<br>
              &gt;&gt;<br>
              &gt;&gt; &nbsp; &nbsp; Hi,<br>
              &gt;&gt;<br>
              &gt;&gt; &nbsp; &nbsp; As part of the ongoing work to refactor
              Guvnor towards a generic<br>
              &gt;&gt; &nbsp; &nbsp; service<br>
              &gt;&gt; &nbsp; &nbsp; repository framework (Guvnor NG. Please check
              this thread for<br>
              &gt;&gt; relevant<br>
              &gt;&gt; &nbsp; &nbsp; discussions: <a moz-do-not-send="true"
                href="http://community.jboss.org/thread/173484"
                target="_blank">http://community.jboss.org/thread/173484</a>),
              I am<br>
              &gt;&gt; going to<br>
              &gt;&gt; &nbsp; &nbsp; split guvnor-webapp module to three modules:
              guvnor-webapp-core,<br>
              &gt;&gt; &nbsp; &nbsp; guvnor-webapp-drools, guvnor-webapp-soa. The
              split has been done<br>
              &gt;&gt; on my<br>
              &gt;&gt; &nbsp; &nbsp; local branch and the plan is to merge the
              change to master this<br>
              &gt;&gt; &nbsp; &nbsp; Friday.<br>
              &gt;&gt; &nbsp; &nbsp; To avoid huge merge conflicts after my split,
              I strongly<br>
              &gt;&gt; encourage you<br>
              &gt;&gt; &nbsp; &nbsp; to commit whatever changes you have to
              &nbsp;master before this<br>
              &gt;&gt; Friday. Let<br>
              &gt;&gt; &nbsp; &nbsp; me know if you have any questions. I will
              start the merge this<br>
              &gt;&gt; Friday<br>
              &gt;&gt; &nbsp; &nbsp; morning (10AM, GMT +8) if I dont hear any
              objections. Thanks.<br>
              &gt;&gt;<br>
              &gt;&gt; &nbsp; &nbsp; Cheers,<br>
              &gt;&gt; &nbsp; &nbsp; Jervis<br>
              &gt;&gt; &nbsp; &nbsp;
              _______________________________________________<br>
              &gt;&gt; &nbsp; &nbsp; rules-dev mailing list<br>
              &gt;&gt; &nbsp; &nbsp; <a moz-do-not-send="true"
                href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a>
              &lt;mailto:<a moz-do-not-send="true"
                href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a>&gt;<br>
              &gt;&gt; &nbsp; &nbsp; <a moz-do-not-send="true"
                href="https://lists.jboss.org/mailman/listinfo/rules-dev"
                target="_blank">https://lists.jboss.org/mailman/listinfo/rules-dev</a><br>
              &gt;&gt;<br>
              &gt;&gt;<br>
              &gt;&gt;
              ------------------------------------------------------------------------<br>
              &gt;&gt;<br>
              &gt;&gt; _______________________________________________<br>
              &gt;&gt; rules-dev mailing list<br>
              &gt;&gt; <a moz-do-not-send="true"
                href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a><br>
              &gt;&gt; <a moz-do-not-send="true"
                href="https://lists.jboss.org/mailman/listinfo/rules-dev"
                target="_blank">https://lists.jboss.org/mailman/listinfo/rules-dev</a><br>
              &gt;<br>
              &gt;<br>
              <br>
              _______________________________________________<br>
              rules-dev mailing list<br>
              <a moz-do-not-send="true"
                href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a><br>
              <a moz-do-not-send="true"
                href="https://lists.jboss.org/mailman/listinfo/rules-dev"
                target="_blank">https://lists.jboss.org/mailman/listinfo/rules-dev</a><br>
            </div>
          </div>
        </blockquote>
      </div>
      <br>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
rules-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:rules-dev@lists.jboss.org">rules-dev@lists.jboss.org</a>
<a class="moz-txt-link-freetext" href="https://lists.jboss.org/mailman/listinfo/rules-dev">https://lists.jboss.org/mailman/listinfo/rules-dev</a>
</pre>
    </blockquote>
    <br>
  </body>
</html>