Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 16.11.01
    • Component/s: tools
    • Labels:
      None

      Description

      Some scripts, at least, are meaningless after the Gradel move. They need to be amended or removed

        Issue Links

          Activity

          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          I fixed missing "./" in rc scripts at revision: 1778576

          Show
          jacques.le.roux Jacques Le Roux added a comment - I fixed missing "./" in rc scripts at revision: 1778576
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          As I said in my last commit

          Completes r1752771, I did not change the not concerned scripts. Though I worked carefully, I could have missed something so please check (notably typos, I did not test all). Note: we had no "apply-ofbiz-patches" Ant target (in applyOfbizPatchesAndHotDeploy.sh)

          So please check, thanks!

          Show
          jacques.le.roux Jacques Le Roux added a comment - As I said in my last commit Completes r1752771, I did not change the not concerned scripts. Though I worked carefully, I could have missed something so please check (notably typos, I did not test all). Note: we had no "apply-ofbiz-patches" Ant target (in applyOfbizPatchesAndHotDeploy.sh) So please check, thanks!
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Fixed and completed: At revision: 1752806

          Show
          jacques.le.roux Jacques Le Roux added a comment - Fixed and completed: At revision: 1752806
          Hide
          taher Taher Alkhateeb added a comment -

          I think the issue with POS happened after the first commit which introduced remote libraries. So it is one of those "jar dependency hell" problems. It is a good idea either way to remove it from the start component and fix it outside with its own bootstrapping code away from the framework.

          Show
          taher Taher Alkhateeb added a comment - I think the issue with POS happened after the first commit which introduced remote libraries. So it is one of those "jar dependency hell" problems. It is a good idea either way to remove it from the start component and fix it outside with its own bootstrapping code away from the framework.
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          I just had a look at the POS, it does not start with Gradle (I think Taher Alkhateeb said it already), we get

          2016-07-15 11:05:35,658 |main                 |ServiceContainer              |I| Created new dispatcher: POSDispatcher
          2016-07-15 11:05:35,706 |main                 |XuiSession                    |I| Created XuiSession [pos-1]
          ERROR: Unable to create the page: home
          2016-07-15 11:05:35,968 |main                 |ContainerLoader               |I| Started container pos-container
          

          No I wonder if I'll try to debug that or we will simply remove the POS, because XUI is totally deprecated (even if it still works perfectly), let's see...

          Show
          jacques.le.roux Jacques Le Roux added a comment - I just had a look at the POS, it does not start with Gradle (I think Taher Alkhateeb said it already), we get 2016-07-15 11:05:35,658 |main |ServiceContainer |I| Created new dispatcher: POSDispatcher 2016-07-15 11:05:35,706 |main |XuiSession |I| Created XuiSession [pos-1] ERROR: Unable to create the page: home 2016-07-15 11:05:35,968 |main |ContainerLoader |I| Started container pos-container No I wonder if I'll try to debug that or we will simply remove the POS, because XUI is totally deprecated (even if it still works perfectly), let's see...
          Hide
          jacques.le.roux Jacques Le Roux added a comment -

          Partly done at revision: 1752771 , WIP...

          Show
          jacques.le.roux Jacques Le Roux added a comment - Partly done at revision: 1752771 , WIP...

            People

            • Assignee:
              jacques.le.roux Jacques Le Roux
              Reporter:
              jacques.le.roux Jacques Le Roux
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development