Re: [PATCH] Add -C support to tgt-admin (against current trunk)

[Chandra Seetharaman <sekharan@xxxxxxxxxx>]

Sorry about the overlook on my part. Will send an updated patch shortly.

Thanks for the polite comments :)... It is very much appreciated.

chandra

On Wed, 2010-07-28 at 10:41 +0200, Tomasz Chmielewski wrote:
> On 28.07.2010 02:47, Chandra Seetharaman wrote:
> > Add support to manage multiple control ports.
> >
> > Basically add the ability to pass through "-C NNNN" (or --control_port
> > NNNN") option to tgtadm
> >
> > Signed-Off-By: Chandra Seetharaman<sekharan@xxxxxxxxxx
> > ---------------------------
> > Index: tgt-469b574/scripts/tgt-admin
> > ===================================================================
> > --- tgt-469b574.orig/scripts/tgt-admin	2010-07-21 17:49:37.000000000 -0700
> > +++ tgt-469b574/scripts/tgt-admin	2010-07-27 17:27:06.000000000 -0700
> > @@ -29,6 +29,7 @@
> >         --update<value>		update configuration for all or selected targets
> >   				(see "--update help" for more info)
> >     -s, --show			show all the targets
> > +  -C, --control_port<num>	specify an alternative configuration file
> >     -c, --conf<conf file>	specify an alternative configuration file
> 
> The above doesn't seem right, or?
> 
> However boring, it would be great to update the documentation as well 
> (document the new option).
> 
> 


--
To unsubscribe from this list: send the line "unsubscribe stgt" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html