dajester2013 / ivysvn

Automatically exported from code.google.com/p/ivysvn
Apache License 2.0
0 stars 0 forks source link

add ability to override default publish commit log message #32

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
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 commented 9 years ago
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 commented 9 years ago
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

GoogleCodeExporter commented 9 years ago
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 commented 9 years ago
Uploading new file not in patch output

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

Attachments:

GoogleCodeExporter commented 9 years ago
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