search

autotest / virt-test

Linux Virtualization Tests
Other
97 stars 140 forks source link

virttest/remote.py: Add new param for RemoteFile.add() #2295

Closed PandaWei closed 9 years ago

PandaWei commented 9 years ago

Add new param 'newline' to indicate the content of file ends up with newline or not.

Signed-off-by: Wei,Jiangang weijg.fnst@cn.fujitsu.com

waynesun09 commented 9 years ago

ACK

lmr commented 9 years ago

Hi @PandaWei. As you might know, we're moving virt-test to avocado-vt, so I please ask you to consider re-implementing this functionality in the later. The new repo is:

https://github.com/avocado-framework/avocado-vt

Please keep in mind the following guidelines when creating the new PR:

1) Most of the time you can pick the patches of your original PR and merge them on a new branch with minimal changes. 3) Don't use autotest APIs, since we're moving away from using them. Please review the code and change things like utils.system -> process.system, utils.run -> process.run, so on and so forth.

Thanks, and I'm sorry for any inconveniences we might have caused your team.