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

Plugin does not start build if there is only one change set

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Major Major
    • tfs-plugin
    • None
    • Hudson ver. 1.363, both released and current tfs plugin
      tfs 2008 server
      MS TEE Command line client 10.0.0

      After checking in changes, TFS log in Hudson shows them consistently (see below). However, no automatic builds are being triggered.

      The problem is the newly released TEE Command line client – it does not output the

      "-------------------------------"

      seperation lines that the parsing algorithm depends upon. We have reverted to using the Microsoft Team System Explorer command line utility and everything seems to be working fine now.

      However, wouldn't it still be possible to change the parser to starting a block with "Changeset:" instead of ending it with "-----------------"?

      If necessary, I could provide a diff...

       
Started on 28.06.2010 18:18:38
[workspace] $ C:\javabuildtools\TEE-CLC-10.0.0\tf.cmd history $/Services/Server/DEV/SP/MavenDefs -noprompt -version:D2010-06-28T12:52:47Z~D2010-06-28T16:18:38Z -recursive -format:detailed -server:http://s-tfs:8080/ ********

Changeset: 364235
User: GRP\ObendorfH
Date: 28.06.2010 17:25:40

Comment:
  Versionierung für EJB EAR

Items:
  edit $/Services/Server/DEV/SP/MavenDefs/pom.xml

Check-in Notes:
  Code Reviewer: 

  Performance Reviewer: 
  Security Reviewer: 

Done. Took 8,6 Sekunden
No changes

        1. mylyn-context.zip
          3 kB
        2. screenshot.5.png
          screenshot.5.png
          9 kB

          [JENKINS-6870] Plugin does not start build if there is only one change set

          hartmut added a comment -

          Setting the polling time to 1, 5, and 10 minutes does not change the plugin's behavior (see attached image).

          hartmut added a comment - Setting the polling time to 1, 5, and 10 minutes does not change the plugin's behavior (see attached image).

          redsolo added a comment -

          If there are several changests, are they separated by anything? Apparently the order of the keywords is not specific and could change (http://social.msdn.microsoft.com/Forums/en-US/tfsversioncontrol/thread/c41d1d0c-dfe1-4a00-b128-c5e3f503170e#3ac1216e-c121-4c9d-ab53-7c1c4fb823e6).

          But please attach an output containing two change sets.

          redsolo added a comment - If there are several changests, are they separated by anything? Apparently the order of the keywords is not specific and could change ( http://social.msdn.microsoft.com/Forums/en-US/tfsversioncontrol/thread/c41d1d0c-dfe1-4a00-b128-c5e3f503170e#3ac1216e-c121-4c9d-ab53-7c1c4fb823e6 ). But please attach an output containing two change sets.

          hartmut added a comment - - edited

          Differences are smaller than we thought – IF there are several changesets in the output, they are separated by a line of "-". IF the output contains only a single changeset, there is NO separation line. That caused the error.

          MS TF.EXE output - WITH leading ---- - line

          $ /cygdrive/c/Programme/Microsoft\ Visual\ Studio\ 10.0/Common7/IDE/TF.exe history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs
-----------------------------------------------------------------------------------------------------------------------
Changeset: 24
User: ObendorfH
Date: Donnerstag, 6. Mai 2010 13:29:24

Comment:
  Initialer Check-In

Items:
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site

Check-in Notes:
  Code Reviewer:
  Performance Reviewer:
  Security Reviewer:

-----------------------------------------------------------------------------------------------------------------------
Changeset: 23
User: ObendorfH
Date: Donnerstag, 6. Mai 2010 13:28:21

Comment:

Items:
  branch $/TFSJavaBuild/Sources/Maven/Current

Check-in Notes:
  Code Reviewer:
  Performance Reviewer:
  Security Reviewer:

          MS TEE TF.cmd - WITHOUT leading ---- - line 

          $ /cygdrive/c/Programme/TEE-CLC-10.0.0/tf.cmd history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs
Changeset: 24
User: GRP\ObendorfH
Date: 06.05.2010 13:29:24

Comment:
  Initialer Check-In

Items:
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs
  add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site

Check-in Notes:
  Code Reviewer:
  Performance Reviewer:
  Security Reviewer:

-----------------------------------------------------------------------------------------------------------------------
Changeset: 23
User: GRP\ObendorfH
Date: 06.05.2010 13:28:21

Comment:


Items:
  branch $/TFSJavaBuild/Sources/Maven/Current

Check-in Notes:
  Code Reviewer:
  Performance Reviewer:
  Security Reviewer:

          hartmut added a comment - - edited Differences are smaller than we thought – IF there are several changesets in the output, they are separated by a line of "-". IF the output contains only a single changeset, there is NO separation line. That caused the error. MS TF.EXE output - WITH leading ---- - line $ /cygdrive/c/Programme/Microsoft\ Visual\ Studio\ 10.0/Common7/IDE/TF.exe history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs ----------------------------------------------------------------------------------------------------------------------- Changeset: 24 User: ObendorfH Date: Donnerstag, 6. Mai 2010 13:29:24 Comment: Initialer Check-In Items: add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: ----------------------------------------------------------------------------------------------------------------------- Changeset: 23 User: ObendorfH Date: Donnerstag, 6. Mai 2010 13:28:21 Comment: Items: branch $/TFSJavaBuild/Sources/Maven/Current Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: MS TEE TF.cmd - WITHOUT leading ---- - line $ /cygdrive/c/Programme/TEE-CLC-10.0.0/tf.cmd history $/TFSJavaBuild -noprompt -version:D2010-05-06T11:00:26Z~D2010-05-06T11:30:17Z -recursive -format:detailed -server:http://v-tfsbuildtest:8080/tfs Changeset: 24 User: GRP\ObendorfH Date: 06.05.2010 13:29:24 Comment: Initialer Check-In Items: add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.project add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.tpignore add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/pom.xml add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/.settings/org.maven.ide.eclipse.prefs add $/TFSJavaBuild/Sources/Maven/DEV/com.haufelexware.project.base/src/site Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer: ----------------------------------------------------------------------------------------------------------------------- Changeset: 23 User: GRP\ObendorfH Date: 06.05.2010 13:28:21 Comment: Items: branch $/TFSJavaBuild/Sources/Maven/Current Check-in Notes: Code Reviewer: Performance Reviewer: Security Reviewer:

          SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : redsolo
          Path:
          trunk/hudson/plugins/tfs/src/main/java/hudson/plugins/tfs/commands/DetailedHistoryCommand.java
          trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870-2.txt
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870.txt
          http://jenkins-ci.org/commit/34965
          Log:
          [FIXED JENKINS-6870] Fixed parsing of history output even if there is no starting separator when using the TEE tf.cmd.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : redsolo Path: trunk/hudson/plugins/tfs/src/main/java/hudson/plugins/tfs/commands/DetailedHistoryCommand.java trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870-2.txt trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6870.txt http://jenkins-ci.org/commit/34965 Log: [FIXED JENKINS-6870] Fixed parsing of history output even if there is no starting separator when using the TEE tf.cmd.

          SCM/JIRA link daemon added a comment -

          Code changed in hudson
          User: : redsolo
          Path:
          trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java
          trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6454.txt
          http://jenkins-ci.org/commit/34967
          Log:
          [FIXED JENKINS-6454] Fixed parsing of the MS TEE tf.cmd in issue JENKINS-6870 fixed also this issue.

          SCM/JIRA link daemon added a comment - Code changed in hudson User: : redsolo Path: trunk/hudson/plugins/tfs/src/test/java/hudson/plugins/tfs/commands/DetailedHistoryCommandTest.java trunk/hudson/plugins/tfs/src/test/resources/hudson/plugins/tfs/commands/issue-6454.txt http://jenkins-ci.org/commit/34967 Log: [FIXED JENKINS-6454] Fixed parsing of the MS TEE tf.cmd in issue JENKINS-6870 fixed also this issue.

            redsolo redsolo
            hartmut hartmut
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: