search

jenkinsci / scm-sync-configuration-plugin

Jenkins scm-sync-configuration plugin
https://plugins.jenkins.io/scm-sync-configuration/
MIT License
115 stars 123 forks source link

[JENKINS-22820] fix - Bump maven.scm to 1.9.1 to avoid issues with Git v1.9 upgrades. #21

Closed rogerhu closed 10 years ago

rogerhu commented 10 years ago

The v1.9.1 version changes to use git status --porcelain mode, which prevents the parsing bug.

maven-release-plugin needs to be bumped to 2.5 since it has a dependency on maven-scm-api 1.9.

image

components.xml had to be updated to support Git-related tests too.

https://issues.jenkins-ci.org/browse/JENKINS-22820

Relates to: https://jira.codehaus.org/browse/SCM-686

cloudbees-pull-request-builder commented 10 years ago

plugins » scm-sync-configuration-plugin #33 UNSTABLE Looks like there's a problem with this pull request

cloudbees-pull-request-builder commented 10 years ago

plugins » scm-sync-configuration-plugin #34 UNSTABLE Looks like there's a problem with this pull request

cloudbees-pull-request-builder commented 10 years ago

plugins » scm-sync-configuration-plugin #35 UNSTABLE Looks like there's a problem with this pull request

jenkinsadmin commented 10 years ago

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

rogerhu commented 10 years ago

Not clear why the test runs fail with the upgraded packages, but the plugin works fine for me now.

rogerhu commented 10 years ago

The one issue is that if your projects have spaces in them, the new Maven SCM plugins will not be able to check in code because of URI exceptions.

rogerhu commented 10 years ago

Updated components.xml so tests will pass now.

cloudbees-pull-request-builder commented 10 years ago

plugins » scm-sync-configuration-plugin #36 SUCCESS This pull request looks good

fcamblor commented 10 years ago

Thanks @rogerhu for this PR which should solves JENKINS-22820 and maybe JENKINS-15128 as well.