• Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • subversion-plugin
    • None
    • Platform: All, OS: All

      Url format was changed for Sventon 2.x, so integration is not working. There
      should be added two option for Repository browser (Sventon 1.x and Sventon 2.x).
      More generic solution will be to add Generic Browser and typing full url for
      browser, only revision should be parametrized.

      New url format is:
      http://SERVER/APP/repos/PROJECT/info?revision=REVISION

      while old url format was:
      http://SERVER/APP/revinfo.svn?name=PROJECT&revision=REVISION

          [JENKINS-3357] Sventon 2.0.x is not compatible with Hudson

          lotus118 added a comment -

          Created an attachment (id=638)
          Diff for that fixes sventon support for sventon versions 2.0.x.

          lotus118 added a comment - Created an attachment (id=638) Diff for that fixes sventon support for sventon versions 2.0.x.

          lotus118 added a comment -

          Damn! You just beat me to reporting this one!
          Anyway, I've attached a diff that, when applied to
          hudson/main/core/src/main/java/hudson/scm/browsers/Sventon.java (rev. 16573),
          fixes the urls. I tested it against Sventon 2.0.5.

          lotus118 added a comment - Damn! You just beat me to reporting this one! Anyway, I've attached a diff that, when applied to hudson/main/core/src/main/java/hudson/scm/browsers/Sventon.java (rev. 16573), fixes the urls. I tested it against Sventon 2.0.5.

          Hudson should be able to retrieve the page given to Hudson and determine Sventon
          version on its own, without asking the user.

          Am I correct in understanding that the current code works with 1.x and the patch
          is meant for 2.x?

          Kohsuke Kawaguchi added a comment - Hudson should be able to retrieve the page given to Hudson and determine Sventon version on its own, without asking the user. Am I correct in understanding that the current code works with 1.x and the patch is meant for 2.x?

          marksmys added a comment -

          Yes, current code work only with 1.x, and patch is only for 2.x

          marksmys added a comment - Yes, current code work only with 1.x, and patch is only for 2.x

          Code changed in hudson
          User: : stephenconnolly
          Path:
          trunk/hudson/main/core/src/main/java/hudson/scm/browsers/Sventon.java
          http://fisheye4.cenqua.com/changelog/hudson/?cs=17917
          Log:
          [FIXED JENKINS-3357]

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : stephenconnolly Path: trunk/hudson/main/core/src/main/java/hudson/scm/browsers/Sventon.java http://fisheye4.cenqua.com/changelog/hudson/?cs=17917 Log: [FIXED JENKINS-3357]

            Unassigned Unassigned
            marksmys marksmys
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

              Created:
              Updated:
              Resolved: