[#COMMONS-109] Miss-implication in logs for Deployment

advertisement
[COMMONS-109] Miss-implication in logs for Deployment Synchronizor Created:
07/Mar/14 Updated: 01/Sep/14
Status:
Project:
Component/s:
Affects
Version/s:
Fix Version/s:
In Progress
WSO2 Commons
None
None
Type:
Reporter:
Resolution:
Labels:
Remaining
Estimate:
Time Spent:
Original
Estimate:
Bug
Priority:
Pushpalanka Jayawardhana Assignee:
Unresolved
Votes:
None
Not Specified
Attachments:
SVNBasedArtifactRepository.patch
Major
Moderate
Severity:
Estimated
Complexity:
Attachment
License:
None
Highest
Kishanthan Thangarajah
0
Not Specified
Not Specified
patch.diff
I agree to grant a license to WSO2 for this work for inclusion in WSO2
works as per the WSO2 Contributor License Agreement and the Apache
License 2.0
Description
This is regarding the code at [1], which is miss-interpreting the revision number of the svn update to be the numb
} else {
filesUpdated = svnClient.update(root, SVNRevision.HEAD,
Depth.infinity, NO_SET_DEPTH,
ignoreExternals, forceUpdate);
if (log.isDebugEnabled())
{ log.debug(filesUpdated + " files were updated using SVN Kit"); }
}
According to [2], what is actually returned is the revision number. The logic inside this function also consider th
is incorrect.
[1] - https://svn.wso2.org/repos/wso2/carbon/platform/branches/turing/components/deploymentsynchronizer/org.wso2.carbon.deployment.synchronizer.subversion/4.2.0/src/main/java/org/wso2/carbon/deploy
[2] - https://svn.wso2.org/repos/wso2/carbon/platform/trunk/dependencies/svnclient/1.6.18.wso2v2/src/main/jav
Comments
Comment by Nipuni Perera [ 01/Sep/14 ]
Hi,
Pull request[1] sent to wso2-dev/carbon-commons.
[1] https://github.com/wso2-dev/carbon-commons/pull/15
Thanks,
Nipuni
Generated at Wed Feb 10 07:38:11 IST 2016 using JIRA 6.0.1#6096sha1:e4a48bd73c6b8a4d99c824976ce5808b4c85857d.
Download