Bug 30266 - gear-create-tag не выводит имя созданного тэга
Summary: gear-create-tag не выводит имя созданного тэга
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gear (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-08-28 17:50 MSK by Anton Farygin
Modified: 2014-09-01 00:42 MSK (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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).