Maven SCM

'Username isn't defined.' when generating SCM report if CVS port number is specified in SCM URL

Details

  • Complexity:
    Intermediate
  • Number of attachments :
    0

Description

I have an CVS connection URL defined as follows:
scm:cvs:pserver:continuum@gnllx002.ebit.be:2401:/data01/cvsroot_bkh:VCG_BKH/uBaseBkh

When I try mvn site, the stage 'Generating "Source Repository" report.' fails as shown below.
If I remove the :2401, it works OK.

[INFO] Generating "Source Repository" report.
[INFO] ------------------------------------------------------------------------
[ERROR] FATAL ERROR
[INFO] ------------------------------------------------------------------------
[INFO] Username isn't defined.
[INFO] ------------------------------------------------------------------------
[INFO] Trace
java.lang.IllegalArgumentException: Username isn't defined.
at org.apache.maven.scm.provider.cvslib.repository.CvsScmProviderRepository.getCvsRootForCvsPass(CvsScmProviderRepository.java:105)
at org.apache.maven.scm.provider.cvslib.repository.CvsScmProviderRepository.getCvsRoot(CvsScmProviderRepository.java:73)
at org.apache.maven.report.projectinfo.ScmReport$ScmRenderer.developerAccessCVS(ScmReport.java:479)
at org.apache.maven.report.projectinfo.ScmReport$ScmRenderer.renderDeveloperAccessSection(ScmReport.java:323)
at org.apache.maven.report.projectinfo.ScmReport$ScmRenderer.renderBody(ScmReport.java:186)
at org.apache.maven.reporting.AbstractMavenReportRenderer.render(AbstractMavenReportRenderer.java:65)
at org.apache.maven.report.projectinfo.ScmReport.executeReport(ScmReport.java:87)
at org.apache.maven.reporting.AbstractMavenReport.generate(AbstractMavenReport.java:101)
at org.apache.maven.plugins.site.ReportDocumentRenderer.renderDocument(ReportDocumentRenderer.java:139)
at org.apache.maven.doxia.siterenderer.DefaultSiteRenderer.renderModule(DefaultSiteRenderer.java:269)

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. SCM-350 CVS URL without username or password exception

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Geert Pante added a comment -

Same exception, different cause, different scenario.

Show
Geert Pante added a comment - Same exception, different cause, different scenario.
Hide
Permalink
Dennis Lundberg added a comment -

If you are using an empty password, please see http://maven.apache.org/scm/cvs.html in the examples section:

 
If you use anonymous access, with an empty password,
you need to add a colon (:) between the username and
"@" like it's done in the first example.
Show
Dennis Lundberg added a comment - If you are using an empty password, please see http://maven.apache.org/scm/cvs.html in the examples section: If you use anonymous access, with an empty password, you need to add a colon (:) between the username and "@" like it's done in the first example.
Hide
Permalink
Geert Pante added a comment -

The password is not empty, I'm just using native CVS, with the password in ~/.cvspass.
I'm using the third form from http://maven.apache.org/scm/cvs.html
scm:cvs<delimiter>pserver<delimiter>[username[<delimiter>password]@]servername[<delimiter>port]<delimiter>path_to_repository<delimiter>module_name
With the [<delimiter>password] left out, and [<delimiter>port] filled in.

Maybe it's better to use an empty password for the continuum user, but we're not an open source company (yet)

Show
Geert Pante added a comment - The password is not empty, I'm just using native CVS, with the password in ~/.cvspass. I'm using the third form from http://maven.apache.org/scm/cvs.html scm:cvs<delimiter>pserver<delimiter>[username [<delimiter>password] @]servername [<delimiter>port] <delimiter>path_to_repository<delimiter>module_name With the [<delimiter>password] left out, and [<delimiter>port] filled in. Maybe it's better to use an empty password for the continuum user, but we're not an open source company (yet)
Hide
Permalink
Dennis Lundberg added a comment -

Can you please try this URL instead:
scm:cvs|pserver|continuum@gnllx002.ebit.be:2401|/data01/cvsroot_bkh|VCG_BKH/uBaseBkh

Show
Dennis Lundberg added a comment - Can you please try this URL instead: scm:cvs|pserver|continuum@gnllx002.ebit.be:2401|/data01/cvsroot_bkh|VCG_BKH/uBaseBkh
Hide
Permalink
Geert Pante added a comment -

with pipes, it works: scm:cvs|pserver|continuum@gnllx002.gent.vcc.ford.com:2401|/data01/cvsroot_xps|VCG_XPS/uBaseXps,

So the bug is probably in the URL parsing part, url.parse(separator) should get a little smarter

Show
Geert Pante added a comment - with pipes, it works: scm:cvs|pserver|continuum@gnllx002.gent.vcc.ford.com:2401|/data01/cvsroot_xps|VCG_XPS/uBaseXps, So the bug is probably in the URL parsing part, url.parse(separator) should get a little smarter
Hide
Permalink
Vincent Siveton added a comment -

I added a test case in r691838
Could you review it?

Show
Vincent Siveton added a comment - I added a test case in r691838 Could you review it?

People

  • Assignee:
    Unassigned
    Reporter:
    Geert Pante
Vote (2)
Watch (2)

Dates

  • Created:
    Updated: