Ivy
  1. Ivy
  2. IVY-251

artifact origin is not included in the default report

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: unspecified
    • Fix Version/s: 1.4-RC1
    • Component/s: Core
    • Labels:
      None

      Description

      The origin of the artifact is not included in the default report. You'll have the create a special 'artifact-report' to retrieve this information.
      I'll upload a patch and junit test

        Activity

        Hide
        Xavier Hanin added a comment -

        I agree, using standard way to store file origin is better. I f you can submit a patch to include the modification in the current code it would be much appreciated, I only have very few time to spend on ivy for the moment...

        Show
        Xavier Hanin added a comment - I agree, using standard way to store file origin is better. I f you can submit a patch to include the modification in the current code it would be much appreciated, I only have very few time to spend on ivy for the moment...
        Hide
        Maarten Coene added a comment -

        See also forum topic: http://www.jayasoft.org/node/646

        Show
        Maarten Coene added a comment - See also forum topic: http://www.jayasoft.org/node/646
        Hide
        Maarten Coene added a comment -

        patch which adds the origin of an artifact to the default report

        Show
        Maarten Coene added a comment - patch which adds the origin of an artifact to the default report
        Hide
        Maarten Coene added a comment -

        new file

        Show
        Maarten Coene added a comment - new file
        Hide
        Maarten Coene added a comment -

        This patch stores the information about the original location of the artifact in the (already existing) cached data file for the module.
        I didn't follow the current strategy to store this information in a .origin file for each artifact (I think that Ivy should create as less files as possible).

        If you don't think this was a good idea, all you have to modify is the Ivy.saveArtifactOrigin and Ivy.getSavedArtifactOrigin methods and rewrite them to use the .origin files.

        In addition, if this patch gets approved, the current code that reads/writes directly from/to the .origin files should be updated to use these new methods on the Ivy class. If you want, I can create a patch for this.

        regards,
        Maarten

        Show
        Maarten Coene added a comment - This patch stores the information about the original location of the artifact in the (already existing) cached data file for the module. I didn't follow the current strategy to store this information in a .origin file for each artifact (I think that Ivy should create as less files as possible). If you don't think this was a good idea, all you have to modify is the Ivy.saveArtifactOrigin and Ivy.getSavedArtifactOrigin methods and rewrite them to use the .origin files. In addition, if this patch gets approved, the current code that reads/writes directly from/to the .origin files should be updated to use these new methods on the Ivy class. If you want, I can create a patch for this. regards, Maarten
        Hide
        Maarten Coene added a comment -

        Fixed

        Show
        Maarten Coene added a comment - Fixed

          People

          • Assignee:
            Maarten Coene
            Reporter:
            Maarten Coene
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Due:
              Created:
              Updated:
              Resolved:

              Development