Maven Shade Plugin
  1. Maven Shade Plugin
  2. MSHADE-115

Temporary dependency-reduced-pom.xml in basedir causes problems within shared build trees

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.6
    • Fix Version/s: 1.7
    • Labels:
      None

      Description

      The runtime dependency-reduced-pom.xml (not the one in outputDirectory) causes problems within a build tree that is shared across multiple machines/builds. This file is cleaned up after it is no longer needed and if more than one build is running shade and the subsequent goals within the same subdirectory at around the same time, the dependency-reduced-pom.xml file can be wiped out by one process while another process still requires it.

        Issue Links

          Activity

          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Closed Closed
          27d 8h 56m 1 Benson Margulies 21/May/12 19:08
          Mark Thomas made changes -
          Workflow jira [ 12963685 ] Default workflow, editable Closed status [ 12999700 ]
          Mark Thomas made changes -
          Project Import Mon Apr 06 01:14:06 UTC 2015 [ 1428282846586 ]
          Mark Thomas made changes -
          Workflow jira [ 12725929 ] Default workflow, editable Closed status [ 12764083 ]
          Mark Thomas made changes -
          Project Import Sun Apr 05 12:26:34 UTC 2015 [ 1428236794734 ]
          Benson Margulies made changes -
          Status Open [ 1 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Fix Version/s 1.7 [ 18528 ]
          Hide
          Benson Margulies added a comment -

          The problem here is that, to avoid calculating a correct relative path from the desired destination of the drp to wherever the parent is, the code was written to write it out temporarily right in the basedir, where collisions are possible.

          The fix I've chosen is to calculate the relative path. An alternative would have been to use a temp file with a unique name in the basedir. In hindsight, that would have been less work.

          Show
          Benson Margulies added a comment - The problem here is that, to avoid calculating a correct relative path from the desired destination of the drp to wherever the parent is, the code was written to write it out temporarily right in the basedir, where collisions are possible. The fix I've chosen is to calculate the relative path. An alternative would have been to use a temp file with a unique name in the basedir. In hindsight, that would have been less work.
          Benson Margulies made changes -
          Summary Runtime dependency-reduced-pom.xml causes problems within shared build trees Temporary dependency-reduced-pom.xml in basedir causes problems within shared build trees
          Benson Margulies made changes -
          Assignee Benson Margulies [ bmargulies ]
          Hide
          Benson Margulies added a comment -

          And while here, note that the d-r-p doesn't get along so well if you try to have multiple shade executions, since the name conflicts.

          Show
          Benson Margulies added a comment - And while here, note that the d-r-p doesn't get along so well if you try to have multiple shade executions, since the name conflicts.
          Hide
          Seth added a comment -

          See Dan's comment about the fix commited to resolve MSHADE-23.

          Show
          Seth added a comment - See Dan's comment about the fix commited to resolve MSHADE-23 .
          Seth made changes -
          Field Original Value New Value
          Link This issue is related to MSHADE-23 [ MSHADE-23 ]
          Seth created issue -

            People

            • Assignee:
              Benson Margulies
              Reporter:
              Seth
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development