Unix style for temporary directory environment variable(s) #8940

New Issue

gsr b3d · 2008-04-13T04:52:53+02:00

gsr b3d commented

2008-04-13 04:52:53 +02:00

%%%Change supported enviroment variables for temporary directory to spread
custom in Unix.

See man environ or mktemp for TMPDIR, or if there must be two, one with higher
level of importance, name it accordingly, thus BLENDERTMPDIR, see man groff,
cvs-pserver or mimep for cases of application specific temporary dirs (or
cscope, cvs or git-config for own editor vars, man or git for own pager,
etc).

As bonus, fix typo in help message.%%%

%%%Change supported enviroment variables for temporary directory to spread custom in Unix. See man environ or mktemp for TMPDIR, or if there must be two, one with higher level of importance, name it accordingly, thus BLENDERTMPDIR, see man groff, cvs-pserver or mimep for cases of application specific temporary dirs (or cscope, cvs or git-config for own editor vars, man or git for own pager, etc). As bonus, fix typo in help message.%%%

gsr b3d commented

2008-04-13 04:52:53 +02:00

Changed status to: 'Open'

Ton Roosendaal commented

2008-09-19 16:46:19 +02:00

%%%This is not so much a patch, but integration decision stuff for platform manager. Let's hear what Kent says.
Note that the patch disables "TMP", whilst the comment says it checks both.%%%

%%%This is not so much a patch, but integration decision stuff for platform manager. Let's hear what Kent says. Note that the patch disables "TMP", whilst the comment says it checks both.%%%

Ken Hughes commented

2008-09-19 19:54:28 +02:00

%%%The only thing I have to compare to is a Solaris system, but since this patch applies to non-Windows (actually, non-WIN32) systems, I googled man pages for FreeBSD and OSX. mktemp(1) refers to TMPDIR on all of those. My questions are why we would need to check separate variables, and why only on non-Windows machines? %%%

gsr b3d commented

2008-09-19 23:53:06 +02:00

%%%I did not add that functionality, I just tried to unixify
what there was already. I also checked different man pages,
and they even mention posix2001 (about TMPDIR) and checked
multiple man pages to confirm the syntax if an app uses own
var (about going with BLENDERTMPDIR).

TMPDIR should go in. But why there was two? No idea, I just
tried to go with the flow and assume one was app specific
to allow more fine control of temp files. In that case, it
would make sense to make it for all OSes and if set, be
prefered over TEMP|TMPDIR.

Probably then it would be wise to set a clear policy for
future and leave the issue "100% solved". For example first
valid dir of BLENDERTMPDIR, .B.blend tmp dir user pref,
TMPDIR|TEMP, some hardcoded value as last resort.%%%

%%%I did not add that functionality, I just tried to unixify what there was already. I also checked different man pages, and they even mention posix2001 (about TMPDIR) and checked multiple man pages to confirm the syntax if an app uses own var (about going with BLENDERTMPDIR). TMPDIR should go in. But why there was two? No idea, I just tried to go with the flow and assume one was app specific to allow more fine control of temp files. In that case, it would make sense to make it for all OSes and if set, be prefered over TEMP|TMPDIR. Probably then it would be wise to set a clear policy for future and leave the issue "100% solved". For example first valid dir of BLENDERTMPDIR, .B.blend tmp dir user pref, TMPDIR|TEMP, some hardcoded value as last resort.%%%

Bastien Montagne commented

2014-08-03 10:59:50 +02:00

Changed status from 'Open' to: 'Archived'

Bastien Montagne closed this issue

2014-08-03 10:59:50 +02:00

Bastien Montagne self-assigned this 2014-08-03 10:59:50 +02:00

Bastien Montagne commented

2014-08-03 10:59:50 +02:00

Thinks changed a lot since then, archiving.

Sign in to join this conversation.

No Label

Download

What's New

Blender Studio

Manual

Developers Blog

Documentation

Benchmark

Blender Conference

Development Fund

One-time Donations

Unix style for temporary directory environment variable(s) #8940