25686 2011-05-30 13:42:40 +0400 hsh --init заполняет кэш, прямо не говоря об этом 2011-09-02 10:45:47 +0400 1 1 4 Development Sisyphus hasher unstable all Linux CLOSED FIXED usability P3 enhancement --- 26217 1 ivk1097 ldv at dd1email glebfm ldv legion mike placeholder radik qa-sisyphus oldest_to_newest 121759 0 ivk1097 2011-05-30 13:42:40 +0400 Когда hsh --init заполняет ~/hasher/aptbox/var/cache/apt/archives, то желательно, чтобы он прямо говорил в логе, что качает пакеты. Если хашерницу собирает начинающий, причём пакеты для неё берутся из инета, и канал тощий (а такое всё-таки бывает!), то "зависание" hsh --init на не очень понятной строке hsh-initroot: Generated initial package file list. может сильно озадачить человека ;) 121788 1 radik 2011-05-31 15:47:55 +0400 Полностью согласен с топикстартером. По началу очень озадачивало, пока не спросил у ребят... 121798 2 mike 2011-06-01 00:08:30 +0400 Думаю, самым простым способом будет что-то вроде echo "Obtaining packages (downloading remotely might take a while)..." перед тем, как. 121799 3 ldv 2011-06-01 00:29:10 +0400 Дело в том, что hasher, запуская apt-get -q -y -o RPM::PM=external -o Dir::Bin::rpm=hsh-rpmi-print-files (см. print_uris() в hsh-sh-functions) не знает, что именно делает apt-get, в частности, скачивает ли он файлы. Например, в случае использования метода file точно ничего не скачивается. Даже не знаю, чем вам помочь. 121800 4 mike 2011-06-01 00:51:14 +0400 Это понятно, потому и такую формулировку предложил. Ну или так: Obtaining packages (might take a while if downloading remotely)... 121801 5 ldv 2011-06-01 01:08:19 +0400 (In reply to comment #4) > Это понятно, потому и такую формулировку предложил. Ну или так: > > Obtaining packages (might take a while if downloading remotely)... 1.3.20-alt1-2-g88b0822 121838 6 4959 dd1email 2011-06-02 09:45:20 +0400 Created attachment 4959 Patch using tee(1) Предлагается иной способ обхода с помощью tee(1) - см. прикладываемый патч. 121839 7 ldv 2011-06-02 09:48:26 +0400 (In reply to comment #6) > Created an attachment (id=4959) [details] > Patch using tee(1) > > Предлагается иной способ обхода с помощью tee(1) - см. прикладываемый патч. Вариант с tee в принципе плох тем, что статус завершения apt-get безвозвратно теряется. 121841 8 4960 dd1email 2011-06-02 10:00:39 +0400 Created attachment 4960 Patch using tee(1) Поправка: чтобы if работал правильно, нужно временно установить pipefail. 121842 9 legion 2011-06-02 10:15:32 +0400 (В ответ на комментарий №8) > Created an attachment (id=4960) [details] > Patch using tee(1) + if ! out="$("$aptbox/apt-get" $options install -- "$@" 2>&1 | tee /dev/stderr)"; then + set +o pipefail Как Дима и сказал, внутрь if вы никогда не попадёте. tee будет завершаться успешно и код возврата apt будет потерян. 121844 10 dd1email 2011-06-02 10:26:45 +0400 $ if ! v=$(false | tee /dev/stderr); then echo a; fi $ set -o pipefail; if ! v=$(false | tee /dev/stderr); then echo a; fi; set +o pipefail a 121845 11 ldv 2011-06-02 10:27:59 +0400 (In reply to comment #9) > (В ответ на комментарий №8) > > Created an attachment (id=4960) [details] [details] > > Patch using tee(1) > > + if ! out="$("$aptbox/apt-get" $options install -- "$@" 2>&1 | tee > /dev/stderr)"; then > + set +o pipefail > > Как Дима и сказал, внутрь if вы никогда не попадёте. tee будет завершаться > успешно и код возврата apt будет потерян. Идея понятна, имелось в виду $(set +o pipefail; apt-get | tee). Честно говоря, я бы, наверное, не очень бы хотел, чтобы _весь_ вывод apt-get транслировался бы пользователю даже в режиме verbose. А в обычном режиме пользователю вообще никакого вывода от apt-get не нужно. 121846 12 legion 2011-06-02 10:33:07 +0400 (В ответ на комментарий №10) > $ if ! v=$(false | tee /dev/stderr); then echo a; fi > $ set -o pipefail; if ! v=$(false | tee /dev/stderr); then echo a; fi; set +o > pipefail > a Прошу прощения, я не заметил этот милый башизм :) 121848 13 dd1email 2011-06-02 10:45:55 +0400 Устанавливает опцию именно -o, а не +o: $ set -o pipefail $ set -o | grep pipefail pipefail on $ set +o pipefail $ set -o | grep pipefail pipefail off Вообще, мне кажется, что данный баг именно о нужности индикатора состояния (progress bar или чего-нибудь похожего). В конце концов, вывод apt-get можно и порезать (grep). Использования башизмов не постеснялся из-за: http://lists.altlinux.org/pipermail/devel/2010-August/183728.html http://lists.altlinux.org/pipermail/devel/2010-August/183729.html =) 121850 14 ldv 2011-06-02 11:03:13 +0400 (In reply to comment #11) > (In reply to comment #9) > > (В ответ на комментарий №8) > > > Created an attachment (id=4960) [details] [details] [details] > > > Patch using tee(1) > > > > + if ! out="$("$aptbox/apt-get" $options install -- "$@" 2>&1 | tee > > /dev/stderr)"; then > > + set +o pipefail > > > > Как Дима и сказал, внутрь if вы никогда не попадёте. tee будет завершаться > > успешно и код возврата apt будет потерян. > > Идея понятна, имелось в виду $(set -o pipefail; apt-get | tee). > Честно говоря, я бы, наверное, не очень бы хотел, чтобы _весь_ вывод apt-get > транслировался бы пользователю даже в режиме verbose. А в обычном режиме > пользователю вообще никакого вывода от apt-get не нужно. Попробуйте 1.3.20-alt1-3-gca1aab7; не кажется ли вам, что режим -v в результате этого изменения стал чрезмерно болтлив? 122082 15 ldv 2011-06-10 19:35:03 +0400 (In reply to comment #14) > Попробуйте 1.3.20-alt1-3-gca1aab7; не кажется ли вам, что режим -v в результате > этого изменения стал чрезмерно болтлив? Кто-нибудь уже попробовал? 122090 16 mike 2011-06-11 02:00:56 +0400 Не-а, но кажется, что одной строчки достаточно (особенно если при загрузке она как раз и болтается последней). 123452 17 repository-robot 2011-07-27 01:10:20 +0400 hasher-1.3.21-alt1 -> sisyphus: * Wed Jul 27 2011 Dmitry V. Levin <ldv@altlinux> 1.3.21-alt1 - hsh-run: disable network isolation when X11 forwarding is requested. - hsh: added --printenv option (closes: #23955) - hsh-sh-functions.in (print_uris): in verbose mode, print a message about obtaining packages (closes: #25686). 4959 2011-06-02 09:45:20 +0400 2011-06-02 10:00:39 +0400 Patch using tee(1) tee.patch text/plain 762 dd1email ZGlmZiAtLWdpdCBhL2hhc2hlci9oc2gtc2gtZnVuY3Rpb25zLmluIGIvaGFzaGVyL2hzaC1zaC1m dW5jdGlvbnMuaW4KaW5kZXggNDYwZmM4OS4uODU0Y2QxZCAxMDA2NDQKLS0tIGEvaGFzaGVyL2hz aC1zaC1mdW5jdGlvbnMuaW4KKysrIGIvaGFzaGVyL2hzaC1zaC1mdW5jdGlvbnMuaW4KQEAgLTYy OCwxMyArNjI4LDEyIEBAIHByaW50X3VyaXMoKQogewogCWxvY2FsIG91dAogCWxvY2FsIG9wdGlv bnM9Jy1xIC15IC1vIFJQTTo6UE09ZXh0ZXJuYWwgLW8gRGlyOjpCaW46OnJwbT1oc2gtcnBtaS1w cmludC1maWxlcycKLQlpZiAhIG91dD0iJCgiJGFwdGJveC9hcHQtZ2V0IiAkb3B0aW9ucyBpbnN0 YWxsIC0tICIkQCIgMj4mMSkiOyB0aGVuCi0JCXByaW50ZiAlc1xcbiAiJG91dCIgPiYyCi0JCWZh dGFsICdmYWlsZWQgdG8gY2FsY3VsYXRlIHBhY2thZ2UgZmlsZSBsaXN0LicKKwlpZiAhIG91dD0i JCgiJGFwdGJveC9hcHQtZ2V0IiAkb3B0aW9ucyBpbnN0YWxsIC0tICIkQCIgMj4mMSB8IHRlZSAv ZGV2L3N0ZGVycikiOyB0aGVuCisJCWZhdGFsICdGYWlsZWQgdG8gY2FsY3VsYXRlIHBhY2thZ2Ug ZmlsZSBsaXN0LicKIAlmaQogCiAJcHJpbnRmICVzICIkb3V0IiB8TENfQUxMPUMgZ3JlcCAnXi8u KlwucnBtJCcgfHwgWyAkPyAtZXEgMSBdCi0JdmVyYm9zZSAnY2FsY3VsYXRlZCBwYWNrYWdlIGZp bGUgbGlzdC4nCisJdmVyYm9zZSAnQ2FsY3VsYXRlZCBwYWNrYWdlIGZpbGUgbGlzdC4nCiB9CiAK IHBhcnNlX3hhdXRoX2VudHJ5KCkK 4960 2011-06-02 10:00:39 +0400 2011-06-02 10:00:39 +0400 Patch using tee(1) tee.patch text/plain 799 dd1email ZGlmZiAtLWdpdCBhL2hhc2hlci9oc2gtc2gtZnVuY3Rpb25zLmluIGIvaGFzaGVyL2hzaC1zaC1m dW5jdGlvbnMuaW4KaW5kZXggNDYwZmM4OS4uMzY0NzZhZCAxMDA2NDQKLS0tIGEvaGFzaGVyL2hz aC1zaC1mdW5jdGlvbnMuaW4KKysrIGIvaGFzaGVyL2hzaC1zaC1mdW5jdGlvbnMuaW4KQEAgLTYy OCwxMyArNjI4LDE0IEBAIHByaW50X3VyaXMoKQogewogCWxvY2FsIG91dAogCWxvY2FsIG9wdGlv bnM9Jy1xIC15IC1vIFJQTTo6UE09ZXh0ZXJuYWwgLW8gRGlyOjpCaW46OnJwbT1oc2gtcnBtaS1w cmludC1maWxlcycKLQlpZiAhIG91dD0iJCgiJGFwdGJveC9hcHQtZ2V0IiAkb3B0aW9ucyBpbnN0 YWxsIC0tICIkQCIgMj4mMSkiOyB0aGVuCi0JCXByaW50ZiAlc1xcbiAiJG91dCIgPiYyCi0JCWZh dGFsICdmYWlsZWQgdG8gY2FsY3VsYXRlIHBhY2thZ2UgZmlsZSBsaXN0LicKKwlzZXQgLW8gcGlw ZWZhaWwKKwlpZiAhIG91dD0iJCgiJGFwdGJveC9hcHQtZ2V0IiAkb3B0aW9ucyBpbnN0YWxsIC0t ICIkQCIgMj4mMSB8IHRlZSAvZGV2L3N0ZGVycikiOyB0aGVuCisJCXNldCArbyBwaXBlZmFpbAor CQlmYXRhbCAnRmFpbGVkIHRvIGNhbGN1bGF0ZSBwYWNrYWdlIGZpbGUgbGlzdC4nCiAJZmkKIAog CXByaW50ZiAlcyAiJG91dCIgfExDX0FMTD1DIGdyZXAgJ14vLipcLnJwbSQnIHx8IFsgJD8gLWVx IDEgXQotCXZlcmJvc2UgJ2NhbGN1bGF0ZWQgcGFja2FnZSBmaWxlIGxpc3QuJworCXZlcmJvc2Ug J0NhbGN1bGF0ZWQgcGFja2FnZSBmaWxlIGxpc3QuJwogfQogCiBwYXJzZV94YXV0aF9lbnRyeSgp Cg==