Bug 20884 - gear-command-* should clear environment
Summary: gear-command-* should clear environment
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: 2009-07-27 23:51 MSD by Sir Raorn
Modified: 2010-09-06 19:05 MSD (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Sir Raorn 2009-07-27 23:51:40 MSD
Есть проект, который использует вызовы git --git-dir ... во время выполнения %build (когда запускает свои тесты).  Поскольку gear экспортирует переменные GIT_DIR и/или GIT_OBJECT_DIRECTORY, эти тесты обламываются во время выполнения gear-rpm и/или gear-buildreq.  Было бы неплохо unset'ить эта переменные на время выполнения --command.
Comment 1 Alexey Gladkov 2009-07-28 16:52:15 MSD
Это в любом случае правильно. Не нужно чтобы во время сборки торчали куски gear.
Comment 2 Dmitry V. Levin 2009-07-28 22:02:31 MSD
(In reply to comment #1)
> Это в любом случае правильно. Не нужно чтобы во время сборки торчали куски
> gear.

И не только во время сборки.  Внутренняя кухня не должна высовываться наружу без необходимости.
Comment 3 Andrey Rahmatullin 2009-07-28 22:05:43 MSD
Дадада, меня ужасает текущая ситуация.
Comment 4 Alexey Gladkov 2009-07-28 22:19:56 MSD
(В ответ на комментарий №3)
> Дадада, меня ужасает текущая ситуация.

Ну не нужно сгущать краски ...
Comment 5 Andrey Rahmatullin 2010-07-13 19:23:42 MSD
Сейчас (gear >= 1.7.1-alt1, видимо) надо сбрасывать ещё и GIT_INDEX_FILE.
Comment 7 Repository Robot 2010-09-06 19:05:39 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).