On 10 Nov 2009 at 13:09, Oren Held wrote:

> 
> I see two independent steps which we can take:
> 
> 1. RENAME: IMO that get-iqn can still be interpreted as "get current iqn". In 
> the SSH spirit, I'd choose iscsi-iqn-gen (like ssh-keygen)
> 
> Renaming, of course, might break 3rd party scripts who look for iscsi-iname.
> 
> 2. Print to STDERR "Generating a new IQN...".
> This one is not supposed to affect 3rd party scripts, which probably redirect 
> only the STDOUT.

In the spirit of UNIX, progress messages going to stderr are unusual, except 
the 
process will need significant toime to complete. (MHO)

Ulrich

> 
> 
> On Tuesday 10 November 2009 09:36:48 Ulrich Windl wrote:
> > On 8 Nov 2009 at 15:42, Oren wrote:
> > > Hello,
> > >
> > > I've encountered too many cases of people thinking the iscsi-iname
> > 
> > What about naming it "gen-iqn" instead (in the UNIX tradition of having a
> >  short name preferrable over
> >  "generate-new-iSCSI-qualified-name-for-initiator")?
> > 
> > Ulrich
> > 
> > > command, just like ifconfig, displays the currently configured IQN,
> > > not being aware that, in fact, it generates a new IQN.
> > > The man page (Debian-supplied, not in open-iscsi git) describes it
> > > best, but the --help string is absolutely misleading or even wrong.
> > >
> > > Here's my patch, that changes the wording to something similar to the
> > > Debian's man page:
> > > diff --git a/utils/iscsi-iname.c b/utils/iscsi-iname.c
> > > index 7cf0e7a..80fc1d1 100644
> > > --- a/utils/iscsi-iname.c
> > > +++ b/utils/iscsi-iname.c
> > > @@ -62,7 +62,7 @@ main(int argc, char *argv[])
> > >                 prefix = argv[1];
> > >                 if (( strcmp(prefix, "-h") == 0 ) ||
> > >                     ( strcmp(prefix, "--help") == 0 )) {
> > > -                       printf("\nDisplays the iSCSI initiator name
> > > \n");
> > > +                       printf("\nGenerates a unique iSCSI node name
> > > on every invocation\n"
> > >                         exit(0);
> > >                 } else if ( strcmp(prefix, "-p") == 0 ) {
> > >                         prefix = argv[2];
> > >
> > >
> > > Best Regards,
> > >
> > > Oren Held
> > > Host-side development
> > > IBM-XIV Storage
> > 
> > > 
> 
> -- 
> Oren Held
> Host-side development
> IBM-XIV
> 
> > 



--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"open-iscsi" group.
To post to this group, send email to open-iscsi@googlegroups.com
To unsubscribe from this group, send email to 
open-iscsi+unsubscr...@googlegroups.com
For more options, visit this group at http://groups.google.com/group/open-iscsi
-~----------~----~----~----~------~----~------~--~---

Reply via email to