Cyp changed blocking which not transferred by tractive

Cyp changed blockedby which not transferred by tractive

Cyp commented

If removing the "/tmp/" and running "src/warzone2100 --crash", the dump appears in the working directory.

pabs commented

Hmmm, I thought mkstemp would be smarter than that.

The options appear to be:

Switch to ~/.warzone2100-$v/crash-dumps. This has the advantage that next time the game starts you can automatically ask the user if they want to submit the crash dumps they have.

chdir to TMP, TMPDIR, TEMP, TEMPDIR or /tmp before mkstemp.

Use tempnam with a loop and exclusive locks to prevent security issues.

Per commented

Use QDir::tempPath()? See http://qt-project.org/doc/qt-4.8/qdir.html#tempPath

pabs commented

QDir::tempPath() sounds like a good choice in the interim.

I would still suggest to switch to a dedicated directory since it has a number of advantages, but thats a topic for another ticket I guess.

clueless commented

With 3.1.1, the crash file for windows machines are in the configuration directory called logs and on Ubuntu they are still in /tmp/
Is there a reason why windows uses the configuration directory, and not for linux?

vexed changed status from new to closed

vexed changed owner from `` to vexed

vexed changed resolution from `` to fixed

vexed commented

In Warzone2100/warzone2100@ff1ac37:

#CommitTicketReference repository="" revision="ff1ac373abec15ca4095a82b76d0e584741a15d4"
Use our configdir (which is the only directory WZ should ever write to) for all crash handlers now.

Fixes #3885
refs #4259