Bug 25117 - bad tmppath in .rpmmacros
Summary: bad tmppath in .rpmmacros
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: librpm (show other bugs)
Version: unstable
Hardware: all Linux
: P3 normal
Assignee: placeholder@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-18 14:41 MSK by Dmitry V. Levin
Modified: 2012-12-24 03:23 MSK (History)
10 users (show)

See Also:


Attachments

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