Bug 25117

Summary: bad tmppath in .rpmmacros
Product: Sisyphus Reporter: Dmitry V. Levin <ldv>
Component: librpmAssignee: placeholder <placeholder>
Status: CLOSED FIXED QA Contact: qa-sisyphus
Severity: normal    
Priority: P3 CC: aen, arseny, glebfm, imz, inger, ldv, mike, placeholder, radik, vt
Version: unstable   
Hardware: all   
OS: Linux   

Description Dmitry V. Levin 2011-02-18 14:41:23 MSK
+++ This bug was initially created as a clone of Bug #25116 +++

Хорошо бы переопределить %_tmppath с нынешнего /var/tmp в ${TMPDIR:-/var/tmp}.
Comment 1 Dmitry V. Levin 2011-02-18 14:50:14 MSK
Можно реализовать встроенный макрос %__tmpdir со значением ${TMPDIR:-/var/tmp} и определить по умолчанию %_tmppath в этот %__tmpdir.  На установку пакетов это повлиять не должно, поскольку в /root/.rpmmacros определен свой %_tmppath.
Comment 2 at@altlinux.org 2011-02-19 03:58:00 MSK
Я думаю во многих случаях %_tmppath как сущность вообще не нужна, это overgeneralization.  Как и понятие URL в rpmio.  Думаю в сишном коде можно забивать на %_tmppath и использовать getenv/mkstemp (как я уже сделал недавно в build/files.c).  %_tmppath есть смысл использовать только для чего-то типа %buildroot.  И да, тут лучше подходит $TMPDIR, чем /var/tmp.

Я видел что недавно где-то реализовали встроенный макрос getenv, наверное это лучше чем ad hoc макрос для TMPDIR.
Comment 3 Dmitry V. Levin 2011-02-19 04:10:00 MSK
(In reply to comment #2)
> Я видел что недавно где-то реализовали встроенный макрос getenv, наверное это
> лучше чем ad hoc макрос для TMPDIR.

%getenv это здорово, но как на нем реализовать ${TMPDIR:-/var/tmp}?
Comment 4 at@altlinux.org 2011-02-19 04:11:39 MSK
Можно сделать второй аргумент - значение по умолчанию, будет
%{getenv TMPDIR /tmp}
Comment 5 Repository Robot 2012-12-24 03:23:19 MSK
rpm-4.0.4-alt100.58 -> sisyphus:

* Mon Dec 24 2012 Dmitry V. Levin <ldv@altlinux> 4.0.4-alt100.58
- rpmrc.in: changed to use -mtune=generic for all x86 flavours.
- pkgconfig.req.files: changed to ignore file type and treat
  all non-symlinks the same way.
- Added %getenv builtin macro.
- Added %_tmpdir builtin macro,
  changed default %_tmppath value to %_tmpdir (closes: #25117).