On Mon, 2010-05-17 at 12:54 +0100, [email protected] wrote:
> From: marios <[email protected]>
> 
> ---
>  .../deltacloud/drivers/rimu/rimu_hosting_client.rb |   87 ------------
>  .../deltacloud/drivers/rimu/rimu_hosting_driver.rb |  143 -------------------
>  .../drivers/rimuhosting/rimuhosting_client.rb      |   88 ++++++++++++
>  .../drivers/rimuhosting/rimuhosting_driver.rb      |  144 
> ++++++++++++++++++++
>  server/lib/drivers.rb                              |    6 +-
>  5 files changed, 234 insertions(+), 234 deletions(-)
>  delete mode 100755 server/lib/deltacloud/drivers/rimu/rimu_hosting_client.rb
>  delete mode 100755 server/lib/deltacloud/drivers/rimu/rimu_hosting_driver.rb
>  create mode 100755 
> server/lib/deltacloud/drivers/rimuhosting/rimuhosting_client.rb
>  create mode 100755 
> server/lib/deltacloud/drivers/rimuhosting/rimuhosting_driver.rb

To keep the change history understandable, it's generally better to
separate moving/renaming files from actual changes. It's really hard in
this patch what's changed because of the rename (and whatever trivial
fixup is needed to make the code work), and what is an actual code
change.

If it's not a terrible amount of work, can you split hte patch in two ?

David


_______________________________________________
deltacloud-devel mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/deltacloud-devel

Reply via email to