[prev in list] [next in list] [prev in thread] [next in thread] 

List:       ntop-dev
Subject:    [Ntop-dev] Change to --set-admin-password in main.c -- why?
From:       "Burton M. Strauss III" <bstrauss3 () attbi ! com>
Date:       2002-11-13 18:11:35
Message-ID: JIEPJGFPFMFIGBNCPKGGEEIEDCAA.bstrauss3 () attbi ! com
[Download RAW message or body]

Luca:

Buried in the change removing -S et al, is this:

-  { "set-admin-password",               optional_argument, NULL, 135 },
+  { "set-admin-password",               optional_argument, NULL, 'A' },

et al


But you didn't remove this:

    case 135:
      /* Dennis Schoen (dennis@cns.dnsalias.org)
allow --set-admin-password=<password> */
      if (optarg) {
        stringSanityCheck(optarg);
        initGdbm(NULL);
        initThreads();
        setAdminPassword(optarg);
        exit(0);
      } else {
        setAdminPw = 1;
      }
      break;

Which is what allows the --set-admin-password= option which is useful for
scripting ntop installations.  Because of the way getopts_long treats =s,
you have to split the option (see the L/131 stuff)

Why was it removed?  Do you have a problem with putting it back?


-----Burton



[prev in list] [next in list] [prev in thread] [next in thread]