Bug 30266

Summary: gear-create-tag не выводит имя созданного тэга
Product: Sisyphus Reporter: Anton Farygin <rider>
Component: gearAssignee: 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 выводил имя созданного тэга.
Для копи-паста было бы удобно.
Comment 1 Alexey Gladkov 2014-08-28 17:58:08 MSK
Да. Сам про это думал. На выходных постараюсь сделать. Если Дима не против, то можете асайнить на меня.
Comment 2 Dmitry V. Levin 2014-08-28 17:59:48 MSK
gear-create-tag -v недостаточно?
Comment 3 Alexey Gladkov 2014-08-28 18:05:45 MSK
Вывод:

gear-create-tag: Creating tag named "TAG" using tag message "REPO VERSION" for commit COMMITISH

но он это говорит до подписи. Если пароль вести неправильно эта строчка окажется сильно выше.
Comment 4 Dmitry V. Levin 2014-08-28 18:08:04 MSK
Я использую gear-create-tag только с -v, потому что мне важно знать, что я подписываю. :)
Comment 5 Anton Farygin 2014-08-28 18:16:39 MSK
сделайте это по умолчанию, пожалуйста. 

ну и вывод после подписи, created tag <тэг> тоже было бы здорово.
Comment 6 Alexey Gladkov 2014-08-28 18:18:23 MSK
Мне не пришло в голову использовать "-v" т.к. обычно в этом режиме утилиты говорят ещё много всего всякого.

Обоснование вывода имени тега без -v может быть такое: так как утилита вычисляет имя тега (не берём случай использования ключа -n) и создаёт его, то неплохо бы сказать это вычисленное имя.
Comment 7 Anton Farygin 2014-08-28 18:21:21 MSK
обоснование простое - было бы разумно сказать что ты сделал и получилось ли это сделать ;)
Comment 8 Alexey Gladkov 2014-08-28 18:29:15 MSK
Мне кажется оба сообщения полезны. Первое сообщает что мы будем подписывать, второе что мы подписали. Дим, первое сообщение выводится независимо от того получилось подписать или нет.

Антон, насчёт успешности операции: есть код возврата утилиты :)
Comment 9 Dmitry V. Levin 2014-08-28 18:34:03 MSK
(In reply to comment #8)
> Мне кажется оба сообщения полезны. Первое сообщает что мы будем подписывать,
> второе что мы подписали.

Я не против второго сообщения, просто намекаю, что без первого сообщения подписывать несколько авантюрно получается.
Comment 10 Alexey Gladkov 2014-08-28 18:40:54 MSK
> Я не против второго сообщения, просто намекаю, что без первого сообщения
> подписывать несколько авантюрно получается.

Так давай сделаем оба сообщения обязательными.
Comment 11 Anton Farygin 2014-08-28 18:42:59 MSK
код возврата - для ботов.
а я человек и хочу получать вменяемую информацию ;)
Comment 12 Alexey Gladkov 2014-08-28 19:14:15 MSK
Посмотрите http://git.altlinux.org/people/legion/packages/gear.git
Comment 13 Dmitry V. Levin 2014-08-28 19:25:23 MSK
(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\""; на мой взгляд, второй вариант немного лучше.
Comment 14 Alexey Gladkov 2014-08-28 19:32:04 MSK
(В ответ на комментарий №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\""; на мой взгляд, второй вариант немного лучше.

Ок. Сейчас.
Comment 15 Alexey Gladkov 2014-08-28 19:43:00 MSK
Готово
Comment 16 Dmitry V. Levin 2014-08-28 19:50:35 MSK
ack
Comment 17 Repository Robot 2014-09-01 00:42:55 MSK
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).