Re: [PATCH 3/3] kconfig and lxdialog, kernel 2.6.13.4

From: Roman Zippel (zippel_at_linux-m68k.org)
Date: 10/27/05

  • Next message: Bernd Eckenfels: "Re: 4GB memory and Intel Dual-Core system" 
    Date:	Thu, 27 Oct 2005 23:27:42 +0200 (CEST)
To: "Fao, Sean" <sean.fao@capitalgenomix.com>

    Hi,

    On Thu, 27 Oct 2005, Fao, Sean wrote:

    > This patch is a functionality modification that implements a proposed design
    > change to lxdialog. Specifically, this patch makes use of the proposed
    > "Abort" functionality of lxdialog, which adds an "Abort" button in *addition*
    > to the "Yes" and "No" buttons that are currently displayed when a user exits
    > kconfig. The Abort button allows a user to return to the root menu of kconfig
    > rather than exiting.

    This reminds me I had a similiar patch lying around, could you use this
    one as a basis.

    bye, Roman

     scripts/kconfig/mconf.c | 49 +++++++++++++++++++++++++-----------------------
     1 file changed, 26 insertions(+), 23 deletions(-)

    Index: linux-2.6/scripts/kconfig/mconf.c
    ===================================================================
    --- linux-2.6.orig/scripts/kconfig/mconf.c 2005-10-27 23:02:15.000000000 +0200
    +++ linux-2.6/scripts/kconfig/mconf.c 2005-10-27 23:04:12.000000000 +0200
    @@ -1064,34 +1064,37 @@ int main(int ac, char **av)
             tcgetattr(1, &ios_org);
             atexit(conf_cleanup);
             init_wsize();
    - conf(&rootmenu);
    + while (1) {
    + conf(&rootmenu);
     
    - do {
    - cprint_init();
    - cprint("--yesno");
    - cprint(_("Do you wish to save your new kernel configuration?"));
    - cprint("5");
    - cprint("60");
    - stat = exec_conf();
    - } while (stat < 0);
    + do {
    + cprint_init();
    + cprint("--yesno");
    + cprint(_("Do you wish to save your new kernel configuration?"));
    + cprint("5");
    + cprint("60");
    + stat = exec_conf();
    + } while (stat < 0);
     
    - if (stat == 0) {
    - if (conf_write(NULL)) {
    + switch (stat) {
    + case 0:
    + if (conf_write(NULL)) {
    + fprintf(stderr, _("\n\n"
    + "Error during writing of the kernel configuration.\n"
    + "Your kernel configuration changes were NOT saved."
    + "\n\n"));
    + return 1;
    + }
    + printf(_("\n\n"
    + "*** End of Linux kernel configuration.\n"
    + "*** Execute 'make' to build the kernel or try 'make help'."
    + "\n\n"));
    + return 0;
    + case 1:
                             fprintf(stderr, _("\n\n"
    - "Error during writing of the kernel configuration.\n"
                                     "Your kernel configuration changes were NOT saved."
                                     "\n\n"));
    - return 1;
    + return 0;
                     }
    - printf(_("\n\n"
    - "*** End of Linux kernel configuration.\n"
    - "*** Execute 'make' to build the kernel or try 'make help'."
    - "\n\n"));
    - } else {
    - fprintf(stderr, _("\n\n"
    - "Your kernel configuration changes were NOT saved."
    - "\n\n"));
             }
    -
    - return 0;
     }
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

  • Next message: Bernd Eckenfels: "Re: 4GB memory and Intel Dual-Core system"