search for: system_temp_directori

Hi, all. I noticed recently that llvm::sys::fs::createTemporaryFile does not use llvm::sys::path::system_temp_directory, instead relying on its platform-specific helper TempFile. Is there any reason for this disparity? The two implementations are not in sync, either: - TempDir honors TMPDIR, TMP, TEMP, TEMPDIR, and a configuration-settable P_tmpdir on Unix-y systems. system_temp_directory just

Hi, llvm::sys::path allows the user to query "known directory" paths: temp, cache, home. I was tweaking this area a bit for some time. I would like to propose a small redesign. What we have so far? home_directory() tries to return what it promises. No issues with that. system_temp_directory(true) returns a path to a directory that can be erased on reboot. On Linux it is usually /tmp.

I have a few concerns: 1. GetTempPathW is buggy, we should be manually using GetEnvironmentVariableW with TMP, TEMP and USERPROFILE arguments and use the first that exists. 2. We should make sure that the platform specific pieces live in their respective Path.inc files. On Thu, Jul 24, 2014 at 9:44 AM, Duncan P. N. Exon Smith < dexonsmith at apple.com> wrote: > > > On

On Jul 25, 2014, at 13:44 , Rafael Espíndola <rafael.espindola at gmail.com> wrote: > On 25 July 2014 16:31, David Majnemer <david.majnemer at gmail.com> wrote: >> I have a few concerns: >> 1. GetTempPathW is buggy, we should be manually using >> GetEnvironmentVariableW with TMP, TEMP and USERPROFILE arguments and use the >> first that exists. >> 2. We

On Mon, Nov 23, 2015 at 3:27 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote: > On 23 November 2015 at 15:11, Aaron Ballman <aaron.ballman at gmail.com> wrote: >> On Mon, Nov 23, 2015 at 3:07 PM, Rafael Espíndola >> <rafael.espindola at gmail.com> wrote: >>>> We appear to use both system_temp_directory(true) and >>>>

On Mon, Nov 23, 2015 at 3:07 PM, Rafael Espíndola <rafael.espindola at gmail.com> wrote: >> We appear to use both system_temp_directory(true) and >> system_temp_directory(false) in ways that seem like they could matter. >> For instance, modules uses a temp directory that does not get erased >> on reboot, possibly for performance reasons. Do we gain something from

> It looks like OS X always has TMPDIR set these days (to the value of > _CS_DARWIN_USER_TEMP_DIR), so I think we should just go with TMPDIR if it's > set and non-empty, and then fall back to confstr. However, we should not be > using TMPDIR if ErasedOnReboot is false (i.e. if we'd prefer > _CS_DARWIN_USER_CACHE_DIR). Unfortunately there's no canonical environment >

> Well, that changes behavior on Linux; previously it did honor TMPDIR when > looking for a cache directory. Is that okay? (The Windows version is also > honoring TEMP in this case.) I *think* that is a bug fix. I don't have those env variables set on my linux machine (fedora 20), but I don't remember seeing them pointing to anything but /tmp in the past and that is even in

On 25 July 2014 17:57, David Majnemer <david.majnemer at gmail.com> wrote: > To be clear, I don't think we need both; I'd like for us to drop one of > them. I just want to make sure that that the final state, post merge, works > well on all of our platforms. :) Cool. The attached patch removes TempDir. It passes all tests, but the OS X case looks a bit suspicious. I first

I'm trying to install DAZ Studio under Wine and immediately after the splash screen it crashes out and I get a window with the following message: > Error creating directory C:/Program Files/***unknown variable system_temp_directory*** The DAZ Studio version is 3.0.1.135 for Windows. I'm running Wine 1.1.12 under Ubuntu 9.04 with GNOME 2.26.1. My video card is an nVidia GeForce

Hi Jake, About a year ago in commit 5049c3422d26b2b68877307c41b35d7e6aae3235, you attempted to solve a race in llvm-objcopy. What was the race? I ask because unless "last change wins" is the result you want, then the race isn't solved. The problem is that `sys::fs::rename` is just a thin wrapper around POSIX semantics, and replacing an existing file is not an error. Dave