14751 2008-03-04 16:46:59 +0300 Лишняя информация в выводе crontab -l 2008-05-13 08:32:29 +0400 1 1 4 Development Sisyphus vixie-cron unstable all Linux CLOSED FIXED patch P2 minor --- 1 php-coder placeholder glebfm ldv placeholder vt qa-sisyphus oldest_to_newest 65035 0 php-coder 2008-03-04 16:47:00 +0300 coder@mvo ~ $ rpm -qf =crontab vixie-cron-4.1.20060426-alt3 coder@mvo ~ $ sudo crontab -l # DO NOT EDIT THIS FILE - edit the master and reinstall. # (/tmp/crontab.EUuzaf installed on Mon Mar 3 20:05:04 2008) # (Cron version V5.0 -- vixie-cron-4.1.20060426-alt3) #minute (0-59), #| hour (0-23), #| | day of the month (1-31), #| | | month of the year (1-12), #| | | | day of the week (0-6 with 0=Sunday). #| | | | | commands 0 21 3 3 1 apt-get install openoffice.org-langpack-ru -y Считаю, что первые две строки в выводе crontab -l абсолютно лишние и даже немного запутывают. Команда печатает на stdout и пользователь не может никакой файл редкатировать. Кроме того, в выводе команды также фигурирует путь к временному файлу (которого не существует кстати), который абсолютно не нужен для пользователя. 69094 1 php-coder 2008-04-29 09:39:27 +0400 При редактировании файла (crontab -e) этой шапки нет, потому что там есть такой код (файл vixie-cron/usr.sbin/cron/crontab.c, ф-ция edit_cmd()): 338 /* ignore the top few comments since we probably put them there. 339 */ 340 x = 0; 341 while (EOF != (ch = get_char(f))) { 342 if ('#' != ch) { 343 putc(ch, NewCrontab); 344 break; 345 } 346 while (EOF != (ch = get_char(f))) 347 if (ch == '\n') 348 break; 349 if (++x >= NHEADER_LINES) 350 break; 351 } Я предлагаю этот код вынести в отдельную ф-цию, после чего использовать её в edit_cmd() и list_cmd(). 69095 2 2596 php-coder 2008-04-29 10:17:03 +0400 Created attachment 2596 Предлагаемый патч. Вот патч. (Посмотрим, насколько он ускорит решение проблемы.) 69408 3 ldv 2008-05-06 17:31:03 +0400 Implemented in 4.1.20060426-alt4 69417 4 php-coder 2008-05-06 19:53:24 +0400 (In reply to comment #3) > Implemented in 4.1.20060426-alt4 Хорошо. Я уже успел отправить свой патч одному из разработчиков OpenBSD (ray@) и он ответил следующее: "I agree with this change, let me see if I can get it into the tree. Thanks!" Т.е., насколько я понял, он их приложит, только когда неизвестно. Возможно, нужно твой патч отправить ему тоже, если твой лучше моего. 69768 5 php-coder 2008-05-13 08:32:28 +0400 Патч (с некоторыми изменениями) был закоммичен в OpenBSD source tree: http://www.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/cron/crontab.c.diff?r1=1.55&r2=1.56 2596 2008-04-29 10:17:03 +0400 2008-04-29 10:17:03 +0400 Предлагаемый патч. 14751.patch text/plain 1774 php-coder ZGlmZiAtLWdpdCBhL3ZpeGllLWNyb24vdXNyLnNiaW4vY3Jvbi9jcm9udGFiLmMgYi92aXhpZS1j cm9uL3Vzci5zYmluL2Nyb24vY3JvbnRhYi5jCmluZGV4IDQwNjM4ZmMuLmMxYTMyNzIgMTAwNjQ0 Ci0tLSBhL3ZpeGllLWNyb24vdXNyLnNiaW4vY3Jvbi9jcm9udGFiLmMKKysrIGIvdml4aWUtY3Jv bi91c3Iuc2Jpbi9jcm9uL2Nyb250YWIuYwpAQCAtNTYsNyArNTYsOSBAQCBzdGF0aWMJdm9pZAkJ bGlzdF9jbWQodm9pZCksCiAJCQljaGVja19lcnJvcihjb25zdCBjaGFyICopLAogCQkJcGFyc2Vf YXJncyhpbnQgYywgY2hhciAqdltdKSwKIAkJCWRpZShpbnQpOwotc3RhdGljCWludAkJcmVwbGFj ZV9jbWQodm9pZCk7CitzdGF0aWMJaW50CQlyZXBsYWNlX2NtZCh2b2lkKSwKKwkJCQlpZ25vcmVf Y29tbWVudHMoRklMRSAqKTsKKwogCiBzdGF0aWMgdm9pZAogdXNhZ2UoY29uc3QgY2hhciAqbXNn KSB7CkBAIC0yNDcsNiArMjQ5LDExIEBAIGxpc3RfY21kKHZvaWQpIHsKIAkvKiBmaWxlIGlzIG9w ZW4uIGNvcHkgdG8gc3Rkb3V0LCBjbG9zZS4KIAkgKi8KIAlTZXRfTGluZU51bSgxKQorCisJLyog aWdub3JlIHRoZSB0b3AgZmV3IGNvbW1lbnRzIHNpbmNlIHdlIHByb2JhYmx5IHB1dCB0aGVtIHRo ZXJlLgorCSAqLworCWNoID0gaWdub3JlX2NvbW1lbnRzKGYpOworCiAJd2hpbGUgKEVPRiAhPSAo Y2ggPSBnZXRfY2hhcihmKSkpCiAJCXB1dGNoYXIoY2gpOwogCWZjbG9zZShmKTsKQEAgLTI4MSw3 ICsyODgsNyBAQCBzdGF0aWMgdm9pZAogZWRpdF9jbWQodm9pZCkgewogCWNoYXIgbltNQVhfRk5B TUVdLCBxW01BWF9URU1QU1RSXSwgKmVkaXRvcjsKIAlGSUxFICpmOwotCWludCBjaCwgdCwgeDsK KwlpbnQgY2gsIHQ7CiAJc3RydWN0IHN0YXQgc3RhdGJ1ZiwgeHN0YXRidWY7CiAJc3RydWN0IHRp bWVzcGVjIG10aW1lc3BlYzsKIAlzdHJ1Y3QgdGltZXZhbCB0dlsyXTsKQEAgLTMzNywxOCArMzQ0 LDcgQEAgZWRpdF9jbWQodm9pZCkgewogCiAJLyogaWdub3JlIHRoZSB0b3AgZmV3IGNvbW1lbnRz IHNpbmNlIHdlIHByb2JhYmx5IHB1dCB0aGVtIHRoZXJlLgogCSAqLwotCXggPSAwOwotCXdoaWxl IChFT0YgIT0gKGNoID0gZ2V0X2NoYXIoZikpKSB7Ci0JCWlmICgnIycgIT0gY2gpIHsKLQkJCXB1 dGMoY2gsIE5ld0Nyb250YWIpOwotCQkJYnJlYWs7Ci0JCX0KLQkJd2hpbGUgKEVPRiAhPSAoY2gg PSBnZXRfY2hhcihmKSkpCi0JCQlpZiAoY2ggPT0gJ1xuJykKLQkJCQlicmVhazsKLQkJaWYgKCsr eCA+PSBOSEVBREVSX0xJTkVTKQotCQkJYnJlYWs7Ci0JfQorCWNoID0gaWdub3JlX2NvbW1lbnRz KGYpOwogCiAJLyogY29weSB0aGUgcmVzdCBvZiB0aGUgY3JvbnRhYiAoaWYgYW55KSB0byB0aGUg dGVtcCBmaWxlLgogCSAqLwpAQCAtNjUxLDMgKzY0NywyNCBAQCBkaWUoaW50IHgpIHsKIAkJKHZv aWQpIHVubGluayhUZW1wRmlsZW5hbWUpOwogCV9leGl0KEVSUk9SX0VYSVQpOwogfQorCitzdGF0 aWMgaW50CitpZ25vcmVfY29tbWVudHMoRklMRSAqZikgeworCWludCBjaCwgeDsKKworCXggPSAw OworCXdoaWxlIChFT0YgIT0gKGNoID0gZ2V0X2NoYXIoZikpKSB7CisJCWlmICgnIycgIT0gY2gp IHsKKwkJCXB1dGMoY2gsIE5ld0Nyb250YWIpOworCQkJYnJlYWs7CisJCX0KKwkJd2hpbGUgKEVP RiAhPSAoY2ggPSBnZXRfY2hhcihmKSkpCisJCQlpZiAoY2ggPT0gJ1xuJykKKwkJCQlicmVhazsK KwkJaWYgKCsreCA+PSBOSEVBREVSX0xJTkVTKQorCQkJYnJlYWs7CisJfQorCisJcmV0dXJuIGNo OworfQorCg==