D17289: KDevelop/Shell: set dedicated TMPDIR

[Pino Toscano]

pino added a comment.


  In D17289#432537 <https://phabricator.kde.org/D17289#432537>, @rjvbb wrote:
  
  > >   OK, I stand corrected on this. OTOH, the rest of my notes about this being wrong anyway still stand.
  >
  > I'm not convinced about those arguments but have no objection either to making the temp dir user-exclusive - it's a detail that should be easy enough. There's probably a reason I'm not using QTemporaryDir though
  
  
  Creating a temporary directory in a race-free way is what mkdtemp <http://pubs.opengroup.org/onlinepubs/9699919799/functions/mkdtemp.html> is for, and it is wrapped by QTemporaryDir. Just creating a temporary directory like `kdevelop.XXXXXXXX` is really enough, no need for making it "deterministic" in any way. There is no benefit in doing that.
  
  There are no other arguments in the whole blob of conversation of this RR that justify //not// using QTemporaryDir. So yes, again, please do use QTemporaryDir here.

REPOSITORY
  R32 KDevelop

REVISION DETAIL
  https://phabricator.kde.org/D17289

To: rjvbb, #kdevelop, kfunk, mwolff, pino
Cc: aaronpuchert, mwolff, pino, kfunk, kdevelop-devel, gennad, glebaccon, domson, antismap, iodelay, alexeymin, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190317/1e6a2935/attachment-0001.html>