26887 2012-02-03 15:05:24 +0400 mkaptbox: возможность вынести aptbox за пределы workdir 2012-02-05 00:54:35 +0400 1 1 4 Development Sisyphus hasher unstable all Linux NEW P3 enhancement --- 1 kopilo4ka ldv at glebfm ldv placeholder qa-sisyphus oldest_to_newest 128686 0 kopilo4ka 2012-02-03 15:05:24 +0400 Первая часть патча - небольшой фикс захардкоденного места песочницы. Вторая часть - исправление ошибки http://lists.altlinux.org/pipermail/devel/2010-October/185347.html 128687 1 5314 kopilo4ka 2012-02-03 15:06:47 +0400 Created attachment 5314 patch 128691 2 ldv 2012-02-03 19:35:56 +0400 (In reply to comment #0) > Первая часть патча - небольшой фикс захардкоденного места песочницы. Скажите, пожалуйста, что, на ваш взгляд, дает эта замена "aptbox" на "$aptbox" в mkaptbox? > Вторая часть - исправление ошибки > http://lists.altlinux.org/pipermail/devel/2010-October/185347.html Что вы называете исправлением ошибки? Упрятывание вышеупомянутой диагностики вида "/usr/bin/mkaptbox: line 212: printf: ошибка записи: Обрыв канала" путем использования операции, которая не обучена диагностировать эту ошибку? 128693 3 kopilo4ka 2012-02-03 21:58:13 +0400 (In reply to comment #2) > (In reply to comment #0) > > Первая часть патча - небольшой фикс захардкоденного места песочницы. > > Скажите, пожалуйста, что, на ваш взгляд, дает эта замена "aptbox" на "$aptbox" > в mkaptbox? Если в файле hsh-sh-functions, в функции set_workdir() меняю aptbox="$workdir/aptbox" на другое, сразу все ломается. Если использовать переменную, то все хорошо. > > Вторая часть - исправление ошибки > > http://lists.altlinux.org/pipermail/devel/2010-October/185347.html > > Что вы называете исправлением ошибки? > Упрятывание вышеупомянутой диагностики вида > "/usr/bin/mkaptbox: line 212: printf: ошибка записи: Обрыв канала" > путем использования операции, которая не обучена диагностировать эту ошибку? Вывод --showrc не маленький и хранится в переменной. Если вызывать mkaptbox из другой программы, вероятно на это не хватает буфера. Я это обошел вычислением на лету. 128694 4 ldv 2012-02-03 22:11:22 +0400 (In reply to comment #3) > (In reply to comment #2) > > (In reply to comment #0) > > > Первая часть патча - небольшой фикс захардкоденного места песочницы. > > > > Скажите, пожалуйста, что, на ваш взгляд, дает эта замена "aptbox" на "$aptbox" > > в mkaptbox? > > Если в файле hsh-sh-functions, в функции set_workdir() меняю > aptbox="$workdir/aptbox" > на другое, сразу все ломается. Интересно, зачем вам понадобилось менять в файле hsh-sh-functions функцию set_workdir? > > > Вторая часть - исправление ошибки > > > http://lists.altlinux.org/pipermail/devel/2010-October/185347.html > > > > Что вы называете исправлением ошибки? > > Упрятывание вышеупомянутой диагностики вида > > "/usr/bin/mkaptbox: line 212: printf: ошибка записи: Обрыв канала" > > путем использования операции, которая не обучена диагностировать эту ошибку? > > Вывод --showrc не маленький и хранится в переменной. Если вызывать mkaptbox из > другой программы, вероятно на это не хватает буфера. Я это обошел вычислением > на лету. Да нет же, прочитайте еще раз разъяснение по той ссылке, которую вы привели раньше. 128702 5 kopilo4ka 2012-02-05 00:30:06 +0400 > Интересно, зачем вам понадобилось менять в файле hsh-sh-functions функцию > set_workdir? Неожиданно ~/hasher было уже занято, и захотелось перенести в другое место. > Да нет же, прочитайте еще раз разъяснение по той ссылке, которую вы привели > раньше. Тот 'sh-скрипт', что вызывает mkaptbox это питоний arepo. Ошибка появляется вообще случайно. Может не быть, а может и все 4 раза (столько там вызовов). 128703 6 ldv 2012-02-05 00:54:35 +0400 (In reply to comment #5) > > Интересно, зачем вам понадобилось менять в файле hsh-sh-functions функцию > > set_workdir? > > Неожиданно ~/hasher было уже занято, и захотелось перенести в другое место. ~/hasher это значение по умолчанию, но ведь никто не мешает переносить workdir в другое место: $ hsh --help |grep path-to-workdir Usage: hsh [options] [<path-to-workdir>] <package>... <path-to-workdir> must be valid writable directory. Даже значение по умолчанию можно поменять (за это отвечает переменная workdir в файле ~/.hasher/config). > > Да нет же, прочитайте еще раз разъяснение по той ссылке, которую вы привели > > раньше. > > Тот 'sh-скрипт', что вызывает mkaptbox это питоний arepo. Ошибка появляется > вообще случайно. Может не быть, а может и все 4 раза (столько там вызовов). Еще раз: та диагностика об ошибке, которая вас смущает, не является ошибкой. Ошибкой в некотором смысле является тот факт, что SIGPIPE игнорируется, но в shell-скрипте это поведение исправить нельзя. И arepo тут не при чем. Чтобы в этом убедиться, достаточно выполнить команду $ grep ^SigIgn: /proc/self/status 5314 2012-02-03 15:06:47 +0400 2012-02-03 15:06:47 +0400 patch mkaptbox.patch text/plain 1705 kopilo4ka LS0tIG1rYXB0Ym94Lm9yaWcJMjAxMS0xMC0zMCAyMjo1MzowMC4wMDAwMDAwMDAgKzA0MDAKKysr IG1rYXB0Ym94CTIwMTItMDItMDMgMTQ6NDM6NTEuODAxOTYyMzk5ICswNDAwCkBAIC0xMzMsMjIg KzEzMywyMiBAQAogcmVwbz0iJChyZWFkbGluayAtZnYgLS0gIiRyZXBvIikiCiAKICMgZG9lcyBh cHRib3ggZXhpc3QgeWV0PwotaWYgWyAtZSBhcHRib3ggLW8gLUwgYXB0Ym94IF07IHRoZW4KK2lm IFsgLWUgJGFwdGJveCAtbyAtTCAkYXB0Ym94IF07IHRoZW4KIAlbIC1uICIkZm9yY2UiIF0gfHwg ZmF0YWwgInJlbW92ZSAkYXB0Ym94IGZpcnN0LiIKLQlbIC1kIGFwdGJveCBdIHx8IGZhdGFsICIk YXB0Ym94OiBpbnZhbGlkIHBhdGhuYW1lLiIKKwlbIC1kICRhcHRib3ggXSB8fCBmYXRhbCAiJGFw dGJveDogaW52YWxpZCBwYXRobmFtZS4iCiAKIAkjIGVtcHR5IGRpcj8KLQlpZiBybWRpciAtLSBh cHRib3ggMj4vZGV2L251bGw7IHRoZW4KLQkJbWtkaXIgLW03MDAgJHZlcmJvc2UgLS0gYXB0Ym94 ID4mMgotCQljZCBhcHRib3gKKwlpZiBybWRpciAtLSAkYXB0Ym94IDI+L2Rldi9udWxsOyB0aGVu CisJCW1rZGlyIC1tNzAwICR2ZXJib3NlIC0tICRhcHRib3ggPiYyCisJCWNkICRhcHRib3gKIAll bHNlCi0JCWNkIGFwdGJveAorCQljZCAkYXB0Ym94CiAJCVsgLWQgLi9ldGMvYXB0IC1hIC1kIC4v dmFyL2xpYi9ycG0gXSB8fAogCQkJZmF0YWwgIiRhcHRib3g6IGRvZXNuJ3QgbG9vayB2YWxpZC4i CiAJZmkKIGVsc2UKLQlta2RpciAtbTcwMCAkdmVyYm9zZSAtLSBhcHRib3ggPiYyCi0JY2QgYXB0 Ym94CisJbWtkaXIgLW03MDAgJHZlcmJvc2UgLS0gJGFwdGJveCA+JjIKKwljZCAkYXB0Ym94CiBm aQogCiB2ZXJib3NlICJDaGFuZ2VkIHdvcmtpbmcgZGlyZWN0b3J5IHRvIFxgJGFwdGJveCcuIgpA QCAtMjAxLDE1ICsyMDEsMTUgQEAKIAogIyBjcmVhdGUgYXB0Ym94IHZlcnNpb24gb2Ygfi8ucnBt cmMgaWYgbmVjZXNzYXJ5LgogYXB0Ym94X2hvbWU9Ci1ycG1fc2hvd3JjPSIkKHJwbSAtLXNob3dy YykiCi1jdXJyZW50X2FyY2g9IiQocHJpbnRmICVzICIkcnBtX3Nob3dyYyIgfHNlZCAtbmUgJ3Mv Xmluc3RhbGwgYXJjaFtbOnNwYWNlOl1dKjpbWzpzcGFjZTpdXSpcKFteWzpzcGFjZTpdXVwrXCku Ki9cMS9wJykiCitycG1fc2hvd3JjPSJycG0gLS1zaG93cmMiCitjdXJyZW50X2FyY2g9IiQoZXZh bCAiJHJwbV9zaG93cmMiIHxzZWQgLW5lICdzL15pbnN0YWxsIGFyY2hbWzpzcGFjZTpdXSo6W1s6 c3BhY2U6XV0qXChbXls6c3BhY2U6XV1cK1wpLiovXDEvcCcpIgogWyAtbiAiJGN1cnJlbnRfYXJj aCIgXSB8fAogCWN1cnJlbnRfYXJjaD0iJCh1bmFtZSAtbSkiCiBbICIkY3VycmVudF9hcmNoIiAh PSAiJHt0YXJnZXQ6LSRkZWZfdGFyZ2V0fSIgXSAmJgogCWFsdGVybmF0ZV9hcmNoPTEgfHwKIAlh bHRlcm5hdGVfYXJjaD0KIGlmIFsgLW4gIiRhbHRlcm5hdGVfYXJjaCIgXSAmJgotICAgISBwcmlu dGYgJXMgIiRycG1fc2hvd3JjIiB8CisgICAhIGV2YWwgIiRycG1fc2hvd3JjIiB8CiAgICAgIExD X0FMTD1DIGdyZXAgLXFzICJeY29tcGF0aWJsZSBhcmNoc1tbOnNwYWNlOl1dKjouKlw8JHt0YXJn ZXQ6LSRkZWZfdGFyZ2V0fVw+IjsgdGhlbgogCWFwdGJveF9ob21lPSIkYXB0Ym94L2hvbWUvdXNl ciIKIAlmb3IgZiBpbiAiJEhPTUUvLnJwbXJjIiAiJEhPTUUvLnJwbW1hY3JvcyI7IGRvCg==