• Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • trac-plugin
    • None
    • Trac plugin 1.10, Hudson 1.365, Trac 0.11.2.1

      I have a problem with the links in the changes page.

      In my subversion repository, the files are located in:

      svn://svn.xxx.com/Repo/Product/Trunk/subdir/file.txt

      The links will then point at:

      http://trac.xxx.com/trac-project/browser/Product/Trunk/subdir/file.txt

      But my Trac installation has its root at Product/Trunk/, so the only link that would work is:

      http://trac.xxx.com/trac-project/browser/subdir/file.txt

      It seems there is no way to tell the plugin that my Trac project has its root set at svn://svn.xxx.com/Repo/Product/Trunk/ instead of svn://svn.xxx.com/Repo/ .

          [JENKINS-7032] Trac plugin creates wrong links

          dmoebius created issue -

          dmoebius added a comment -

          FYI, I created a patch to enhance the plugin with a new string parameter called "Trac source repository browser prefix" (couldn't come up with a better name). You may enter any path into this textfield. The plugin will subtract this prefix from the URL before opening the Trac browser.

          The online help for this new field needs more explanation.

          dmoebius added a comment - FYI, I created a patch to enhance the plugin with a new string parameter called "Trac source repository browser prefix" (couldn't come up with a better name). You may enter any path into this textfield. The plugin will subtract this prefix from the URL before opening the Trac browser. The online help for this new field needs more explanation.
          dmoebius made changes -
          Attachment New: JENKINS-7032.patch [ 19606 ]

          mattis added a comment -

          dmoebius: Thanks a lot for creating this patch, looking forward to seeing a fix.

          mattis added a comment - dmoebius: Thanks a lot for creating this patch, looking forward to seeing a fix.
          Gerd Zanker made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Gerd Zanker made changes -
          Assignee New: Gerd Zanker [ gerd_zanker ]

          Code changed in jenkins
          User: Gerd Zanker
          Path:
          pom.xml
          src/main/java/hudson/plugins/trac/TracRepositoryBrowser.java
          src/test/java/hudson/plugins/trac/TracSvnRepositoryBrowserTest.java
          src/test/resources/hudson/plugins/trac/changelog-with-deleted-file.xml
          src/test/resources/hudson/plugins/trac/changelog_unsorted.xml
          http://jenkins-ci.org/commit/trac-plugin/70dbb9210b77cf46c76dc2330b7d8d6dabd6293e
          Log:
          Added test for existing subversion TracRepositoryBrowser class to have a
          clean, tested base for JENKINS-7032 and JENKINS-13366

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Gerd Zanker Path: pom.xml src/main/java/hudson/plugins/trac/TracRepositoryBrowser.java src/test/java/hudson/plugins/trac/TracSvnRepositoryBrowserTest.java src/test/resources/hudson/plugins/trac/changelog-with-deleted-file.xml src/test/resources/hudson/plugins/trac/changelog_unsorted.xml http://jenkins-ci.org/commit/trac-plugin/70dbb9210b77cf46c76dc2330b7d8d6dabd6293e Log: Added test for existing subversion TracRepositoryBrowser class to have a clean, tested base for JENKINS-7032 and JENKINS-13366

          Gerd Zanker added a comment -

          Merged with pull request #5 into trac master branch.

          Gerd Zanker added a comment - Merged with pull request #5 into trac master branch.
          Gerd Zanker made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Progress [ 3 ] New: Resolved [ 5 ]
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 137129 ] New: JNJira + In-Review [ 187374 ]

            gerd_zanker Gerd Zanker
            dmoebius dmoebius
            Votes:
            1 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: