public inbox for hail-devel@vger.kernel.org
 help / color / mirror / Atom feed
* [cld patch 1/1] use specified username in cldcli
@ 2010-05-03  4:07 Pete Zaitcev
  2010-05-03  5:59 ` Jeff Garzik
  0 siblings, 1 reply; 2+ messages in thread
From: Pete Zaitcev @ 2010-05-03  4:07 UTC (permalink / raw)
  To: jeff; +Cc: Project Hail List

I suspect I copy-pasted over it when I converted to ncld, but anyhow this
patch seems work and do what's expected for --user flag.

Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>

---
 tools/cldcli.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/cldcli.c b/tools/cldcli.c
index 7c73091..79a1009 100644
--- a/tools/cldcli.c
+++ b/tools/cldcli.c
@@ -712,7 +712,7 @@ int main (int argc, char *argv[])
 	dr = host_list->data;
 
 	nsess = ncld_sess_open(dr->host, dr->port, &error, sess_event, NULL,
-			     "cldcli", "cldcli", &cli_log);
+			     our_user, our_user, &cli_log);
 	if (!nsess) {
 		if (error < 1000) {
 			fprintf(stderr, TAG ": cannot open CLD session: %s\n",

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [cld patch 1/1] use specified username in cldcli
  2010-05-03  4:07 [cld patch 1/1] use specified username in cldcli Pete Zaitcev
@ 2010-05-03  5:59 ` Jeff Garzik
  0 siblings, 0 replies; 2+ messages in thread
From: Jeff Garzik @ 2010-05-03  5:59 UTC (permalink / raw)
  To: Pete Zaitcev; +Cc: Project Hail List

On 05/03/2010 12:07 AM, Pete Zaitcev wrote:
> I suspect I copy-pasted over it when I converted to ncld, but anyhow this
> patch seems work and do what's expected for --user flag.
>
> Signed-off-by: Pete Zaitcev<zaitcev@redhat.com>
>
> ---
>   tools/cldcli.c |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/cldcli.c b/tools/cldcli.c
> index 7c73091..79a1009 100644
> --- a/tools/cldcli.c
> +++ b/tools/cldcli.c
> @@ -712,7 +712,7 @@ int main (int argc, char *argv[])
>   	dr = host_list->data;
>
>   	nsess = ncld_sess_open(dr->host, dr->port,&error, sess_event, NULL,
> -			     "cldcli", "cldcli",&cli_log);
> +			     our_user, our_user,&cli_log);
>   	if (!nsess) {
>   		if (error<  1000) {

applied

PS. you sent this to jeff@garzik.com, and unfortunately, I don't own 
that domain.  I should.  :)  For future reference, jgarzik@pobox.com or 
jeff@garzik.org are equivalent and should exist for the long term (even 
if I get fired from Red Hat or somesuch :)).

	Jeff




^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-05-03  5:59 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-05-03  4:07 [cld patch 1/1] use specified username in cldcli Pete Zaitcev
2010-05-03  5:59 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox