Hello,

whenever I select "OpenGrok" as CVS Repository Browser, it doesn't seem to be used. There are no diff- and file-links to OpenGrok in the Changes-Page as expected.

The browser is configured in a job's scm section, the config.xml reflects this perfectly.

<repositories>
<hudson.scm.CvsRepository>
[...]
<repositoryBrowser class="hudson.scm.browsers.OpenGrok">
<url>http:/192.168.100.12:59990/source/xref/project/</url>
</repositoryBrowser>
[...]

The RepositoryBrowser is used in "cvs-plugin/src/main/resources/hudson/scm/CVSChangeLogSet/index.jelly"

The RepositoryBrowser is usually referenced in "CVSSCM.java", where "getEffectiveBrowser()" (inherited from "SCM.java") should return the result of "getBrowser()", which should return the current CvsFacadeRepositoryBrowser, which is created in "readResolve()", if the variable repositoryBrowser is not null.

But: The variable "repositoryBrowser" in "CVSSCM.java" does not seem to be set ever, except for some deprecated constructors. I can not see, which constructor was used, but there are constructors, that don't set the Member field "repositoryBrowser", but the member field "repositories". In this case, the Method "readResolve()" would return whithout initializing the variable "facadeRepositoryBrowser". If the "facadeRepositoryBrowser" is not initialized, "getBrowser()" returns an CvsFacadeRepositoryBrowser with Member field "legacyBrowser" set to null.

This seems to activate a mode, where the RepositoryBrowser is looked up via the "CvsFacadeRepositoryBrowser".In "calculateRepositoryBrowser(CVSChangeLogSet.CVSChangeLog changelog)" it is referenced as "changelog.getRepository().getRepositoryBrowser()".

The CVSChangeLog is created with information from the omnipresent "final CvsRepository[] repositories". I have tried to analyze all the places, where "CvsRepository" is instantiated, as i assume, the RepositoryBrowser-information might get lost somewhere in between.

the changes-page is described in: "jenkins/core/src/main/resources/hudson/scm/SCM/project-changes.jelly". In this file, "browser.getChangeSetLink" is called, which returns "null" if OpenGrok is selected. there is a comment:

<!-- This anchor is meaningless if there is >1 ChangeLogSet/index.jelly for the build; an SCM should rather use an actual commit ID for the anchor: -->

I do not really understand the part "This anchor is meaningless if there is >1 ChangeLogSet/index.jelly for the build". The only file, where OpenGrok would be asked for a valid link would be "cvs-plugin/src/main/resources/hudson/scm/CVSChangeLogSet/index.jelly".

please note, that "CVSChangeLogSet" is not "ChangeLogSet" as mentioned above. CVSChangeLogSet\index.jelly resolves the browser like this:

<j:set var="browser" value="${it.build.parent.scm.effectiveBrowser}"/>