Summary: | gear-create-tag не выводит имя созданного тэга | ||
---|---|---|---|
Product: | Sisyphus | Reporter: | Anton Farygin <rider> |
Component: | gear | Assignee: | Dmitry V. Levin <ldv> |
Status: | CLOSED FIXED | QA Contact: | qa-sisyphus |
Severity: | normal | ||
Priority: | P3 | CC: | evg, glebfm, ldv, legion, placeholder, rider |
Version: | unstable | ||
Hardware: | all | ||
OS: | Linux |
Description
Anton Farygin
2014-08-28 17:50:06 MSK
Да. Сам про это думал. На выходных постараюсь сделать. Если Дима не против, то можете асайнить на меня. gear-create-tag -v недостаточно? Вывод: gear-create-tag: Creating tag named "TAG" using tag message "REPO VERSION" for commit COMMITISH но он это говорит до подписи. Если пароль вести неправильно эта строчка окажется сильно выше. Я использую gear-create-tag только с -v, потому что мне важно знать, что я подписываю. :) сделайте это по умолчанию, пожалуйста. ну и вывод после подписи, created tag <тэг> тоже было бы здорово. Мне не пришло в голову использовать "-v" т.к. обычно в этом режиме утилиты говорят ещё много всего всякого. Обоснование вывода имени тега без -v может быть такое: так как утилита вычисляет имя тега (не берём случай использования ключа -n) и создаёт его, то неплохо бы сказать это вычисленное имя. обоснование простое - было бы разумно сказать что ты сделал и получилось ли это сделать ;) Мне кажется оба сообщения полезны. Первое сообщает что мы будем подписывать, второе что мы подписали. Дим, первое сообщение выводится независимо от того получилось подписать или нет. Антон, насчёт успешности операции: есть код возврата утилиты :) (In reply to comment #8) > Мне кажется оба сообщения полезны. Первое сообщает что мы будем подписывать, > второе что мы подписали. Я не против второго сообщения, просто намекаю, что без первого сообщения подписывать несколько авантюрно получается. > Я не против второго сообщения, просто намекаю, что без первого сообщения
> подписывать несколько авантюрно получается.
Так давай сделаем оба сообщения обязательными.
код возврата - для ботов. а я человек и хочу получать вменяемую информацию ;) (In reply to comment #12) > Посмотрите http://git.altlinux.org/people/legion/packages/gear.git 1) -q и -v, наверное, не очень сочетаются; может быть, это надо диагностировать? 2) можно написать либо "Failed to create a tag", либо "Failed to create tag \"$tag_name\""; на мой взгляд, второй вариант немного лучше. (В ответ на комментарий №13) > (In reply to comment #12) > > Посмотрите http://git.altlinux.org/people/legion/packages/gear.git > > 1) -q и -v, наверное, не очень сочетаются; может быть, это надо > диагностировать? Скорее нужно привести поведение к поведению других утилит, у которых есть -q и -v. Они включают одно и выключают другое. > 2) можно написать либо "Failed to create a tag", либо "Failed to create tag > \"$tag_name\""; на мой взгляд, второй вариант немного лучше. Ок. Сейчас. Готово ack gear-2.0.4-alt1 -> sisyphus: * Sun Aug 31 2014 Dmitry V. Levin <ldv@altlinux> 2.0.4-alt1 - gear-update: added --strip-components option (by Alexey Gladkov). - gear-create-tag: + added --quiet option (by Alexey Gladkov); + unless in quiet mode, print messages about the tag being created both before and after the action (by Alexey Gladkov; closes: #30266). |