Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make the commands executed for SSH+SCP, SSH+SUDO and SSH+INTERACTIVE_SUDO connections configurable #52

Closed
demobox opened this issue Sep 14, 2012 · 5 comments

Comments

@demobox
Copy link
Contributor

demobox commented Sep 14, 2012

Currently, the sudo connection methods use a two-step approach to copy files to a target location:

  1. Files are copied to a staging directory using the connecting user
  2. Files are moved to the target location using the sudo user

The command that carries out the second step is (depending on the value of the sudoPreserveAttributesOnCopyFromTempFile property), cp -r or cp -pr. It would be more flexible if the user could configure this command to support arbitrary flags.

@demobox
Copy link
Contributor Author

demobox commented Oct 17, 2012

See http://support.xebialabs.com/tickets/632 (sorry, private link) for a motivation for making this configurable. A short summary:

"the target user I use on a SunOS machine has the following alias :
alias -x rm="rm -i"
...
The project team says that Overthere should use "/bin/rm" instead of "rm" ... It doesn't seem clean to me and Overthere doesn't allow this at the moment...
And this kind of settings could also be necessary with other unix commands (cp for example maybe ?)"

@vpartington
Copy link
Contributor

That's actually a motivation to make all the commands Overthere use for SSH+SCP, SSH+SUDO and SSH+INTERACTIVE_SUDO configurable.

@demobox
Copy link
Contributor Author

demobox commented Oct 17, 2012

One Issue To Rule Them All? Or do we consider this simply an incompatible installation/setup of the target environment?

@vpartington
Copy link
Contributor

I'd love to consider it so but the reality is that such hosts exist.

@vpartington
Copy link
Contributor

Renamed the issue to cover all commands executed for SSH+SCP, SSH+SUDO and SSH+INTERACTIVE_SUDO connections. This means this issue also includes #51.

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

No branches or pull requests

3 participants