Oh yeah, the c not followed by a colon should be an s. You can just fix it
on your end, that's easier.

On Tue, Jul 26, 2016 at 3:23 PM barret rhoden <[email protected]> wrote:

> Hi -
>
> I think this line is wrong:
>
> > +     while ((c = getopt(argc, argv, "dvm:c:gcf:k:")) != -1) {
>                  should be an s not a c here---^
>
> Meaning, we don't have an s option in the getopt line.
>
> Here's the full switch:
>
> > +             switch (c) {
> > +                     case 'd':
> > +                             debug++;
> > +                             break;
> > +                     case 'v':
> > +                             vmmflags |= VMM_VMCALL_PRINTF;
> > +                             break;
> > +                     case 'm':
> > +                             maxresume = strtoull(optarg, 0, 0);
> > +                             break;
> > +                     case 'c':
> > +                             cmdline_extra = optarg;
> > +                     case 'g':       /* greedy */
> > +                             parlib_never_yield = TRUE;
> > +                             break;
> > +                     case 's':       /* scp */
> > +                             parlib_wants_to_be_mcp = FALSE;
> > +                             break;
> > +                     case 'f':       /* file to pass to blk_init */
> > +                             disk_image_file = optarg;
> > +                             break;
> > +                     case 'k':       /* specify file to get cmdline
> args from */
>
> I can touch that up on my end if you'd like.  If not, feel free to post
> another patch/patchset.
>
> Barret
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Akaros" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to