Rafael Espíndola
2014-Aug-14 16:47 UTC
[LLVMdev] sys::path::system_temp_directory vs. sys::fs::createTemporaryFile
> 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 tmpfs these days.> What are we using the cache dir for besides the module cache?Only that. There is one call in lvm and 2 in clang-tools-extra, but they both pass true to erasedOnReboot.> JordanCheers, Rafael
Rafael Espíndola
2014-Aug-19 16:08 UTC
[LLVMdev] sys::path::system_temp_directory vs. sys::fs::createTemporaryFile
ping. Rebased version attached. On 14 August 2014 12:47, Rafael Espíndola <rafael.espindola at gmail.com> wrote:>> 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 tmpfs > these days. > >> What are we using the cache dir for besides the module cache? > > Only that. There is one call in lvm and 2 in clang-tools-extra, but > they both pass true to erasedOnReboot. > >> Jordan > > Cheers, > Rafael-------------- next part -------------- A non-text attachment was scrubbed... Name: t.patch Type: text/x-patch Size: 7578 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140819/5a407de8/attachment.bin>
Rafael Espíndola
2014-Aug-25 19:25 UTC
[LLVMdev] sys::path::system_temp_directory vs. sys::fs::createTemporaryFile
Ping2. Rebased patch attached. On 19 August 2014 12:08, Rafael Espíndola <rafael.espindola at gmail.com> wrote:> ping. Rebased version attached. > > On 14 August 2014 12:47, Rafael Espíndola <rafael.espindola at gmail.com> wrote: >>> 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 tmpfs >> these days. >> >>> What are we using the cache dir for besides the module cache? >> >> Only that. There is one call in lvm and 2 in clang-tools-extra, but >> they both pass true to erasedOnReboot. >> >>> Jordan >> >> Cheers, >> Rafael-------------- next part -------------- A non-text attachment was scrubbed... Name: t.patch Type: text/x-patch Size: 7578 bytes Desc: not available URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20140825/7dc1f481/attachment.bin>