Continuum
  1. Continuum
  2. CONTINUUM-1640

No changes and no committer name extracted from SVN

    Details

    • Type: Bug Bug
    • Status: Reopened Reopened
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.1
    • Fix Version/s: 1.5.0
    • Component/s: SCM
    • Labels:
      None
    • Environment:
      Windows XP, Java 1.5, SVN 1.4.3
    • Complexity:
      Intermediate
    • Patch Submitted:
      Yes
    • Number of attachments :
      1

      Description

      When continuum detects changes and makes builds, no data is extracted on change date and committer.

      It always looks like this

      ****************************************************************************
      SCM Changes:
      ****************************************************************************
      Changed: no author @ no date
      Comment: no comment
      Files changed:
      (here are files)

      It is the same machine (SVN/continuum) so we rule out ime difference immediately.

      Moreover I've checked in continuum logs, there is "svn --non-interactive log" command executed, and when I type this command in, proper dates and commiters are displayed.

      looking at org.apache.maven.continuum.scm.DefaultContinuumScm, I've found that change date/commiters are taken from ScmResult that comes from this method
      scmResult = scmManager.getProviderByRepository( repository ).update( repository, fileSet, tag, getLatestUpdateDate( project ) );
      result = convertScmResult( scmResult );

      Maven SVN SCM Manager returns no changes in scmResult.getChanges()

      Where this svn log command is coming from then? Not from Maven SCM but from continuum itself?

        Activity

        Hide
        Emmanuel Venisse added a comment -

        Baptiste,

        To try to reproduce, rename or move a file in your svn, then let Continuum build your project.
        Let me know if you can reproduce it.

        Show
        Emmanuel Venisse added a comment - Baptiste, To try to reproduce, rename or move a file in your svn, then let Continuum build your project. Let me know if you can reproduce it.
        Hide
        Kalle Korhonen added a comment -

        In our case, the problem really was the differing time between Continuum server and svn server; two hours off on Continuum server was enough to throw it off every now and then. It'd be nicer if it'd fail somewhat more predictably but thanks to Emmanuel for the tip.

        Show
        Kalle Korhonen added a comment - In our case, the problem really was the differing time between Continuum server and svn server; two hours off on Continuum server was enough to throw it off every now and then. It'd be nicer if it'd fail somewhat more predictably but thanks to Emmanuel for the tip.
        Hide
        Brett Porter added a comment -

        pushing out, but we should be able to measure against a single time source

        Show
        Brett Porter added a comment - pushing out, but we should be able to measure against a single time source
        Hide
        Immanuel Scheerer added a comment -

        I stepped into the same problem because our LDAP users contain a space character. The current regular expressions from the project "maven-scm-provider-svnexec" used to parse the svn log do not match when the author contains a space. I will attach the fix that works for me.

        Show
        Immanuel Scheerer added a comment - I stepped into the same problem because our LDAP users contain a space character. The current regular expressions from the project "maven-scm-provider-svnexec" used to parse the svn log do not match when the author contains a space. I will attach the fix that works for me.
        Hide
        Immanuel Scheerer added a comment - - edited

        Attached the fix that works for me.

        Show
        Immanuel Scheerer added a comment - - edited Attached the fix that works for me.

          People

          • Assignee:
            Emmanuel Venisse
            Reporter:
            Timur Evdokimov
          • Votes:
            3 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated: