[00:08:04] slavazanko/h вышел из конференции [04:04:15] slavazanko/w вышел из конференции [04:05:04] slavazanko/w зашёл в конференцию [05:10:07] andrew_b зашёл в конференцию [05:17:57] slavazanko/w вышел из конференции [05:18:51] slavazanko/w зашёл в конференцию [06:30:14] theMIROn зашёл в конференцию [06:30:26] theMIROn вышел из конференции [06:31:47] theMIROn зашёл в конференцию [06:31:52] theMIROn вышел из конференции: Miranda IM. Menshe. Pushche. Proshhe! [06:32:08] theMIROn зашёл в конференцию [06:32:13] theMIROn вышел из конференции: Miranda IM. Menshe. Pushche. Proshhe! [06:32:29] theMIROn зашёл в конференцию [06:32:34] theMIROn вышел из конференции: Miranda IM. Menshe. Pushche. Proshhe! [06:32:51] theMIROn зашёл в конференцию [06:32:55] theMIROn вышел из конференции: Miranda IM. Menshe. Pushche. Proshhe! [06:49:15] <iliamaslakov> andrew_b, посмотри 1535 [06:49:21] <iliamaslakov> там рай [06:49:58] <andrew_b> Я там кое-что подправил. [06:51:33] <andrew_b> Чё-то не понял. Зачем дублирование? vfs_print_message (_("sftp: Invalid host name.")); + sftpfs_add_error (_("sftp: Invalid host name.")); [06:52:34] <andrew_b> vfs_print_message (_("sftp: Invalid host name."));
vfs_print_message (_("sftp: connection interrupted by user"));
Бардак. То с маленькой буквы, то с большой. То есть точка в конце, то нет. [06:53:22] <andrew_b> vfs_print_message (_("sftp: Password is empty.")); + sftpfs_add_error (_("sftp: Required password is empty"));
Почему разные сообщения? [06:54:02] <andrew_b> + sftpfs_error_string = g_string_sized_new (250); + g_string_assign (sftpfs_error_string, "");
Вторая строчка не нужна. [06:54:41] <andrew_b> + (void) query_dialog (_("sftpfs"), sftpfs_error_string->str, D_ERROR, 1, _("&OK")); return result;
Чтроку не надо удалять? [07:19:08] <iliamaslakov> vfs_print_message хочу выгасить [07:19:25] <iliamaslakov> >Вторая строчка не нужна. ок [07:19:59] <iliamaslakov> >vfs_print_message пока оставил чтоьы видно было где надо передавать параметры [07:20:22] <iliamaslakov> кое где код ошибки возвращается [07:20:42] <iliamaslakov> надо его через (%i) подставить [07:20:50] <iliamaslakov> вчера перед сном не успел [07:20:59] <iliamaslakov> но на функционал не влияет [07:21:25] <iliamaslakov> >Бардак. То с маленькой буквы, то с большой. То есть точка в конце, то нет. это сейчас пробегусь приведу к единому виду [07:47:57] <iliamaslakov> andrew_b, поправил [08:14:15] iliamaslakov вышел из конференции [08:14:55] iliamaslakov зашёл в конференцию [08:15:10] iliamaslakov вышел из конференции [08:15:23] iliamaslakov зашёл в конференцию [08:33:39] <slavazanko/w> всем привет [08:33:57] <andrew_b> Привет. [08:59:30] <iliamaslakov> slavazanko/w, Привет! [08:59:46] <iliamaslakov> slavazanko/w, все твои замечания поправил! [08:59:53] <slavazanko/w> о. гуд. [09:00:12] <slavazanko/w> я вчера немного подкоммитился [09:00:14] <iliamaslakov> там рай! [09:00:28] <iliamaslakov> а чего коммитил то? [09:00:38] <iliamaslakov> я что то не помню :) [09:00:58] <slavazanko/w> https://www.midnight-commander.org/ticket/55 https://www.midnight-commander.org/ticket/2695
Оба на ревью [09:02:12] <slavazanko/w> по 2695: там чуток коммитов, штук сорок. Понимаю, что много, поэтому энтузиазма при ревьюве не жду. [09:03:13] <andrew_b> По 55: я местами против. После обеда расскажу. [09:03:18] <slavazanko/w> ок [09:03:57] <slavazanko/w> в 55 моя основная идея: вынести флаги автодополнения "наружу", в том числе и с quick_dialog [09:11:49] <iliamaslakov> slavazanko/w, может пока соберёшь 1535? :) [09:15:59] <slavazanko/w> ща [09:33:24] <andrew_b> slavazanko/w: идея правильная, но... [09:33:39] <andrew_b> INPUT_COMPLETE_DEFAULT_AND_FILES выглядит ужасно. :) [09:34:11] <slavazanko/w> гм. а как тогда обозвать? Переименовать - как два пальца :) [09:34:13] <andrew_b> Во-вторых, по умолчанию, автодополнение должно быть выключено. [09:34:48] <andrew_b> Иначе оно появляется в странных для него местах, например, в диалоге поиска/замены в редакторе. [09:35:04] <andrew_b> Там ведь автодополнение файлов не нужно? [09:35:26] <slavazanko/w> угу, есть такое. Значит, два вопроса: переименовать константу и упорядочить использование автодополнения [09:35:37] <slavazanko/w> во что переименовать? [09:35:57] <andrew_b> Во, первых, INPUT_COMPLETE_DEFAULT = 0. [09:37:32] <andrew_b> Дальше надо думать. [09:37:52] <slavazanko/w> iliamaslakov: я подпушил 1535 [09:37:59] <andrew_b> INPUT_COMPLETE_FILES_ESC для чего? [09:38:53] <slavazanko/w> ну по идее для игнорирования пробела в имени файла. Например, оно нужно при автодополнении в диалоге копирования/перемещения, но не нужно в автодополнении комстроки под панелями [09:42:14] <andrew_b> Можно не вводить новое имя для группировки нескольких флагов, я прописывать их всех явно, которые нужны в данном месте. [09:46:13] <slavazanko/w> ну.. пожалуй да. [09:46:34] <slavazanko/w> тогда ввести INPUT_COMPLETE_NONE = 0 [09:46:46] <andrew_b> Нет. [09:46:51] <andrew_b> Именно DEFAULT. [09:47:02] <slavazanko/w> ок [09:47:03] <andrew_b> По дефолту ничего не дополняем. [09:53:27] <iliamaslakov> >slavazanko/w: собрал? [09:53:36] <slavazanko/w> да [09:54:28] <slavazanko/w> - Press S-F4 to edit new session +Press S-F4 to createt new session [09:54:46] <andrew_b> Да. [09:55:29] <andrew_b> Более того, я ожидаю, что если я нажму на Enter на это строке, то это таки означает createt new session. А оно фигу. [09:56:10] <slavazanko/w> :) -createt +create [09:56:13] <iliamaslakov> написано же нажми S-F4 [09:56:26] <andrew_b> Enter -- это интуитивно. [09:56:28] <slavazanko/w> а я тупой. жму энтер [09:56:36] <slavazanko/w> и вижу форму ввода пароля [09:56:40] <iliamaslakov> ВЫ ВСЕ ТУПЫЕ! [09:56:47] <slavazanko/w> а потом вижу sftp: Authentication by public key failed. [09:56:56] <slavazanko/w> атто. мы ещё хуже [09:57:00] <iliamaslakov> ща, сделаю [09:57:39] <andrew_b> А куда Public key исчез? [09:58:13] <andrew_b> Почему при первом запуске Auth method ничего не выбрано? [09:58:51] <slavazanko/w> (*) Public key SSH private key:
как-то оно звучит не очень.. выбираем public, а вводим путь к private
Может. (*) SSH-key ? [09:59:58] <slavazanko/w> но уже можно вполне пользоваться, наши придирки - это больше уже косметика. Илья, ты крут неимоверно. :) Молодца [10:02:16] <iliamaslakov> ?? [10:02:51] <slavazanko/w> молодец, грю. Выстрадал таки sftp [10:03:12] <iliamaslakov> а... [10:03:25] <iliamaslakov> я был бы крут если бы понимал как оно работает.. [10:20:29] <iliamaslakov> с паблик кейс [10:20:43] <iliamaslakov> ну это же терминология [10:20:50] <iliamaslakov> общепринятая [10:35:05] <iliamaslakov> настаиваешь на переименовании? [10:36:33] <slavazanko/w> ну.. пусть будет SSH-Key [10:36:52] <andrew_b> SSH key [10:39:42] <iliamaslakov> сделал [13:11:05] <iliamaslakov> slavazanko/w, больше не пушил? [13:11:13] <slavazanko/w> неа [13:11:51] <andrew_b> iliamaslakov: почему ты затёр мои изменения? [13:12:03] <iliamaslakov> ?? [13:12:10] <iliamaslakov> не затирал [13:12:17] <iliamaslakov> засквошил наверное [13:12:26] <iliamaslakov> про что были? [13:12:30] <andrew_b> А куда же они делись? [13:12:43] <iliamaslakov> я не брал [13:12:49] <andrew_b> Запушь, я восстановлю. [13:12:51] <iliamaslakov> Это славик [13:13:02] <iliamaslakov> запушил [13:13:18] <iliamaslakov> в смысле что я запушил [13:13:46] <iliamaslakov> и в том плане что если что то плохое то это точно не я а кто то другой [13:14:06] <slavazanko/w> нутыпипец стрелочник [13:14:41] <slavazanko/w> славег ничего не ребайзал и мощно не пушил. Так шта того... подставляй опу под развальцофку [13:14:53] <iliamaslakov> ) [13:14:58] <andrew_b> -SFTP config dialog +SFTP configuration dialog [13:14:59] <iliamaslakov> а ты здесь [13:15:48] <andrew_b> Ну пипец.
+.\"NODE " SFTP config dialog " +.SH " SFTP config dialog " +This dialog is intended for entering and adjustment information on SFTP server. +Here you may assign the following parameters: +.I session name +small description of SFTP server +.I host +hostname or IP address SFTP server +.I port +specify the port used by remote server +.I user name +element, the Midnight Commander will try to login on the remote +machine as that user, otherwise it will use your login name. +.I auth method +select authentication method of SFTP server (password, public keys, ssh-agent). +.I SSH private key +path to private key. [13:16:00] <iliamaslakov> чоо? [13:16:17] <andrew_b> Плохо, чо. [13:16:31] <iliamaslakov> опять отмонтировать забыла? (с) [13:16:55] <andrew_b> - This dialog is intended for entering and adjustment information on SFTP server. +This dialog is intended to enter and adjust information of SFTP server [13:17:29] <andrew_b> the following parameters the с множественным числом не употребляется. [13:17:52] <andrew_b> -hostname or IP address SFTP server +hostname or IP address of SFTP server [13:18:24] <andrew_b> +.I host +hostname or IP address SFTP server +.I port +specify the port used by remote server
То не было глагола, то вдруг появился. [13:19:04] <iliamaslakov> ProxyKit for FAR manager [13:19:10] <iliamaslakov> Vasily V. Moshninov [13:19:11] <andrew_b> +.I auth method +select authentication method of SFTP server (password, public keys, ssh-agent). +.I SSH private key +path to private key.
То был глагол, то вдруг ичсчез. [13:19:19] <iliamaslakov> ох уж этот василий [13:19:23] <andrew_b> iliamaslakov: да похеру [13:19:30] <iliamaslakov> английского нихрена не знает [13:20:03] <iliamaslakov> это я у него украл [13:20:22] <iliamaslakov> я ж не знал что вы проверять будете [13:21:15] <iliamaslakov> чо мне править, да? [13:21:23] <andrew_b> Погоди. [13:21:28] <iliamaslakov> ты то своё уже запушил? [13:21:32] <andrew_b> Тебе, кому ж ещё. [13:21:33] <iliamaslakov> понял, гадю [13:21:39] <andrew_b> Нет ещё, не запушил. [13:25:35] <andrew_b> Запушил. [13:26:08] <andrew_b> Немного историю подчистил. [13:26:56] <andrew_b> 114 char buffer1[BUF_TINY] = "\0"; /* port */ 115 char buffer2[BUF_1K] = "\0"; /* host */ 117 char buffer5[BUF_4K] = "\0"; /* session name */ 118 char buffer6[BUF_1K] = "\0"; /* user name */
Не дохрена места-то под недлинные строки? [13:49:57] <iliamaslakov> andrew_b, жадный, да? [13:50:14] <iliamaslakov> предложения? [13:50:27] <andrew_b> Уменьшить. [13:50:32] <iliamaslakov> 512? [13:50:36] <andrew_b> Очевидно же. [13:50:49] <iliamaslakov> уменьшить то очевидно [13:50:58] <iliamaslakov> вопрос на сколько [13:52:23] <andrew_b> Для порта вообще хватит 6. [13:59:10] <iliamaslakov> ок, жадина, будет 6 [14:00:34] <andrew_b> slavazanko/w:
@@ -3760,6 +3776,11 @@
/* Restore old right path */ - if (curdir[0] != '\0') - err = mc_chdir (curdir); + { + vfs_path_t *vpath = vfs_path_from_str (curdir); + + if (curdir[0] != '\0') + err = mc_chdir (vpath); + vfs_path_free (vpath); + }
return panel;
Предлагаю сначала проверять, потом vfs_path_from_str (curdir) делать. [14:06:16] <iliamaslakov> Центос 6 ставить? [14:11:16] <slavazanko/w> > Предлагаю сначала проверять принято, сделаю [14:22:39] <andrew_b> slavazanko/w: вроде ж меняли GList на какой-то GArray. 711 int 712 vfs_path_elements_count (const vfs_path_t * vpath) 713 { 714 return (vpath != NULL && vpath->path != NULL) ? g_list_length (vpath->path) : 0; 715 } 716
Слишком часто приходится по списку бегать, чтобы длину посчитать. [14:23:08] <slavazanko/w> не, не меняли, только разговоры были. Ок. сменю. [14:26:45] andrew_b вышел из конференции [14:45:54] <slavazanko/w> подпушил 55_filename_complete [15:11:15] <iliamaslakov> 1535 [16:46:33] andrew_b зашёл в конференцию [16:56:03] <andrew_b> Чё за срач в каментах?
modified: lib/fileloc.h modified: src/filemanager/cmd.c modified: src/vfs/sftpfs/Makefile.am modified: src/vfs/sftpfs/dialogs.c new file: src/vfs/sftpfs/hostlist.c new file: src/vfs/sftpfs/hostlist.h modified: src/vfs/sftpfs/sftpfs.c [17:08:49] monkey зашёл в конференцию [18:59:24] <andrew_b> slavazanko/w: slavazanko/h: осторожнее с 2695_expansion_to_src. Я там подкоммитил немного. Но это ещё не всё. Буду продолжать. [19:00:49] andrew_b вышел из конференции [20:04:47] monkey вышел из конференции [20:05:39] <iliamaslakov> >Чё за срач в каментах? так и знал что не понравится [20:32:14] slavazanko/h зашёл в конференцию [20:42:54] slavazanko/h вышел из конференции [20:48:15] slavazanko/h зашёл в конференцию [20:49:38] <slavazanko/h> > осторожнее с 2695_expansion_to_src. Я там подкоммитил немного. Но это ещё не всё. Буду продолжать. Ок, не вопрос. Я стараюсь не ребайзать бранчи, опка они на ревьюве. Дабы коммиты не терялись
