Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-7026

Console view Ant target highlight cannot cope with colon in target name.

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • ant-plugin
    • None

      Only the part up to and including the first colon is highlighted.

          [JENKINS-7026] Console view Ant target highlight cannot cope with colon in target name.

          voorth added a comment -

          Looks like allowing colons would simplify the regex in AntTargetNote.java to

          .*(?=:)
          

          unless you want to rule out a colon as start character, of course

          voorth added a comment - Looks like allowing colons would simplify the regex in AntTargetNote.java to .*(?=:) unless you want to rule out a colon as start character, of course

          voorth added a comment -

          console output looks like this:

          test:prepare:

          test:generate:

          test:compile:
          [echo] src.test.dir=/data01/buildserver/swarm-client/workspace/TRUNK_MGS/mgs/modules/core-ext/src/test
          [echo] build.test.classes.dir=/data01/buildserver/swarm-client/workspace/TRUNK_MGS/target/build/mgs-ext-test/WEB-INF/classes

          and the corresponding links in the ant target sidebar look like:

          test
          test
          test

          voorth added a comment - console output looks like this: test :prepare: test :generate: test :compile: [echo] src.test.dir=/data01/buildserver/swarm-client/workspace/TRUNK_MGS/mgs/modules/core-ext/src/test [echo] build.test.classes.dir=/data01/buildserver/swarm-client/workspace/TRUNK_MGS/target/build/mgs-ext-test/WEB-INF/classes and the corresponding links in the ant target sidebar look like: test test test

          voorth added a comment -

          A small patch that just replaces the used regex.
          Needs to be tested, since I have not been able to get Hudson building on my machin.

          I did test the regex on different target names, though.

          voorth added a comment - A small patch that just replaces the used regex. Needs to be tested, since I have not been able to get Hudson building on my machin. I did test the regex on different target names, though.

          Code changed in hudson
          User: : kohsuke
          Path:
          trunk/hudson/main/core/src/main/java/hudson/tasks/_ant/AntTargetNote.java
          trunk/www/changelog.html
          http://jenkins-ci.org/commit/33897
          Log:
          [FIXED JENKINS-7026] applied the patch to fix the regular expression.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : kohsuke Path: trunk/hudson/main/core/src/main/java/hudson/tasks/_ant/AntTargetNote.java trunk/www/changelog.html http://jenkins-ci.org/commit/33897 Log: [FIXED JENKINS-7026] applied the patch to fix the regular expression.

          dogfood added a comment -

          Integrated in hudson_main_trunk #186
          [FIXED JENKINS-7026] applied the patch to fix the regular expression.

          kohsuke :
          Files :

          • /trunk/www/changelog.html
          • /trunk/hudson/main/core/src/main/java/hudson/tasks/_ant/AntTargetNote.java

          dogfood added a comment - Integrated in hudson_main_trunk #186 [FIXED JENKINS-7026] applied the patch to fix the regular expression. kohsuke : Files : /trunk/www/changelog.html /trunk/hudson/main/core/src/main/java/hudson/tasks/_ant/AntTargetNote.java

            Unassigned Unassigned
            voorth voorth
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: