Bug 18739 - gear-update-tag(1) is misnamed
Summary: gear-update-tag(1) is misnamed
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: gear (show other bugs)
Version: unstable
Hardware: all Linux
: P2 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-04 10:08 MSK by Mikhail Gusarov
Modified: 2010-09-06 19:05 MSD (History)
8 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mikhail Gusarov 2009-02-04 10:08:40 MSK 
gear-create-tag и gear-update-tag постоянно путают из-за сходства названий.

Предлагаю переименовать gear-update-tag в gear-update-tags-file, и оставить старое имя для обратной совместимости, но нигде его не упоминать.

Патч с переименованием воспоследствует.
Comment 1 Dmitry V. Levin 2009-02-12 00:51:15 MSK 
gear-update-tag обновляет каталог .gear/tags/, поэтому имя gear-update-tags-file не подходит.

gear-update-gear-tags?
Comment 2 Mikhail Gusarov 2009-02-12 00:53:22 MSK 
А почему не подходит? "утилита gear, обновляющая файл с тэгами". То, что он называется .gear/tags, а не такой-нибудь .tags.gear - деталь реализации.
Comment 3 Alexey Gladkov 2009-02-12 01:30:31 MSK 
(В ответ на комментарий №1)
> gear-update-tag обновляет каталог .gear/tags/, поэтому имя
> gear-update-tags-file не подходит.
> 
> gear-update-gear-tags?

Очень уж длинно и содержит дублирование. Как насчёт gear-stored-tags? ... Ведь эта утилита не только обновляет теги, но ещё и удаляет, и проверяет их.
Comment 4 Mikhail Gusarov 2009-02-12 01:33:24 MSK 
gear-store-tags или gear-update-stored-tags тогда. gear-stored-tags выглядит как тул для выдачи состояния, а не изменения его.
Comment 5 Dmitry V. Levin 2009-02-12 01:36:59 MSK 
Из всего предложенного gear-update-stored-tags мне нравится больше всех.
Comment 6 Alexey Gladkov 2009-02-12 01:42:27 MSK 
(В ответ на комментарий №5)
> Из всего предложенного gear-update-stored-tags мне нравится больше всех.

"update" в названии портит tab-completion. Я хотел бы его убрать/заменить ещё и с этой целью.
Comment 7 Alexey Gladkov 2009-02-12 02:26:51 MSK 
(В ответ на комментарий №6)
> (В ответ на комментарий №5)
> > Из всего предложенного gear-update-stored-tags мне нравится больше всех.
> 
> "update" в названии портит tab-completion. Я хотел бы его убрать/заменить ещё и
> с этой целью.

Предлагаю gear-{manage,change}-tags или gear-named-tags. Последнее название навеяно описанием директивы tags в gear-rules(5).
Comment 8 Anton Farygin 2009-02-12 03:21:40 MSK 
gear-sync-tagfile ?
Comment 9 Alexey Gladkov 2009-02-12 12:42:29 MSK 
(В ответ на комментарий №8)
> gear-sync-tagfile ?

sync не совсем то т.к. эта утилита делает и проверку этого файла.
Может тогда gear-tagfile ?
Comment 10 Michael Shigorin 2009-07-21 21:58:45 MSD 
gear-tags?
Comment 11 Dmitry V. Levin 2010-08-29 16:28:19 MSD 
(In reply to comment #7)
> (В ответ на комментарий №6)
> > (В ответ на комментарий №5)
> > > Из всего предложенного gear-update-stored-tags мне нравится больше всех.
> > 
> > "update" в названии портит tab-completion. Я хотел бы его убрать/заменить ещё и
> > с этой целью.
> 
> Предлагаю gear-{manage,change}-tags или gear-named-tags. Последнее название
> навеяно описанием директивы tags в gear-rules(5).

.gear/tags может хранить как named tags (т.е. коммиты), так и полноценные tag objects, так что gear-named-tags не вполне в тему.

Видимо, всё же gear-store-tags это наиболее компромиссный вариант?
Comment 12 Alexey Gladkov 2010-08-29 16:35:54 MSD 
(В ответ на комментарий №11)
> Видимо, всё же gear-store-tags это наиболее компромиссный вариант?

Можно и так.
Comment 14 Repository Robot 2010-09-06 19:05:40 MSD 
gear-1.7.2.2-alt1 -> sisyphus:

* Mon Sep 06 2010 Dmitry V. Levin <ldv@altlinux> 1.7.2.2-alt1
- gear-import: pass --no-verify option to git-commit.
- Renamed gear-update-tag to gear-store-tags (closes: #18739).
- gear-command-{hasher,rpmbuild}: unset GIT_* environment
  variables used by gear itself (closes: #20884).
- tests: fixed false positive on non-tmpfs filesystems
  (closes: #24012).