Bug 48478 - Add missing exec features to SSHExec: error, errorproperty, resultproperty
Summary: Add missing exec features to SSHExec: error, errorproperty, resultproperty
Status: RESOLVED FIXED
Alias: None
Product: Ant
Classification: Unclassified
Component: Optional Tasks (show other bugs)
Version: 1.7.1
Hardware: PC Mac OS X 10.4
: P2 enhancement (vote)
Target Milestone: 1.9.4
Assignee: Ant Notifications List
URL:
Keywords: PatchAvailable
Depends on:
Blocks:
 
Reported: 2010-01-02 19:37 UTC by Adrian Cole
Modified: 2014-01-07 18:11 UTC (History)
1 user (show)


Attachments
adding missing features (5.59 KB, patch)
2013-06-15 18:02 UTC, Gilbert Rebhan 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian Cole 2010-01-02 19:37:18 UTC 
jsch supports error streams and capturing the result code.  Please add functionality into the SSHExec task.

error  	The file to which the standard error of the command should be redirected. 
errorproperty  	The name of a property in which the standard error of the command should be stored.
resultproperty  	The name of a property in which the return code of the command should be stored
Comment 1 Gilbert Rebhan 2013-06-15 18:02:56 UTC 
Created attachment 30439 [details]
adding missing features
Comment 2 Gilbert Rebhan 2013-06-15 18:03:08 UTC 
Just uploaded a patch for https://issues.apache.org/bugzilla/show_bug.cgi?id=48478,
which provides the following enhancements :
+ errorProperty
+ errorFile
+ appenderr
+ suppressSystemErr
+ resultProperty
Regards, Gilbert Rebhan
Comment 3 Stefan Bodewig 2014-01-06 15:41:35 UTC 
patch applied with svn revision 1555866 - thanks
Comment 4 Stefan Bodewig 2014-01-06 15:56:15 UTC 
Gilbert, I think I spotted a bug in your patch, see svn revision 1555873

Could you please have a look?
Comment 5 Gilbert Rebhan 2014-01-07 18:11:39 UTC 
(In reply to Stefan Bodewig from comment #4)
> Gilbert, I think I spotted a bug in your patch, see svn revision 1555873
> 
> Could you please have a look?

Thanks for the hint Stefan, you're right of course !
A dumb copy & paste error ;-)