Skip to content
This repository has been archived by the owner on Jun 27, 2020. It is now read-only.

add ability to override default publish commit log message #32

Open
GoogleCodeExporter opened this issue Jul 31, 2015 · 5 comments
Open

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1. invoke ivysvn resolver via ivy publish

What is the expected output? What do you see instead?
Expect: ability to specify a commit message based on current build script
Instead: hard coded message was used when artifacts were published to svn repo

What version of the product are you using? On what operating system?
latest, rhel

Please provide any additional information below.
Would like ability to set (override) default message with a message from my 
build.xml script.
<target ...>
  <property name="ivysvn.publish.commit.message" value="my commit message not yours" />
  <ivy:publish ... configured to use ivysvn resolver ... />
</target>


Original issue reported on code.google.com by doug.ros...@gmail.com on 10 Mar 2011 at 4:13

@GoogleCodeExporter
Copy link
Author

Can't edit above entry: this is NOT a defect it is a feature request. Priority 
for me is high and will attempt to implement myself to help speed up delivery 
of this feature.

Original comment by doug.ros...@gmail.com on 10 Mar 2011 at 4:15

@GoogleCodeExporter
Copy link
Author

Probably worth looking at how the values like ssh port, user, password etc. get 
set on SvnResolver as this might be a better approach, so the ivysettings.xml 
could have an additional, optional attribute like "commitComment" which can 
then be filled via an ant property, a hardcoded string etc.

Original comment by massdosage on 10 Mar 2011 at 4:30

  • Added labels: Type-Enhancement
  • Removed labels: Type-Defect

@GoogleCodeExporter
Copy link
Author

Added publishMessage attribute on SvnResolver. Attaching patch file with code 
changes.

Patch file was created from local working copy based on 
http://ivysvn.googlecode.com/svn/trunk
svn diff -r HEAD >changes.patch

Summary of changes:

M       test/java/fm/last/ivy/plugins/svnresolver/test/TestProperties.java
M       
test/java/fm/last/ivy/plugins/svnresolver/BaseSvnRepositoryPublishTestCase.java
M       test/java/fm/last/ivy/plugins/svnresolver/BaseTestCase.java
M       
test/java/fm/last/ivy/plugins/svnresolver/SvnRepositoryPublishAndRetrieveTest.ja
va
M       test/java/fm/last/ivy/plugins/svnresolver/BaseIvyTestCase.java
A       test/data/ivysettings/ivysettings-issue32.xml
M       src/java/fm/last/ivy/plugins/svnresolver/SvnPublishTransaction.java
M       src/java/fm/last/ivy/plugins/svnresolver/SvnRepository.java
M       src/java/fm/last/ivy/plugins/svnresolver/SvnResolver.java

Original comment by doug.ros...@gmail.com on 11 Mar 2011 at 10:10

Attachments:

@GoogleCodeExporter
Copy link
Author

Uploading new file not in patch output

Original comment by doug.ros...@gmail.com on 11 Mar 2011 at 10:11

Attachments:

@GoogleCodeExporter
Copy link
Author

the above code is submitted and to be included in the ivysvn code base in 
accrodance and agreement with the Apache license

Original comment by doug.ros...@gmail.com on 12 Mar 2011 at 12:40

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant