PATCH: Improve help info for copyfile

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



        For some reason the command-line arguments for copyfile can only be 
found online or in the source code.  They are not part of the "usage" info, 
which is literally just a string defined in the copyfile command module.  This 
patch makes that string a little more useful, as follows:


[root@centos1 ~]# func '*' copyfile
Usage: func [--options] "hostname glob" module method [arg1] [arg2] ...  func 
[--options] "hostname glob" module method [arg1] [arg2] ...  
  copyfile -f localfile --remotepath /remotepath/filename
    localfile = the file on the overlord to copy
    /remotepath/filename = the full path for the copied file on the minion



        This patch changes one line of code. :)

--
Marcus
>From 220bad7ca66d82ea492eb19bce793d0b301464a3 Mon Sep 17 00:00:00 2001
From: Marcus Lauer <marcuslauer@xxxxxxxxxx>
Date: Wed, 12 Sep 2012 18:55:51 -0400
Subject: [PATCH] CopyFile: improve the usage (help) info for copyfile

---
 func/overlord/cmd_modules/copyfile.py |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/func/overlord/cmd_modules/copyfile.py b/func/overlord/cmd_modules/copyfile.py
index 3f2263d..d427992 100644
--- a/func/overlord/cmd_modules/copyfile.py
+++ b/func/overlord/cmd_modules/copyfile.py
@@ -25,7 +25,7 @@ from func.overlord import client
 
 class CopyFile(base_command.BaseCommand):
     name = "copyfile"
-    usage = "copy a file to a client"
+    usage = "\n  copyfile -f localfile --remotepath /remotepath/filename\n    localfile = the file on the overlord to copy\n    /remotepath/filename = the full path for the copied file on the minion"
     summary = "copy a file to a minion"
 
 
-- 
1.7.7

_______________________________________________
func mailing list
func@xxxxxxxxxxxxxxxxxxxxxx
https://lists.fedorahosted.org/mailman/listinfo/func

[Index of Archives]     [Fedora Users]     [Linux Networking]     [Fedora Legacy List]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux