18343 2008-12-25 11:03:07 +0300 Сделать выход по ^D из apt-shell 2009-05-10 09:33:59 +0400 1 1 4 Development Sisyphus apt unstable all Linux CLOSED FIXED patch P2 minor --- 1 wrar placeholder boyarsh dottedmag evg glebfm imz iv ldv led mike php-coder placeholder rider vt qa-sisyphus oldest_to_newest 83411 0 wrar 2008-12-25 11:03:07 +0300 Сделайте выход из apt-shell по ^D, а то без него неудобно. 83678 1 php-coder 2008-12-30 22:37:48 +0300 Ctrl+D в readline используется для удаления текущего символа под курсором. Если бы apt-shell не использовал libreadline, то можно было бы вручную проверять на EOF и выходить. 83682 2 dottedmag 2008-12-30 22:45:51 +0300 Слава, см. документацию на readline: rhttp://tiswww.case.edu/php/chet/readline/readline.html#SEC16 "delete-char (C-d) Delete the character at point. If point is at the beginning of the line, there are no characters in the line, and the last character typed was not bound to delete-char, then return EOF." http://tiswww.case.edu/php/chet/readline/readline.html#SEC24 "If readline encounters an EOF while reading the line, and the line is empty at that point, then (char *)NULL is returned." В apt-shell.cc: line = readline(_config->Find("APT::Shell::Prompt", "apt> ").c_str()); if (!line || !*line) { free(line); continue; } Так что здесь всего лишь надо случай if(!line) рассмотреть отдельно. 83683 3 php-coder 2008-12-30 22:51:11 +0300 2dottedmag@: о, спасибо. Попробую сделать. 83685 4 php-coder 2008-12-30 23:15:09 +0300 Вот предлагаемый патч: http://git.altlinux.org/people/php_coder/packages/apt.git?p=apt.git;a=commitdiff;h=7956d444db6d6c92ec675a2b00926b166672c2da 83709 5 php-coder 2008-12-31 16:47:26 +0300 *** Bug 6264 has been marked as a duplicate of this bug. *** 83802 6 ldv 2009-01-02 03:25:10 +0300 0.5.15lorg2-alt22.1 83836 7 3182 iv 2009-01-03 16:04:14 +0300 Created attachment 3182 add feedback to Ctrl-D Огромное спасибо за проделанную работу! Однако, есть небольшое замечание. При нажатии Ctrl-D не выполняется переход на новую строку, в результате чего получается следующее: apt> [root@deimos ~]# Как-то... не эстетично ;) Например, bash делает так: http://git.altlinux.org/people/ldv/packages/bash.git?p=bash.git;a=blob;f=bash/builtins/exit.def;h=ddaa5d315cd44f215a0b7e928f7f369ec9106a1a;hb=13bd4196973ac0d4d299b2b24ffb22efbae6d055#l63 Предлагаю дополненный одной строчкой патч, который делает практически также, только на C++. 83841 8 php-coder 2009-01-03 18:11:19 +0300 (In reply to comment #7) > Однако, есть небольшое замечание. При нажатии Ctrl-D не выполняется переход на > новую строку, в результате чего получается следующее: > apt> [root@deimos ~]# apt> % [c0der@rock ~]$ У меня в zsh вот так, поэтому я и не заметил неэстетичности. 83978 9 led 2009-01-08 20:49:06 +0300 (In reply to comment #7) > Предлагаю дополненный одной строчкой патч, который делает практически > также, только на C++. Присоединяюсь к пожеланию: нужно убрать эту неэстетичность. достаточно: c1out << endl; 83979 10 led 2009-01-08 20:50:10 +0300 лучше всё же зафиксить 84054 11 php-coder 2009-01-09 21:14:08 +0300 (In reply to comment #10) > лучше всё же зафиксить Будет исправлено в 0.5.15lorg2-alt22.2 84215 12 php-coder 2009-01-11 00:09:59 +0300 Патч принят апстримом: http://apt-rpm.org/scm/?p=apt.git;a=commitdiff;h=c4822097f9a52a5d498640e5d3a16c6fd555b5bc 3182 2009-01-03 16:04:14 +0300 2009-01-03 16:04:14 +0300 add feedback to Ctrl-D apt-0.5.15lorg2-alt-apt-shell_quit_by_ctrld.patch text/plain 759 iv Cmh0dHBzOi8vYnVnemlsbGEuYWx0bGludXgub3JnL3Nob3dfYnVnLmNnaT9pZD0xODM0MwoKZGlm ZiAtLWdpdCBhL2FwdC9jbWRsaW5lL2FwdC1zaGVsbC5jYyBiL2FwdC9jbWRsaW5lL2FwdC1zaGVs bC5jYwppbmRleCA2ZDFmNzY3Li5mZjRmZDJkIDEwMDY0NAotLS0gYS9hcHQvY21kbGluZS9hcHQt c2hlbGwuY2MKKysrIGIvYXB0L2NtZGxpbmUvYXB0LXNoZWxsLmNjCkBAIC00NDc3LDEwICs0NDc3 LDE4IEBAIGludCBtYWluKGludCBhcmdjLGNvbnN0IGNoYXIgKmFyZ3ZbXSkKICAgICAgIH0KICAg ICAgIAogICAgICAgbGluZSA9IHJlYWRsaW5lKF9jb25maWctPkZpbmQoIkFQVDo6U2hlbGw6OlBy b21wdCIsICJhcHQ+ICIpLmNfc3RyKCkpOwotICAgICAgaWYgKCFsaW5lIHx8ICEqbGluZSkgewot CSBmcmVlKGxpbmUpOwotCSBjb250aW51ZTsKKyAgICAgIGlmICghbGluZSkgeworICAgICAgICAv LyBleGl0IGZyb20gc2hlbGwgYnkgQ3RybC1ECisgICAgICAgICBjMW91dCA8PCAicXVpdCIgPDwg ZW5kbDsKKyAgICAgICAgIF9jb25maWctPlNldCgicXVpdCIsICJ0cnVlIik7CisgICAgICAgICBj b250aW51ZTsKKyAgICAgIH0KKworICAgICAgaWYgKCEqbGluZSkgeworICAgICAgICAgZnJlZShs aW5lKTsKKyAgICAgICAgIGNvbnRpbnVlOwogICAgICAgfQorCiAgICAgICBhZGRfaGlzdG9yeShs aW5lKTsKIAogICAgICAgbGFyZ2MgPSAxOyAvLyBDb21tYW5kTGluZS5QYXJzZSgpIGlnbm9yZXMg dGhlIGZpcnN0IG9wdGlvbi4K