Page MenuHomePhabricator
Create Task
Maniphest T17199

[dashboard] mitk::NavigationToolStorageSerializer uses directory without write permission on windows continous dart client
Closed, WontfixPublic
Actions

Description

The error message is: "Could not write tool to tempory directory: C:\Users\admin\AppData\Local\Temp\/8oEMsd"

Examples:

http://cdash.mitk.org/testDetails.php?test=7443117&build=411116

and

http://cdash.mitk.org/testDetails.php?test=7444511&build=411165

This bug was found by tests which use this method. The first step would be to extend the test of mitk::IOUtil itself, so that it detects the bug as well.

Related Objects
Search...

Event Timeline

franza added projects: MITK, Bugzilla.Feb 13 2014, 5:16 PM
• git-user added a subscriber: • git-user.Feb 13 2014, 5:22 PM

User franza has pushed new remote branch:

bug-17199-IOUtil-TemporaryDirectoryWithoutWritePermission
franza added a comment.Feb 13 2014, 5:24 PM

[6bbba2]: Merge branch 'bug-17199-IOUtil-TemporaryDirectoryWithoutWritePermissio

Merged commits:

2014-02-13 17:22:32 Alfred Franz [0136f1]
COMP(#17199): temporarily deactivated tests because of a bug in mitk::IOUtil (17199)

franza added a comment.Feb 13 2014, 5:32 PM

Last comments were wrong. The temporary directory is not from mitk::IOUtil but from mitkStandardFileLocations. In fact, since mitkStandardFileLocations should not be used anymore, using mitk::IOUtil instead might fix is bug

• git-user added a comment.Feb 13 2014, 5:53 PM

User franza has pushed new remote branch:

bug-17199-FixTempDirectory
• git-user added a comment.Feb 19 2014, 4:43 PM

User franza has pushed new remote branch:

bug-17199-TempDirectoryFixIntegration
franza added a comment.Feb 19 2014, 4:45 PM

[a14e43]: Merge branch 'bug-17199-TempDirectoryFixIntegration'

Merged commits:

2014-02-19 16:43:20 Alfred Franz [b61211]
activated tests again

2014-02-19 16:42:23 Alfred Franz [2b263f]
Merge branch 'bug-17199-FixTempDirectory' into bug-17199-TempDirectoryFixIntegration

2014-02-19 16:37:32 Alfred Franz [065808]
renamed some variables

2014-02-19 16:36:16 Alfred Franz [2a0931]
fixed test ... (no idea why order of method calls is important in this case...)

2014-02-19 16:20:38 Alfred Franz [b23605]
added another check to the setup method

2014-02-19 16:19:45 Alfred Franz [041276]
navigation tool writer is now also using the temporary directories of IOUtil

2014-02-13 17:53:12 Alfred Franz [398fef]
changed the temp directory to mitk::IOUtil::CreateTemporaryDirectory(), but it does not work yet

franza added a comment.Feb 25 2014, 10:17 AM

Test failed again, see http://cdash.mitk.org/testDetails.php?test=7569768&build=416542

This fail might be caused by T17310.

Try to replace the method mitk::IOUtil::CreateTemporaryFile() as described in T17310, this might help.

franza added a comment.Feb 26 2014, 4:27 PM

[2fac5a]: Merge branch 'bug-17199-SerializerTempDirectory'

Merged commits:

2014-02-26 16:25:08 Alfred Franz [5f7002]
Changed temp directory to fix random fails on dart clients

franza added a comment.Feb 26 2014, 4:33 PM

[b39751]: Merge branch 'bug-17199-SerializerTempDirectory'

Merged commits:

2014-02-26 16:31:33 Alfred Franz [b860ca]
also changed name and path of the temporary file of the test

• git-user added a comment.Apr 14 2014, 11:21 AM

User franza has pushed new remote branch:

bug-17199-SerializerTempDirectoryOldBranch
kislinsk edited projects, added MITK (2016-11); removed MITK.Aug 1 2016, 10:48 PM
kislinsk moved this task from Backlog to IGT on the MITK (2016-11) board.Oct 28 2016, 10:06 AM
kislinsk edited projects, added MITK; removed MITK (2016-11).Feb 6 2017, 11:24 AM
kislinsk closed this task as Wontfix.May 26 2020, 11:16 AM
kislinsk added a project: Auto-closed.
kislinsk added a subscriber: kislinsk.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs

MITK · MIC · IMSY · DKFZ