Bug 15442 - gear-update не работает если есть пустые каталоги
Summary: gear-update не работает если есть пустые каталоги
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: gear (show other bugs)
Version: unstable
Hardware: all Linux
: P2 major
Assignee: Alexey Gladkov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-04-23 22:54 MSD by swi
Modified: 2008-04-25 18:07 MSD (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description swi 2008-04-23 22:54:35 MSD
если в git репо присутствуют пустые каталоги (git их вообще не видит получается)
то gear-update не срабатывает жалуясь на то что якобы присутствуют
модифицированные или неконтролируемые git файлы. Удаление этих каталогов решает
проблему, но в tar-ball они могут присутствовать (ошметки, CVS).
Comment 1 Alexey Gladkov 2008-04-23 23:10:47 MSD
Эту проблему можно решить через --exclude или через .gitignore.

git не поддерживает пустые каталоги. Пустые каталоги это проблема, но не
проблема gear. В общем случае добавить решить эту проблему нельзя т.к. если
добавить в пустой каталог нечто, то это может создать ошибку.

Поэтому в этих случаях мантейнер должен сам решать эту проблему и решений может
быть несколько.

Так что это не бага, это фича.
Comment 2 swi 2008-04-23 23:28:15 MSD
(In reply to comment #1)
> git не поддерживает пустые каталоги. Пустые каталоги это проблема, но не
> проблема gear. В общем случае добавить решить эту проблему нельзя т.к. если
> добавить в пустой каталог нечто, то это может создать ошибку.
А может алгоритм проверки? Ведь git этих каталогов не видит не обрабатывает...
чего ж тогда gear на них реагирует? Может тоже пусть игнорирует?

Comment 3 Alexey Gladkov 2008-04-24 00:35:43 MSD
gear-update ругается на модифицированные файлы только перед началом работы. Эта
проверка может быть отключена через --force. Эта проверка защищиает от ошибок,
когда обновляются исходники в рабочем каталоге с незакоммиченными данными.
Конечно можно в эту проверку добавить --no-empty-directory, но при наличии
--force я не вижу в этом смысла.
Comment 4 Andrey Rahmatullin 2008-04-24 08:44:06 MSD
(In reply to comment #3)
> gear-update ругается на модифицированные файлы только перед началом работы. 
Эта
> проверка может быть отключена через --force. Эта проверка защищиает от ошибок,
> когда обновляются исходники в рабочем каталоге с незакоммиченными данными.
Ещё раз: поскольку наш любимый git не считает и не может считать пустые 
каталоги "незакоммиченными данными", как и вообще данными, то почему они должны 
препятствовать обновлению?
Comment 5 Slava Semushin 2008-04-24 08:49:33 MSD
Я вешал когда-то bug #13583 об этом же.

Не завидую судьбе этого бага -- он или INVALID или DUPLICATE..
Comment 6 Alexey Gladkov 2008-04-24 14:00:38 MSD
(In reply to comment #4)
> Ещё раз: поскольку наш любимый git не считает и не может считать пустые 
> каталоги "незакоммиченными данными", как и вообще данными, то почему они должны 
> препятствовать обновлению?

Потому что эти директории могут быть важны. То что эти директории не могут быть
добавлены это ещё не повод для того чтобы их игнорировать. gear-update не делает
предположений о назначении изменений в рабочем каталоге. Повторю комментарий из
bug #13583#c5:

У gear-update есть ключ --force на случай когда вы уверены что обновление не
удалит ничего важного или не закоммиченные данные вам не нужны.
Comment 7 swi 2008-04-24 14:02:49 MSD
(In reply to comment #6)
> Потому что эти директории могут быть важны. То что эти директории не могут быть
> добавлены это ещё не повод для того чтобы их игнорировать.
Поэтому чтобы сделать gear-update приходится ух удалять или вносить в ignore -
т.е. игнорировать. Я чего-то недопонимаю в логике
Comment 8 Alexey Gladkov 2008-04-24 14:23:36 MSD
(In reply to comment #7)
> Поэтому чтобы сделать gear-update приходится ух удалять или вносить в ignore -
> т.е. игнорировать. Я чего-то недопонимаю в логике

Неправильно :)

Поэтому если вам нужно обновить директорию с незакоммиченными данными, вам нужно
выполнять gear-update c ключём --force. В этом случае gear-update обновит
директорию и удалит эти данные.
Comment 9 Alexey Gladkov 2008-04-24 16:11:49 MSD
Вообщем, это документированное поведение и есть документированное решение:

$ gear-update --help |grep -A1 -- --force
  -f,--force          remove files from <subdirectory> even
                      if untracked or modified files found;

Также есть другие решения этой же проблемы.

Resolution => INVALID
Comment 10 Dmitry V. Levin 2008-04-24 16:21:26 MSD
(In reply to comment #1)
> Эту проблему можно решить через --exclude или через .gitignore.
> 
> git не поддерживает пустые каталоги. Пустые каталоги это проблема, но не
> проблема gear. В общем случае добавить решить эту проблему нельзя т.к. если
> добавить в пустой каталог нечто, то это может создать ошибку.

А, может быть, просто удалять пустые каталоги, не спрашивая пользователя, нужны
они ему или нет?

find -type d -empty -delete
Comment 11 Alexey Gladkov 2008-04-24 16:31:46 MSD
А ты можешь с уверенностью сказать что этот каталог в <subdirectory> не был
сделан с каким-то смыслом ?

Тогда уж можно проще сделать: добавить к git-ls-files ключ --no-empty-directory.
Всё равно в этом каталоге все данные будут потеряны и незачем удалять что-либо
до полного удаления.

Но мне бы очень не хотелось удалять _любые_ несохранёные данные в <subdirectory>
без явного согласия пользователя.
Comment 12 Dmitry V. Levin 2008-04-24 16:34:37 MSD
(In reply to comment #11)
> А ты можешь с уверенностью сказать что этот каталог в <subdirectory> не был
> сделан с каким-то смыслом ?

Как правило, да, но в gear-srpmimport я сделал в точности наоборот. :)
Вот упрощённая цитата оттуда:

# Add empty .gitignore file to each empty directory.
find -type d -empty -exec touch -- '{}/.gitignore' ';'
Comment 13 Alexey Gladkov 2008-04-24 16:45:51 MSD
(In reply to comment #12)
> # Add empty .gitignore file to each empty directory.
> find -type d -empty -exec touch -- '{}/.gitignore' ';'

Такой подход также опасен как и удаление пустых каталогов. Так как в случае
gear-update речь идёт об удалении данных, а не об их добавлении в репозиторий,
то мне кажется, что лучшим решением будет дать выбрать пользователю нужны ли ему
эти данные или нет. Тем более эта проверка работает быстро.

Кстати, ты поднял интересный вопрос: когда gear-update обновляет каталог из
архива, то в архиве запросто могут быть пустые каталоги, которые не попадут в
репозиторий. И тут я согласен, что стоит делать как gear-srpmimport. Но эта бага
уже другая история.