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

Wrong URL in diff

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • viewVC-plugin
    • None
    • Centos 6.4 x64 (Linux kera 2.6.32-358.6.2.el6.x86_64 #1 SMP Thu May 16 20:59:36 UTC 2013 x86_64 x86_64 x86_64 GNU/Linux)
      Jenkins 1.517 (jenkins-1.517-1.1.noarch)
      ViewVC 1.6

      Hello,
      At Changes tab, for job, ViewVC links are correct, but for specified revision diff URLs lost part of the link.

      Project configuration (Source Code Management):
      URL - http://example.org/viewvc/branches/name/
      Repository - repo

      Example URL's:
      ViewVC URL - http://example.org/viewvc/branches/name/?view=rev&root=repo&revision=10
      Diff URL - http://example.org/branches/name/java/src/main/File.java?root=repo&r1=9&r2=10&diff_format=h

      Diff URL lost /viewvc/ part.

          [JENKINS-18283] Wrong URL in diff

          Duc Quoc added a comment -

          I've opened a pull request for this issue
          https://github.com/jenkinsci/viewVC-plugin/pull/4

          Duc Quoc added a comment - I've opened a pull request for this issue https://github.com/jenkinsci/viewVC-plugin/pull/4

          Adam Gabryś added a comment -

          Can you prepare a new release and close this bug?

          Adam Gabryś added a comment - Can you prepare a new release and close this bug?

          Alexander Ost added a comment -

          The problem also exists on the LTS branch (1.509.4).

          Since the pull request is already available, is there any chance to get this released soon?

          Alexander Ost added a comment - The problem also exists on the LTS branch (1.509.4). Since the pull request is already available, is there any chance to get this released soon?

          Idan Bidani added a comment - - edited

          We have the same issue
          I've had to compile it and manually upload the plugin.
          any chance to approve this pull request
          https://github.com/jenkinsci/viewVC-plugin/pull/1

          Idan Bidani added a comment - - edited We have the same issue I've had to compile it and manually upload the plugin. any chance to approve this pull request https://github.com/jenkinsci/viewVC-plugin/pull/1

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ducquoc
          Path:
          src/main/java/hudson/plugins/viewVC/ViewVCRepositoryBrowser.java
          http://jenkins-ci.org/commit/viewVC-plugin/93e996630c54211e8fe0ddfa769bc0e929e76060
          Log:
          fixed JENKINS-18283 : Wrong URL in diff

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ducquoc Path: src/main/java/hudson/plugins/viewVC/ViewVCRepositoryBrowser.java http://jenkins-ci.org/commit/viewVC-plugin/93e996630c54211e8fe0ddfa769bc0e929e76060 Log: fixed JENKINS-18283 : Wrong URL in diff

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ducquoc
          Path:
          src/main/webapp/help-url.html
          http://jenkins-ci.org/commit/viewVC-plugin/659a0cf21103845a72094259dbc1a5fb121a5bd6
          Log:
          JENKINS-18283 updated sample URL in help-url to reflect non-root
          installation of viewVC

          Compare: https://github.com/jenkinsci/viewVC-plugin/compare/6f5be7804741...659a0cf21103

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ducquoc Path: src/main/webapp/help-url.html http://jenkins-ci.org/commit/viewVC-plugin/659a0cf21103845a72094259dbc1a5fb121a5bd6 Log: JENKINS-18283 updated sample URL in help-url to reflect non-root installation of viewVC Compare: https://github.com/jenkinsci/viewVC-plugin/compare/6f5be7804741...659a0cf21103

          Michael Rumpf added a comment -

          Fixes have been merged

          Michael Rumpf added a comment - Fixes have been merged

            mirumpf Michael Rumpf
            agabrys Adam Gabryś
            Votes:
            4 Vote for this issue
            Watchers:
            7 Start watching this issue

              Created:
              Updated:
              Resolved: