Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.11
    • Fix Version/s: 1.12
    • Labels:
      None
    • Environment:
      m11b3
    • Number of attachments :
      0

      Description

      The test test-classpath-has-overridden-jar in src/plugin-test/maintest/maven.xml fails. For some reason, the string MAVEN_REPO/maven/jars/maven.jar is replaced by $MAVEN_HOME/lib/maven.jar. If this is intentional, the test should be rewritten.

        Activity

        Hide
        Stéphane Nicoll added a comment -

        Ok found it, cactus was reseting the testSrcDir to a wrong value.

        Show
        Stéphane Nicoll added a comment - Ok found it, cactus was reseting the testSrcDir to a wrong value.
        Hide
        Stéphane Nicoll added a comment -

        This is fixed, execpt the cactus dependency thingy.

        Show
        Stéphane Nicoll added a comment - This is fixed, execpt the cactus dependency thingy.
        Hide
        Lukas Theussl added a comment -

        It's not fixed for me, plugin:test still fails for the same reason that I stated above. (I never noticed the cactus problem as I never used cactus and don't have it installed.) I can fix it easily but I don't know what is actually the expected entry in .classpath, is it

        <classpathentry kind="lib" path="MAVEN_REPO/maven/jars/maven.jar"></classpathentry>

        or is it

        <classpathentry kind="lib" path="$

        {MAVEN_HOME}

        /lib/maven.jar"></classpathentry>
        ?

        Show
        Lukas Theussl added a comment - It's not fixed for me, plugin:test still fails for the same reason that I stated above. (I never noticed the cactus problem as I never used cactus and don't have it installed.) I can fix it easily but I don't know what is actually the expected entry in .classpath, is it <classpathentry kind="lib" path="MAVEN_REPO/maven/jars/maven.jar"></classpathentry> or is it <classpathentry kind="lib" path="$ {MAVEN_HOME} /lib/maven.jar"></classpathentry> ?
        Hide
        Stéphane Nicoll added a comment -

        Well. I did not have to change anything for me, it worked in a first place.

        Expected entry is 'MAVEN_REPO/maven/jars/maven.jar'

        Show
        Stéphane Nicoll added a comment - Well. I did not have to change anything for me, it worked in a first place. Expected entry is 'MAVEN_REPO/maven/jars/maven.jar'
        Hide
        Lukas Theussl added a comment -

        Fixed. We now check for both classpathentries, at least one of them has to be present.

        Show
        Lukas Theussl added a comment - Fixed. We now check for both classpathentries, at least one of them has to be present.

          People

          • Assignee:
            Stéphane Nicoll
            Reporter:
            Lukas Theussl
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: