Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.1.0
    • None

    Description

      The FileBasedProvider was a bundle we used in the early days of ACE to get started quickly with a very simple file based repository. The code for that has since been replaced by the RepositoryBasedProvider. I propose we remove support for the FileBasedProvider completely.

      Attachments

        1. ACE-423.patch
          48 kB
          Bram Pouwelse

        Activity

          This is a bit more tricky than just throwing it away, as there is an integration test that currently depends on it. That will have to be refactored first.

          marrs Marcel Offermans added a comment - This is a bit more tricky than just throwing it away, as there is an integration test that currently depends on it. That will have to be refactored first.
          jajans J.W. Janssen added a comment -

          Maybe we could move this code to the test packages of ACE?

          jajans J.W. Janssen added a comment - Maybe we could move this code to the test packages of ACE?

          I think that is a good suggestion.

          marrs Marcel Offermans added a comment - I think that is a good suggestion.
          brampouwelse Bram Pouwelse added a comment -

          Attached patch to move the FileBasedProvider to the org.apache.ace.deployment.itest

          Also removed org.apache.ace.repository.impl bundle from the build path for some itest projects

          brampouwelse Bram Pouwelse added a comment - Attached patch to move the FileBasedProvider to the org.apache.ace.deployment.itest Also removed org.apache.ace.repository.impl bundle from the build path for some itest projects
          jajans J.W. Janssen added a comment -

          Code is moved to the itest bundle. See rev #1727373.

          jajans J.W. Janssen added a comment - Code is moved to the itest bundle. See rev #1727373.

          People

            Unassigned Unassigned
            marrs Marcel Offermans
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: